From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javi Merino Subject: Re: [RFC PATCH v6 9/9] of: thermal: Introduce sustainable power for a thermal zone Date: Tue, 6 Jan 2015 09:42:15 +0000 Message-ID: <20150106094215.GB2885@e104805> References: <1417806260-9264-1-git-send-email-javi.merino@arm.com> <1417806260-9264-10-git-send-email-javi.merino@arm.com> <20150102155256.GD10683@developer> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20150102155256.GD10683@developer> Sender: linux-kernel-owner@vger.kernel.org To: Eduardo Valentin Cc: "linux-pm@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Punit Agrawal , "broonie@kernel.org" , Zhang Rui List-Id: linux-pm@vger.kernel.org On Fri, Jan 02, 2015 at 03:53:00PM +0000, Eduardo Valentin wrote: > On Fri, Dec 05, 2014 at 07:04:20PM +0000, Javi Merino wrote: > > From: Punit Agrawal > > > > Introduce an optional property called, sustainable-power, which > > represents the power (in mW) which the thermal zone can safely > > dissipate. > > > > If provided the property is parsed and associated with the thermal > > zone via the thermal zone parameters. > > > > Cc: Zhang Rui > > Cc: Eduardo Valentin > > Signed-off-by: Punit Agrawal > > --- > > Documentation/devicetree/bindings/thermal/thermal.txt | 4 ++++ > > drivers/thermal/of-thermal.c | 4 ++++ > > 2 files changed, 8 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/thermal/thermal.txt b/Documentation/devicetree/bindings/thermal/thermal.txt > > index f5db6b72a36f..c6eb9a8d2aed 100644 > > --- a/Documentation/devicetree/bindings/thermal/thermal.txt > > +++ b/Documentation/devicetree/bindings/thermal/thermal.txt > > @@ -167,6 +167,10 @@ Optional property: > > by means of sensor ID. Additional coefficients are > > interpreted as constant offset. > > > > +- sustainable-power: An estimate of the sustainable power (in mW) that the > > + Type: unsigned thermal zone can dissipate. > > + Size: one cell > > + > > Please, include examples of this property, as you mentioned in the > governor documentation. I'd rather put a pointer to the documentation instead of repeating the same thing here. What do you think?