From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757048AbcAZCmH (ORCPT ); Mon, 25 Jan 2016 21:42:07 -0500 Received: from mail.kernel.org ([198.145.29.136]:33268 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756715AbcAZCmC (ORCPT ); Mon, 25 Jan 2016 21:42:02 -0500 Date: Mon, 25 Jan 2016 20:41:57 -0600 From: Rob Herring To: Wei Ni Cc: thierry.reding@gmail.com, rui.zhang@intel.com, MLongnecker@nvidia.com, swarren@wwwdotorg.org, mikko.perttunen@kapsi.fi, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH V4 07/11] of: Add bindings of hw-trip-points for soctherm Message-ID: <20160126024157.GA15270@rob-hp-laptop> References: <1453718796-4393-1-git-send-email-wni@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1453718796-4393-1-git-send-email-wni@nvidia.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 25, 2016 at 06:46:36PM +0800, Wei Ni wrote: > Add hw-trips sub-node for soctherm, which is > used to describe the hardware trip points for > each soctherm sensors. This seems like something that should be common. There was a hwmon binding just the other day with temperature set points. > Signed-off-by: Wei Ni > --- > .../devicetree/bindings/thermal/tegra-soctherm.txt | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt > index 6b68cd150405..75dc9dbe7e1a 100644 > --- a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt > +++ b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt > @@ -26,6 +26,15 @@ Required properties : > of this property. See for a > list of valid values when referring to thermal sensors. > > +Optional properties: > +- hw-trip-points : A sub-node which is a container of hardware trip points > + for each sensors. > + * sensors: Sub-nodes which are used to describe the HW trip points info, > + must be named as "cpu", "gpu", "mem", "pll". > + Properties: > + - thermtrip-temperature : Shutdown or reset temperature in millicelsius, > + once the temperature of this sensor is higher than the > + thermtrip-temperature, the system will be shutdown or reset. Append units and temperature is a bit redundant. Something like thermtrip-mC or thermtrip-milliC You could easily have multiple levels of trip points, so perhaps the name needs to reflect the action (i.e. shutdown). Rob