* [PATCH] thermal: allow building dove_thermal with mvebu @ 2014-05-30 12:28 Riku Voipio 2014-05-31 11:50 ` [PATCH v2] " Riku Voipio 0 siblings, 1 reply; 6+ messages in thread From: Riku Voipio @ 2014-05-30 12:28 UTC (permalink / raw) To: linux-pm Cc: Eduardo Valentin, Riku Voipio, Zhang Rui, linux-arm-kernel, Sebastian Hesselbarth DT-enabled Dove has moved from ARCH_DOVE in mach-dove to MACH_DOVE in mach-mvebu. As non-DT ARCH_DOVE will stay to rot for a while, add a new DT-only MACH_DOVE to thermal Kconfig. This was originally supposed to go in via "ARM: dove: prepare new Dove DT Kconfig" patch from Sebastian Hesselbarth for 3.15, but slipped through the cracks. I've tested on CuBox that without this patch you can't compile dove_thermal into a mach-mvebu based kernel, and with this patch I can build the driver and it works as expected run-time. Signed-off-by: Riku Voipio <riku.voipio@linaro.org> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> Cc: linux-pm@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org --- drivers/thermal/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index 2d51912..5783c72 100644 --- a/drivers/thermal/Kconfig +++ b/drivers/thermal/Kconfig @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL config DOVE_THERMAL tristate "Temperature sensor on Marvell Dove SoCs" - depends on ARCH_DOVE + depends on ARCH_DOVE || MACH_DOVE depends on OF help Support for the Dove thermal sensor driver in the Linux thermal -- 2.0.0.rc2 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2] thermal: allow building dove_thermal with mvebu 2014-05-30 12:28 [PATCH] thermal: allow building dove_thermal with mvebu Riku Voipio @ 2014-05-31 11:50 ` Riku Voipio 2014-05-31 12:54 ` Sebastian Hesselbarth 0 siblings, 1 reply; 6+ messages in thread From: Riku Voipio @ 2014-05-31 11:50 UTC (permalink / raw) To: linux-pm Cc: Riku Voipio, Sebastian Hesselbarth, Zhang Rui, Eduardo Valentin, linux-arm-kernel DT-enabled Dove has moved from ARCH_DOVE in mach-dove to MACH_DOVE in mach-mvebu. As non-DT ARCH_DOVE will stay to rot for a while, add a new DT-only MACH_DOVE to thermal Kconfig. This was originally supposed to go in via "ARM: dove: prepare new Dove DT Kconfig" patch from Sebastian Hesselbarth for 3.15, but slipped through the cracks. I've tested on CuBox that without this patch you can't compile dove_thermal into a mach-mvebu based kernel, and with this patch I can build the driver and it works as expected run-time. v2: non-ascii char creeped in somehow Signed-off-by: Riku Voipio <riku.voipio@linaro.org> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> Cc: linux-pm@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org --- drivers/thermal/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index 2d51912..5783c72 100644 --- a/drivers/thermal/Kconfig +++ b/drivers/thermal/Kconfig @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL config DOVE_THERMAL tristate "Temperature sensor on Marvell Dove SoCs" - depends on ARCH_DOVE + depends on ARCH_DOVE || MACH_DOVE depends on OF help Support for the Dove thermal sensor driver in the Linux thermal -- 2.0.0.rc2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] thermal: allow building dove_thermal with mvebu 2014-05-31 11:50 ` [PATCH v2] " Riku Voipio @ 2014-05-31 12:54 ` Sebastian Hesselbarth [not found] ` <CAAqcGHkcnm_JjyzWewEcdGuGyyBwtDT_6EWxFzKSqS7NEUaerA@mail.gmail.com> 0 siblings, 1 reply; 6+ messages in thread From: Sebastian Hesselbarth @ 2014-05-31 12:54 UTC (permalink / raw) To: Riku Voipio, linux-pm; +Cc: Zhang Rui, Eduardo Valentin, linux-arm-kernel On 05/31/2014 01:50 PM, Riku Voipio wrote: > DT-enabled Dove has moved from ARCH_DOVE in mach-dove to MACH_DOVE > in mach-mvebu. As non-DT ARCH_DOVE will stay to rot for a while, add a new > DT-only MACH_DOVE to thermal Kconfig. > > This was originally supposed to go in via "ARM: dove: prepare new Dove DT Kconfig" > patch from Sebastian Hesselbarth for 3.15, but slipped through the cracks. > > I've tested on CuBox that without this patch you can't compile > dove_thermal into a mach-mvebu based kernel, and with this patch I can > build the driver and it works as expected run-time. Riku, looking through my mails, the corresponding patch should have already been applied [1]. If not, you get my Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> [1] https://lkml.kernel.org/r/1395295065.3571.10.camel@rzhang1-mobl4 > v2: non-ascii char creeped in somehow > > Signed-off-by: Riku Voipio <riku.voipio@linaro.org> > Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: linux-pm@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > --- > drivers/thermal/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig > index 2d51912..5783c72 100644 > --- a/drivers/thermal/Kconfig > +++ b/drivers/thermal/Kconfig > @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL > > config DOVE_THERMAL > tristate "Temperature sensor on Marvell Dove SoCs" > - depends on ARCH_DOVE > + depends on ARCH_DOVE || MACH_DOVE > depends on OF > help > Support for the Dove thermal sensor driver in the Linux thermal > ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CAAqcGHkcnm_JjyzWewEcdGuGyyBwtDT_6EWxFzKSqS7NEUaerA@mail.gmail.com>]
* Re: [PATCH v2] thermal: allow building dove_thermal with mvebu [not found] ` <CAAqcGHkcnm_JjyzWewEcdGuGyyBwtDT_6EWxFzKSqS7NEUaerA@mail.gmail.com> @ 2014-07-01 2:04 ` Zhang Rui 2014-07-01 8:50 ` Sebastian Hesselbarth 0 siblings, 1 reply; 6+ messages in thread From: Zhang Rui @ 2014-07-01 2:04 UTC (permalink / raw) To: Riku Voipio, sebastian.hesselbarth, andrew; +Cc: linux-pm, Eduardo Valentin On Mon, 2014-06-02 at 09:14 +0300, Riku Voipio wrote: > > On 31 May 2014 15:54, Sebastian Hesselbarth > <sebastian.hesselbarth@gmail.com> wrote: > > Riku, > > > looking through my mails, the corresponding patch should have > > already been applied [1]. > > > If not, you get my > > > Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> > > I don't see that patch applied in 3.15-rc8 or linux-next: > > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/thermal/Kconfig#n152 > > I'd like to be sure it gets in 3.16. > Sorry, I missed Sebastian' patch... Now I've seen three versions of patches that do the similar thing. This one, the one from Sebastian and the one in https://patchwork.kernel.org/patch/4445141/ from Andrew. Can anybody tell me the relationship between ARCH_DOVE and MARC_DOVE, and which one should be the proper fix please? Eduardo, I would queue the proper patch for next rc, what's your opinion? thanks, rui > > [1] https://lkml.kernel.org/r/1395295065.3571.10.camel@rzhang1-mobl4 > > > > > v2: non-ascii char creeped in somehow > > > > > > Signed-off-by: Riku Voipio <riku.voipio@linaro.org> > > > Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> > > > Cc: Zhang Rui <rui.zhang@intel.com> > > > Cc: Eduardo Valentin <edubezval@gmail.com> > > > Cc: linux-pm@vger.kernel.org > > > Cc: linux-arm-kernel@lists.infradead.org > > > --- > > > drivers/thermal/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig > > > index 2d51912..5783c72 100644 > > > --- a/drivers/thermal/Kconfig > > > +++ b/drivers/thermal/Kconfig > > > @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL > > > > > > config DOVE_THERMAL > > > tristate "Temperature sensor on Marvell Dove SoCs" > > > - depends on ARCH_DOVE > > > + depends on ARCH_DOVE || MACH_DOVE > > > depends on OF > > > help > > > Support for the Dove thermal sensor driver in the Linux > thermal > > > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] thermal: allow building dove_thermal with mvebu 2014-07-01 2:04 ` Zhang Rui @ 2014-07-01 8:50 ` Sebastian Hesselbarth 2014-07-15 14:00 ` Riku Voipio 0 siblings, 1 reply; 6+ messages in thread From: Sebastian Hesselbarth @ 2014-07-01 8:50 UTC (permalink / raw) To: Zhang Rui, Riku Voipio, andrew; +Cc: linux-pm, Eduardo Valentin On 07/01/2014 04:04 AM, Zhang Rui wrote: > On Mon, 2014-06-02 at 09:14 +0300, Riku Voipio wrote: >> >> On 31 May 2014 15:54, Sebastian Hesselbarth >> <sebastian.hesselbarth@gmail.com> wrote: >>> Riku, >> >>> looking through my mails, the corresponding patch should have >>> already been applied [1]. >> >>> If not, you get my >> >>> Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> >> >> I don't see that patch applied in 3.15-rc8 or linux-next: >> >> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/thermal/Kconfig#n152 >> >> I'd like to be sure it gets in 3.16. >> > Sorry, I missed Sebastian' patch... > Now I've seen three versions of patches that do the similar thing. > This one, the one from Sebastian and the one in > https://patchwork.kernel.org/patch/4445141/ from Andrew. > > Can anybody tell me the relationship between ARCH_DOVE and MARC_DOVE, > and which one should be the proper fix please? Rui, ARCH_DOVE represents legacy, non-DT arch/arm/mach-dove while MACH_DOVE is DT-only Dove in arch/arm/mach-mvebu. The idea was to have both, legacy and DT-only Dove, for some cycles to allow us to finalize the DT transition. Due to a justified request of Russell, we will keep legacy mach-dove for a while longer. The correct patch is the one below, adding both MACH_DOVE and ARCH_DOVE to DOVE_THERMAL. Sebastian > Eduardo, > I would queue the proper patch for next rc, what's your opinion? > >>> [1] https://lkml.kernel.org/r/1395295065.3571.10.camel@rzhang1-mobl4 >>> >>>> v2: non-ascii char creeped in somehow >>>> >>>> Signed-off-by: Riku Voipio <riku.voipio@linaro.org> >>>> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> >>>> Cc: Zhang Rui <rui.zhang@intel.com> >>>> Cc: Eduardo Valentin <edubezval@gmail.com> >>>> Cc: linux-pm@vger.kernel.org >>>> Cc: linux-arm-kernel@lists.infradead.org >>>> --- >>>> drivers/thermal/Kconfig | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig >>>> index 2d51912..5783c72 100644 >>>> --- a/drivers/thermal/Kconfig >>>> +++ b/drivers/thermal/Kconfig >>>> @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL >>>> >>>> config DOVE_THERMAL >>>> tristate "Temperature sensor on Marvell Dove SoCs" >>>> - depends on ARCH_DOVE >>>> + depends on ARCH_DOVE || MACH_DOVE >>>> depends on OF >>>> help >>>> Support for the Dove thermal sensor driver in the Linux >> thermal >>>> >>> > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] thermal: allow building dove_thermal with mvebu 2014-07-01 8:50 ` Sebastian Hesselbarth @ 2014-07-15 14:00 ` Riku Voipio 0 siblings, 0 replies; 6+ messages in thread From: Riku Voipio @ 2014-07-15 14:00 UTC (permalink / raw) To: Sebastian Hesselbarth; +Cc: Zhang Rui, andrew, linux-pm, Eduardo Valentin On 1 July 2014 11:50, Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> wrote: > On 07/01/2014 04:04 AM, Zhang Rui wrote: >> Can anybody tell me the relationship between ARCH_DOVE and MARC_DOVE, >> and which one should be the proper fix please? > Rui, > ARCH_DOVE represents legacy, non-DT arch/arm/mach-dove while > MACH_DOVE is DT-only Dove in arch/arm/mach-mvebu. > The idea was to have both, legacy and DT-only Dove, for some > cycles to allow us to finalize the DT transition. > Due to a justified request of Russell, we will keep legacy > mach-dove for a while longer. > The correct patch is the one below, adding both MACH_DOVE and ARCH_DOVE > to DOVE_THERMAL. Here on patchwork: https://patches.linaro.org/31238/ It would be really nice to get it to 3.16 since Sebastians original patch already slipped from 3.15 :( Riku > Sebastian > >> Eduardo, >> I would queue the proper patch for next rc, what's your opinion? >> >>>> [1] https://lkml.kernel.org/r/1395295065.3571.10.camel@rzhang1-mobl4 >>>> >>>>> v2: non-ascii char creeped in somehow >>>>> >>>>> Signed-off-by: Riku Voipio <riku.voipio@linaro.org> >>>>> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> >>>>> Cc: Zhang Rui <rui.zhang@intel.com> >>>>> Cc: Eduardo Valentin <edubezval@gmail.com> >>>>> Cc: linux-pm@vger.kernel.org >>>>> Cc: linux-arm-kernel@lists.infradead.org >>>>> --- >>>>> drivers/thermal/Kconfig | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig >>>>> index 2d51912..5783c72 100644 >>>>> --- a/drivers/thermal/Kconfig >>>>> +++ b/drivers/thermal/Kconfig >>>>> @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL >>>>> >>>>> config DOVE_THERMAL >>>>> tristate "Temperature sensor on Marvell Dove SoCs" >>>>> - depends on ARCH_DOVE >>>>> + depends on ARCH_DOVE || MACH_DOVE >>>>> depends on OF >>>>> help >>>>> Support for the Dove thermal sensor driver in the Linux >>> >>> thermal >>>>> >>>>> >>>> >> >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-15 14:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-30 12:28 [PATCH] thermal: allow building dove_thermal with mvebu Riku Voipio
2014-05-31 11:50 ` [PATCH v2] " Riku Voipio
2014-05-31 12:54 ` Sebastian Hesselbarth
[not found] ` <CAAqcGHkcnm_JjyzWewEcdGuGyyBwtDT_6EWxFzKSqS7NEUaerA@mail.gmail.com>
2014-07-01 2:04 ` Zhang Rui
2014-07-01 8:50 ` Sebastian Hesselbarth
2014-07-15 14:00 ` Riku Voipio
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).