On 04/06/2019 07:56, Andy Tang wrote: > Hi Edubezval, Rui, > > Any further comments?
>From my POV, this patch makes sense. We may be interested to show up the thermal zones in sysfs and optionally mitigate them via an userspace governor. Acked-by: Daniel Lezcano <[email protected]> >> -----Original Message----- >> From: Yuantian Tang <[email protected]> >> Sent: 2019年5月15日 17:37 >> To: [email protected]; [email protected] >> Cc: [email protected]; [email protected]; [email protected]; >> [email protected]; [email protected]; >> [email protected]; Andy Tang <[email protected]> >> Subject: [PATCH] dt-bindings: thermal: Make cooling-maps property optional >> >> There may be no cooling device on system, or there are no enough cooling >> devices for each thermal zone in multiple thermal zone cases since cooling >> devices can't be shared. >> So make this property optional to remove such limitations. >> >> Signed-off-by: Yuantian Tang <[email protected]> >> --- >> .../devicetree/bindings/thermal/thermal.txt | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/thermal/thermal.txt >> b/Documentation/devicetree/bindings/thermal/thermal.txt >> index ca14ba9..694e834 100644 >> --- a/Documentation/devicetree/bindings/thermal/thermal.txt >> +++ b/Documentation/devicetree/bindings/thermal/thermal.txt >> @@ -142,11 +142,11 @@ Required properties: >> - trips: A sub-node which is a container of only trip point nodes >> Type: sub-node required to describe the thermal zone. >> >> + >> +Optional property: >> - cooling-maps: A sub-node which is a container of only cooling >> device >> Type: sub-node map nodes, used to describe the relation between >> trips >> and cooling devices. >> - >> -Optional property: >> - coefficients: An array of integers (one signed cell) >> containing >> Type: array coefficients to compose a linear relation >> between >> Elem size: one cell the sensors listed in the thermal-sensors >> property. >> -- >> 1.7.1 > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
