From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 6/8] ARM: sun8i: a33: add CPU thermal throttling
Date: Wed, 5 Apr 2017 14:13:10 +0200 [thread overview]
Message-ID: <20170405121310.4lgf5st5oqc52szb@lukather> (raw)
In-Reply-To: <20170405090634.4649-7-quentin.schulz@free-electrons.com>
On Wed, Apr 05, 2017 at 11:06:32AM +0200, Quentin Schulz wrote:
> This adds CPU thermal throttling for the Allwinner A33. It uses the
> thermal sensor present in the SoC's GPADC.
>
> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>
> ---
>
> v3:
> - switched to new phandle because of modified DT node name for the GPADC
> (named THS),
> - got rid of cooling-min-level and cooling-max-level as it's not used in any
> code in the kernel,
>
> v2:
> - updated cooling-max-level to reflect newly added OPPs,
>
> arch/arm/boot/dts/sun8i-a33.dtsi | 45 ++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 45 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
> index 9734e63..b88c107 100644
> --- a/arch/arm/boot/dts/sun8i-a33.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a33.dtsi
> @@ -43,6 +43,7 @@
> */
>
> #include "sun8i-a23-a33.dtsi"
> +#include <dt-bindings/thermal/thermal.h>
>
> / {
> cpu0_opp_table: opp_table0 {
> @@ -127,6 +128,7 @@
> clocks = <&ccu CLK_CPUX>;
> clock-names = "cpu";
> operating-points-v2 = <&cpu0_opp_table>;
> + #cooling-cells = <2>;
> };
>
> cpu at 1 {
> @@ -170,6 +172,49 @@
> };
> };
>
> + thermal-zones {
> + cpu_thermal {
> + /* milliseconds */
> + polling-delay-passive = <250>;
> + polling-delay = <1000>;
> + thermal-sensors = <&ths>;
> +
> + cooling-maps {
> + map0 {
> + trip = <&cpu_alert0>;
> + cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> + };
> + map1 {
> + trip = <&cpu_alert1>;
> + cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> + };
> + };
> +
> + trips {
> + cpu_alert0: cpu_alert0 {
> + /* milliCelsius */
> + temperature = <75000>;
> + hysteresis = <2000>;
> + type = "passive";
> + };
> +
> + cpu_alert1: cpu_alert1 {
> + /* milliCelsius */
> + temperature = <90000>;
> + hysteresis = <2000>;
> + type = "hot";
> + };
> +
> + cpu_crit: cpu_crit {
> + /* milliCelsius */
> + temperature = <110000>;
> + hysteresis = <2000>;
> + type = "critical";
> + };
> + };
> + };
> + };
> +
This wasn't sorted properly (thermal is not between mali- and memory
in the alphabetical order). Fixed and applied, thanks!
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170405/6d886d1a/attachment.sig>
next prev parent reply other threads:[~2017-04-05 12:13 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-05 9:06 [PATCH v4 0/8] add thermal throttling to Allwinner A33 SoC Quentin Schulz
2017-04-05 9:06 ` [PATCH v4 1/8] dt-bindings: mfd: add A33 GPADC binding Quentin Schulz
2017-04-05 12:11 ` Lee Jones
2017-04-05 9:06 ` [PATCH v4 2/8] dt-bindings: input: touschcreen: remove sun4i documentation Quentin Schulz
2017-04-05 12:11 ` Lee Jones
2017-04-05 9:06 ` [PATCH v4 3/8] iio: adc: sun4i-gpadc-iio: move code used in MFD probing to new function Quentin Schulz
2017-04-08 17:26 ` Jonathan Cameron
2017-04-05 9:06 ` [PATCH v4 4/8] iio: adc: sun4i-gpadc-iio: add support for A33 thermal sensor Quentin Schulz
2017-04-08 16:36 ` Jonathan Cameron
2017-04-05 9:06 ` [PATCH v4 5/8] ARM: sun8i: a33: add " Quentin Schulz
2017-04-05 12:12 ` Maxime Ripard
2017-04-05 9:06 ` [PATCH v4 6/8] ARM: sun8i: a33: add CPU thermal throttling Quentin Schulz
2017-04-05 12:13 ` Maxime Ripard [this message]
2017-04-05 9:06 ` [PATCH v4 7/8] ARM: sun8i: a33: Add devfreq-based GPU cooling Quentin Schulz
2017-04-05 12:16 ` Maxime Ripard
2017-04-05 9:06 ` [PATCH v4 8/8] ARM: sun8i: sina33: add highest OPP of CPUs Quentin Schulz
2017-04-05 12:17 ` Maxime Ripard
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=20170405121310.4lgf5st5oqc52szb@lukather \
--to=maxime.ripard@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox