From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH RFC 1/3] thermal: allow hwmon devices to be created for of-thermal zones Date: Mon, 20 Mar 2017 12:15:52 -0500 Message-ID: <20170320171552.o2ye73c2okcvhluq@rob-hp-laptop> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Russell King Cc: Mark Rutland , Andrew Lunn , Jason Cooper , linux-pm@vger.kernel.org, Eduardo Valentin , devicetree@vger.kernel.org, Gregory Clement , Zhang Rui , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: linux-pm@vger.kernel.org On Sun, Mar 12, 2017 at 07:07:40PM +0000, Russell King wrote: > Allow hwmon devices to be optionally created for of-thermal zones, > rather than permanently denying them "in case" there is a hwmon > driver duplicating the thermal driver. > > Signed-off-by: Russell King > --- > Without this, "sensors" is unable to report the temperature of the > Dove SoC when it supports thermal zones. > > Documentation/devicetree/bindings/thermal/thermal.txt | 3 +++ > arch/arm/boot/dts/dove.dtsi | 1 + > drivers/thermal/of-thermal.c | 3 ++- > 3 files changed, 6 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/thermal/thermal.txt b/Documentation/devicetree/bindings/thermal/thermal.txt > index 88b6ea1ad290..1478735fff85 100644 > --- a/Documentation/devicetree/bindings/thermal/thermal.txt > +++ b/Documentation/devicetree/bindings/thermal/thermal.txt > @@ -175,6 +175,9 @@ containing trip nodes and one sub-node containing all the zone cooling maps. > 2000mW, while on a 10'' tablet is around > 4500mW. > > +- linux,hwmon: Allow Linux to create hwmon devices for the thermal > + Type: bool zone. > + > Note: The delay properties are bound to the maximum dT/dt (temperature > derivative over time) in two situations for a thermal zone: > (i) - when passive cooling is activated (polling-delay-passive); and > diff --git a/arch/arm/boot/dts/dove.dtsi b/arch/arm/boot/dts/dove.dtsi > index 40fb98687230..00f5971cd039 100644 > --- a/arch/arm/boot/dts/dove.dtsi > +++ b/arch/arm/boot/dts/dove.dtsi > @@ -804,6 +804,7 @@ > > thermal-zones { > soc-thermal { > + linux,hwmon; I'd prefer to see a black or white list of sensor compatibles here. Then the dtb doesn't need to change if things move between hwmon and thermal zones. Rob