Re: [PATCH v2] thermal: allow building dove_thermal with mvebu

[Zhang Rui <rui.zhang@intel.com>]

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
> > >
> >