LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] spmi: pmic_arb: enable build on arm64 platforms
@ 2015-02-03 13:50 Stanimir Varbanov
2015-02-03 20:42 ` Paul Bolle
0 siblings, 1 reply; 9+ messages in thread
From: Stanimir Varbanov @ 2015-02-03 13:50 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Kumar Gala, Stephen Boyd, linux-kernel, linux-arm-msm, Stanimir Varbanov
This enables pmic arbiter driver to be build on arm64
platforms.
Signed-off-by: Stanimir Varbanov <svarbanov@mm-sol.com>
---
drivers/spmi/Kconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
index bf1295e..115348c 100644
--- a/drivers/spmi/Kconfig
+++ b/drivers/spmi/Kconfig
@@ -12,7 +12,7 @@ if SPMI
config SPMI_MSM_PMIC_ARB
tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
- depends on ARM
+ depends on ARM || ARM64
depends on IRQ_DOMAIN
depends on ARCH_QCOM || COMPILE_TEST
default ARCH_QCOM
--
1.7.0.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-03 13:50 [PATCH] spmi: pmic_arb: enable build on arm64 platforms Stanimir Varbanov
@ 2015-02-03 20:42 ` Paul Bolle
2015-02-04 15:05 ` Stanimir Varbanov
0 siblings, 1 reply; 9+ messages in thread
From: Paul Bolle @ 2015-02-03 20:42 UTC (permalink / raw)
To: Stanimir Varbanov
Cc: Greg Kroah-Hartman, Kumar Gala, Stephen Boyd, linux-kernel,
linux-arm-msm
On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
> This enables pmic arbiter driver to be build on arm64
> platforms.
>
> Signed-off-by: Stanimir Varbanov <svarbanov@mm-sol.com>
> ---
> drivers/spmi/Kconfig | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
> index bf1295e..115348c 100644
> --- a/drivers/spmi/Kconfig
> +++ b/drivers/spmi/Kconfig
> @@ -12,7 +12,7 @@ if SPMI
>
> config SPMI_MSM_PMIC_ARB
> tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
> - depends on ARM
> + depends on ARM || ARM64
> depends on IRQ_DOMAIN
> depends on ARCH_QCOM || COMPILE_TEST
> default ARCH_QCOM
But only if COMPILE_TEST is set too, isn't it? Is that intended?
Paul Bolle
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-03 20:42 ` Paul Bolle
@ 2015-02-04 15:05 ` Stanimir Varbanov
2015-02-04 15:14 ` Paul Bolle
0 siblings, 1 reply; 9+ messages in thread
From: Stanimir Varbanov @ 2015-02-04 15:05 UTC (permalink / raw)
To: Paul Bolle
Cc: Greg Kroah-Hartman, Kumar Gala, Stephen Boyd, linux-kernel,
linux-arm-msm
On 02/03/2015 10:42 PM, Paul Bolle wrote:
> On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
>> This enables pmic arbiter driver to be build on arm64
>> platforms.
>>
>> Signed-off-by: Stanimir Varbanov <svarbanov@mm-sol.com>
>> ---
>> drivers/spmi/Kconfig | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
>> index bf1295e..115348c 100644
>> --- a/drivers/spmi/Kconfig
>> +++ b/drivers/spmi/Kconfig
>> @@ -12,7 +12,7 @@ if SPMI
>>
>> config SPMI_MSM_PMIC_ARB
>> tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
>> - depends on ARM
>> + depends on ARM || ARM64
>> depends on IRQ_DOMAIN
>> depends on ARCH_QCOM || COMPILE_TEST
>> default ARCH_QCOM
>
> But only if COMPILE_TEST is set too, isn't it? Is that intended?
No, the driver will be used on arm64 qcom platforms, also.
--
regards,
Stan
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-04 15:05 ` Stanimir Varbanov
@ 2015-02-04 15:14 ` Paul Bolle
2015-02-04 15:27 ` Stanimir Varbanov
0 siblings, 1 reply; 9+ messages in thread
From: Paul Bolle @ 2015-02-04 15:14 UTC (permalink / raw)
To: Stanimir Varbanov
Cc: Greg Kroah-Hartman, Kumar Gala, Stephen Boyd, linux-kernel,
linux-arm-msm
On Wed, 2015-02-04 at 17:05 +0200, Stanimir Varbanov wrote:
> On 02/03/2015 10:42 PM, Paul Bolle wrote:
> > On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
> >> diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
> >> index bf1295e..115348c 100644
> >> --- a/drivers/spmi/Kconfig
> >> +++ b/drivers/spmi/Kconfig
> >> @@ -12,7 +12,7 @@ if SPMI
> >>
> >> config SPMI_MSM_PMIC_ARB
> >> tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
> >> - depends on ARM
> >> + depends on ARM || ARM64
> >> depends on IRQ_DOMAIN
> >> depends on ARCH_QCOM || COMPILE_TEST
> >> default ARCH_QCOM
> >
> > But only if COMPILE_TEST is set too, isn't it? Is that intended?
>
> No, the driver will be used on arm64 qcom platforms, also.
But ARCH_QCOM currently is (32 bit) arch/arm only, isn't it?
Paul Bolle
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-04 15:14 ` Paul Bolle
@ 2015-02-04 15:27 ` Stanimir Varbanov
2015-02-04 18:46 ` Paul Bolle
0 siblings, 1 reply; 9+ messages in thread
From: Stanimir Varbanov @ 2015-02-04 15:27 UTC (permalink / raw)
To: Paul Bolle
Cc: Greg Kroah-Hartman, Kumar Gala, Stephen Boyd, linux-kernel,
linux-arm-msm
On 02/04/2015 05:14 PM, Paul Bolle wrote:
> On Wed, 2015-02-04 at 17:05 +0200, Stanimir Varbanov wrote:
>> On 02/03/2015 10:42 PM, Paul Bolle wrote:
>>> On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
>>>> diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
>>>> index bf1295e..115348c 100644
>>>> --- a/drivers/spmi/Kconfig
>>>> +++ b/drivers/spmi/Kconfig
>>>> @@ -12,7 +12,7 @@ if SPMI
>>>>
>>>> config SPMI_MSM_PMIC_ARB
>>>> tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
>>>> - depends on ARM
>>>> + depends on ARM || ARM64
>>>> depends on IRQ_DOMAIN
>>>> depends on ARCH_QCOM || COMPILE_TEST
>>>> default ARCH_QCOM
>>>
>>> But only if COMPILE_TEST is set too, isn't it? Is that intended?
>>
>> No, the driver will be used on arm64 qcom platforms, also.
>
> But ARCH_QCOM currently is (32 bit) arch/arm only, isn't it?
Currently yes, but we want to be prepared to support PMIC's which will
be used on arm64 platforms, although they are not upstream yet.
--
regards,
Stan
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-04 15:27 ` Stanimir Varbanov
@ 2015-02-04 18:46 ` Paul Bolle
2015-02-23 12:57 ` Ivan T. Ivanov
0 siblings, 1 reply; 9+ messages in thread
From: Paul Bolle @ 2015-02-04 18:46 UTC (permalink / raw)
To: Stanimir Varbanov
Cc: Greg Kroah-Hartman, Kumar Gala, Stephen Boyd, linux-kernel,
linux-arm-msm
On Wed, 2015-02-04 at 17:27 +0200, Stanimir Varbanov wrote:
> On 02/04/2015 05:14 PM, Paul Bolle wrote:
> > On Wed, 2015-02-04 at 17:05 +0200, Stanimir Varbanov wrote:
> >> On 02/03/2015 10:42 PM, Paul Bolle wrote:
> >>> On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
> >>>> diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
> >>>> index bf1295e..115348c 100644
> >>>> --- a/drivers/spmi/Kconfig
> >>>> +++ b/drivers/spmi/Kconfig
> >>>> @@ -12,7 +12,7 @@ if SPMI
> >>>>
> >>>> config SPMI_MSM_PMIC_ARB
> >>>> tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
> >>>> - depends on ARM
> >>>> + depends on ARM || ARM64
> >>>> depends on IRQ_DOMAIN
> >>>> depends on ARCH_QCOM || COMPILE_TEST
> >>>> default ARCH_QCOM
> >>>
> >>> But only if COMPILE_TEST is set too, isn't it? Is that intended?
> >>
> >> No, the driver will be used on arm64 qcom platforms, also.
> >
> > But ARCH_QCOM currently is (32 bit) arch/arm only, isn't it?
>
> Currently yes, but we want to be prepared to support PMIC's which will
> be used on arm64 platforms, although they are not upstream yet.
Please make the commit summary reflect that, as in:
"spmi: pmic_arb: enable test build on arm64"
Commit explanation likewise.
Paul Bolle
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-04 18:46 ` Paul Bolle
@ 2015-02-23 12:57 ` Ivan T. Ivanov
2015-03-09 16:53 ` Stanimir Varbanov
0 siblings, 1 reply; 9+ messages in thread
From: Ivan T. Ivanov @ 2015-02-23 12:57 UTC (permalink / raw)
To: Paul Bolle
Cc: Stanimir Varbanov, Greg Kroah-Hartman, Kumar Gala, Stephen Boyd,
linux-kernel, linux-arm-msm
On Wed, 2015-02-04 at 19:46 +0100, Paul Bolle wrote:
> On Wed, 2015-02-04 at 17:27 +0200, Stanimir Varbanov wrote:
> > On 02/04/2015 05:14 PM, Paul Bolle wrote:
> > > On Wed, 2015-02-04 at 17:05 +0200, Stanimir Varbanov wrote:
> > > > On 02/03/2015 10:42 PM, Paul Bolle wrote:
> > > > > On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
> > > > > > diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
> > > > > > index bf1295e..115348c 100644
> > > > > > --- a/drivers/spmi/Kconfig
> > > > > > +++ b/drivers/spmi/Kconfig
> > > > > > @@ -12,7 +12,7 @@ if SPMI
> > > > > >
> > > > > > config SPMI_MSM_PMIC_ARB
> > > > > > tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
> > > > > > - depends on ARM
> > > > > > + depends on ARM || ARM64
> > > > > > depends on IRQ_DOMAIN
> > > > > > depends on ARCH_QCOM || COMPILE_TEST
> > > > > > default ARCH_QCOM
> > > > >
> > > > > But only if COMPILE_TEST is set too, isn't it? Is that intended?
> > > >
> > > > No, the driver will be used on arm64 qcom platforms, also.
> > >
> > > But ARCH_QCOM currently is (32 bit) arch/arm only, isn't it?
> >
> > Currently yes, but we want to be prepared to support PMIC's which will
> > be used on arm64 platforms, although they are not upstream yet.
>
> Please make the commit summary reflect that, as in:
> "spmi: pmic_arb: enable test build on arm64"
>
> Commit explanation likewise.
What if we just drop ARM dependency?
Regards,
Ivan
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-02-23 12:57 ` Ivan T. Ivanov
@ 2015-03-09 16:53 ` Stanimir Varbanov
2015-03-09 18:57 ` Paul Bolle
0 siblings, 1 reply; 9+ messages in thread
From: Stanimir Varbanov @ 2015-03-09 16:53 UTC (permalink / raw)
To: Ivan T. Ivanov, Paul Bolle
Cc: Greg Kroah-Hartman, Kumar Gala, Stephen Boyd, linux-kernel,
linux-arm-msm
On 02/23/2015 02:57 PM, Ivan T. Ivanov wrote:
>
> On Wed, 2015-02-04 at 19:46 +0100, Paul Bolle wrote:
>> On Wed, 2015-02-04 at 17:27 +0200, Stanimir Varbanov wrote:
>>> On 02/04/2015 05:14 PM, Paul Bolle wrote:
>>>> On Wed, 2015-02-04 at 17:05 +0200, Stanimir Varbanov wrote:
>>>>> On 02/03/2015 10:42 PM, Paul Bolle wrote:
>>>>>> On Tue, 2015-02-03 at 15:50 +0200, Stanimir Varbanov wrote:
>>>>>>> diff --git a/drivers/spmi/Kconfig b/drivers/spmi/Kconfig
>>>>>>> index bf1295e..115348c 100644
>>>>>>> --- a/drivers/spmi/Kconfig
>>>>>>> +++ b/drivers/spmi/Kconfig
>>>>>>> @@ -12,7 +12,7 @@ if SPMI
>>>>>>>
>>>>>>> config SPMI_MSM_PMIC_ARB
>>>>>>> tristate "Qualcomm MSM SPMI Controller (PMIC Arbiter)"
>>>>>>> - depends on ARM
>>>>>>> + depends on ARM || ARM64
>>>>>>> depends on IRQ_DOMAIN
>>>>>>> depends on ARCH_QCOM || COMPILE_TEST
>>>>>>> default ARCH_QCOM
>>>>>>
>>>>>> But only if COMPILE_TEST is set too, isn't it? Is that intended?
>>>>>
>>>>> No, the driver will be used on arm64 qcom platforms, also.
>>>>
>>>> But ARCH_QCOM currently is (32 bit) arch/arm only, isn't it?
>>>
>>> Currently yes, but we want to be prepared to support PMIC's which will
>>> be used on arm64 platforms, although they are not upstream yet.
>>
>> Please make the commit summary reflect that, as in:
>> "spmi: pmic_arb: enable test build on arm64"
>>
>> Commit explanation likewise.
>
> What if we just drop ARM dependency?
I'm fine with this, Paul?
--
regards,
Stan
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spmi: pmic_arb: enable build on arm64 platforms
2015-03-09 16:53 ` Stanimir Varbanov
@ 2015-03-09 18:57 ` Paul Bolle
0 siblings, 0 replies; 9+ messages in thread
From: Paul Bolle @ 2015-03-09 18:57 UTC (permalink / raw)
To: Stanimir Varbanov
Cc: Ivan T. Ivanov, Greg Kroah-Hartman, Kumar Gala, Stephen Boyd,
linux-kernel, linux-arm-msm
Stanimir Varbanov schreef op ma 09-03-2015 om 18:53 [+0200]:
> On 02/23/2015 02:57 PM, Ivan T. Ivanov wrote:
> > What if we just drop ARM dependency?
>
> I'm fine with this, Paul?
Well I'm fine with anything you do, as long as the Kconfig dependencies
make sense and the commit summary and the commit explanation match the
change. In other words: if you submit an updated patch and I don't bark
you may assume I'm fine with the change. This doesn't need an Ack on my
side anyhow (as if anything does).
Thanks,
Paul Bolle
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-03-09 18:57 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-03 13:50 [PATCH] spmi: pmic_arb: enable build on arm64 platforms Stanimir Varbanov
2015-02-03 20:42 ` Paul Bolle
2015-02-04 15:05 ` Stanimir Varbanov
2015-02-04 15:14 ` Paul Bolle
2015-02-04 15:27 ` Stanimir Varbanov
2015-02-04 18:46 ` Paul Bolle
2015-02-23 12:57 ` Ivan T. Ivanov
2015-03-09 16:53 ` Stanimir Varbanov
2015-03-09 18:57 ` Paul Bolle
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).