All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhang Rui <rui.zhang@intel.com>
To: Francesco Lavra <francescolavra.fl@gmail.com>
Cc: "hongbo.zhang" <hongbo.zhang@linaro.org>,
	linaro-kernel@lists.linaro.org, linaro-dev@lists.linaro.org,
	patches@linaro.org, linux-pm@vger.kernel.org,
	linux-kernel@vger.kernel.org, amit.kachhap@linaro.org,
	STEricsson_nomadik_linux@list.st.com, kernel@igloocommunity.org,
	"hongbo.zhang" <hongbo.zhang@linaro.com>
Subject: Re: [PATCH V6 1/2] Thermal: Add ST-Ericsson DB8500 thermal driver.
Date: Mon, 19 Nov 2012 08:42:45 +0800	[thread overview]
Message-ID: <1353285765.2204.2.camel@rzhang1-mobl4> (raw)
In-Reply-To: <50A8F0DF.8040509@gmail.com>

On Sun, 2012-11-18 at 15:29 +0100, Francesco Lavra wrote:
> On 11/15/2012 01:51 PM, Zhang Rui wrote:
> > On Thu, 2012-11-15 at 18:56 +0800, hongbo.zhang wrote:
> >> From: "hongbo.zhang" <hongbo.zhang@linaro.com>
> >>
> >> This driver is based on the thermal management framework in thermal_sys.c. A
> >> thermal zone device is created with the trip points to which cooling devices
> >> can be bound, the current cooling device is cpufreq, e.g. CPU frequency is
> >> clipped down to cool the CPU, and other cooling devices can be added and bound
> >> to the trip points dynamically.  The platform specific PRCMU interrupts are
> >> used to active thermal update when trip points are reached.
> >>
> >> Signed-off-by: hongbo.zhang <hongbo.zhang@linaro.com>
> >> Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
> >> Reviewed-by: Francesco Lavra <francescolavra.fl@gmail.com>
> > 
> > Patch is refreshed and applied to thermal next.
> > refreshed patch attached.
> [...]
> > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> > index 99b6587..d96da07 100644
> > --- a/drivers/thermal/Kconfig
> > +++ b/drivers/thermal/Kconfig
> > @@ -101,5 +101,25 @@ config EXYNOS_THERMAL
> >  	  If you say yes here you get support for TMU (Thermal Managment
> >  	  Unit) on SAMSUNG EXYNOS series of SoC.
> >  
> > +config DB8500_THERMAL
> > +	bool "DB8500 thermal management"
> > +	depends on ARCH_U8500
> 
> Shouldn't it depend on THERMAL as well, as in Hongbo's original patch?
> 
all of these options are available only if CONFIG_THERMAL is set.
please refer to commit 72e198978223f2020f7f59a6e2520f2b7d005e72 in the
thermal next tree.

thanks,
rui
> > +	default y
> > +	help
> > +	  Adds DB8500 thermal management implementation according to the thermal
> > +	  management framework. A thermal zone with several trip points will be
> > +	  created. Cooling devices can be bound to the trip points to cool this
> > +	  thermal zone if trip points reached.
> > +
> > +config DB8500_CPUFREQ_COOLING
> > +	tristate "DB8500 cpufreq cooling"
> > +	depends on ARCH_U8500
> > +	depends on CPU_THERMAL
> > +	default y
> > +	help
> > +	  Adds DB8500 cpufreq cooling devices, and these cooling devices can be
> > +	  bound to thermal zone trip points. When a trip point reached, the
> > +	  bound cpufreq cooling device turns active to set CPU frequency low to
> > +	  cool down the CPU.
> >  
> >  endif
> 
> --
> Francesco



  reply	other threads:[~2012-11-19  0:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-15 10:56 [PATCH V6 0/2] Upstream ST-Ericsson " hongbo.zhang
2012-11-15 10:56 ` [PATCH V6 1/2] Thermal: Add ST-Ericsson DB8500 " hongbo.zhang
2012-11-15 10:56   ` hongbo.zhang
2012-11-15 12:51   ` Zhang Rui
2012-11-18 14:29     ` Francesco Lavra
2012-11-18 14:29       ` Francesco Lavra
2012-11-19  0:42       ` Zhang Rui [this message]
2012-11-15 10:56 ` [PATCH V6 2/2] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data hongbo.zhang
2012-11-15 10:56   ` hongbo.zhang
2012-11-15 12:51   ` Zhang Rui

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1353285765.2204.2.camel@rzhang1-mobl4 \
    --to=rui.zhang@intel.com \
    --cc=STEricsson_nomadik_linux@list.st.com \
    --cc=amit.kachhap@linaro.org \
    --cc=francescolavra.fl@gmail.com \
    --cc=hongbo.zhang@linaro.com \
    --cc=hongbo.zhang@linaro.org \
    --cc=kernel@igloocommunity.org \
    --cc=linaro-dev@lists.linaro.org \
    --cc=linaro-kernel@lists.linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=patches@linaro.org \
    --subject='Re: [PATCH V6 1/2] Thermal: Add ST-Ericsson DB8500 thermal driver.' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.