linux-kernel - Re: [PATCH V6 1/2] Thermal: Add ST-Ericsson DB8500 thermal driver.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <50A8F0DF.8040509@gmail.com>
Date:	Sun, 18 Nov 2012 15:29:51 +0100
From:	Francesco Lavra <francescolavra.fl@...il.com>
To:	Zhang Rui <rui.zhang@...el.com>
CC:	"hongbo.zhang" <hongbo.zhang@...aro.org>,
	linaro-kernel@...ts.linaro.org, linaro-dev@...ts.linaro.org,
	patches@...aro.org, linux-pm@...r.kernel.org,
	linux-kernel@...r.kernel.org, amit.kachhap@...aro.org,
	STEricsson_nomadik_linux@...t.st.com, kernel@...oocommunity.org,
	"hongbo.zhang" <hongbo.zhang@...aro.com>
Subject: Re: [PATCH V6 1/2] Thermal: Add ST-Ericsson DB8500 thermal driver.

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@...aro.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@...aro.com>
>> Reviewed-by: Viresh Kumar <viresh.kumar@...aro.org>
>> Reviewed-by: Francesco Lavra <francescolavra.fl@...il.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?

> +	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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/