lkml.org 
[lkml]   [2016]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 2/3] thermal: hisilicon: fix for dependency
    From
    Date
    On Sat, 2016-11-12 at 20:05 +0800, Leo Yan wrote:
    > Hi Rui, Eduardo,
    >
    > On Wed, Aug 31, 2016 at 04:50:16PM +0800, Leo Yan wrote:
    > >
    > > The thermal driver is standalone driver which is used to enable
    > > thermal sensors, so it can be used with any cooling device and
    > > should not bind with CPU cooling device driver.
    > >
    > > This original patch is suggested by Amit Kucheria; so it's to
    > > polish the dependency in Kconfig, and remove the dependency with
    > > CPU_THERMAL.
    > Could you help review this patch? Or need me resend this patch? Sorry
    > I have not tracked this patches well before, this is one missed
    > patch for 96board Hikey.
    >
    as it still applies cleanly, the patch is queued for 4.10.

    thanks,
    rui
    > Thanks,
    > Leo Yan
    >
    > >
    > > Signed-off-by: Leo Yan <leo.yan@linaro.org>
    > > ---
    > >  drivers/thermal/Kconfig | 4 +++-
    > >  1 file changed, 3 insertions(+), 1 deletion(-)
    > >
    > > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
    > > index 2d702ca..91ebab3 100644
    > > --- a/drivers/thermal/Kconfig
    > > +++ b/drivers/thermal/Kconfig
    > > @@ -177,8 +177,10 @@ config THERMAL_EMULATION
    > >  
    > >  config HISI_THERMAL
    > >   tristate "Hisilicon thermal driver"
    > > - depends on (ARCH_HISI && CPU_THERMAL && OF) ||
    > > COMPILE_TEST
    > > + depends on ARCH_HISI || COMPILE_TEST
    > >   depends on HAS_IOMEM
    > > + depends on OF
    > > + default y
    > >   help
    > >     Enable this to plug hisilicon's thermal sensor driver
    > > into the Linux
    > >     thermal framework. cpufreq is used as the cooling device
    > > to throttle

    \
     
     \ /
      Last update: 2016-11-15 13:26    [W:4.477 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site