[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20160209221026.GD28102@localhost.localdomain>
Date: Tue, 9 Feb 2016 14:10:27 -0800
From: Eduardo Valentin <edubezval@...il.com>
To: Lee Jones <lee.jones@...aro.org>
Cc: Arnd Bergmann <arnd@...db.de>, Zhang Rui <rui.zhang@...el.com>,
linux-arm-kernel@...ts.infradead.org, linux-pm@...r.kernel.org,
spear-devel@...t.st.com, Linus Walleij <linus.walleij@...aro.org>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
On Wed, Jan 27, 2016 at 07:33:00AM +0000, Lee Jones wrote:
> On Mon, 25 Jan 2016, Arnd Bergmann wrote:
>
> > When the thermal subsystem is a loadable module, the u8500 driver
> > fails to build:
> >
> > drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> > db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> > drivers/thermal/built-in.o: In function `db8500_thermal_work':
> > db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
> >
> > This changes the symbol to a tristate, so Kconfig can track the
> > dependency correctly.
> >
> > Signed-off-by: Arnd Bergmann <arnd@...db.de>
> > ---
> > drivers/mfd/db8500-prcmu.c | 3 +++
> > drivers/thermal/Kconfig | 4 ++--
> > 2 files changed, 5 insertions(+), 2 deletions(-)
>
> As this touches a core Thermal file, I think it's better off going in
> via that tree. If that's a problem, I can take it with the relevant
> Acks.
>
> Acked-by: Lee Jones <lee.jones@...aro.org>
Thanks, I am adding to linux-next to see how it goes.
Download attachment "signature.asc" of type "application/pgp-signature" (474 bytes)
Powered by blists - more mailing lists