From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH V4 1/3] OPP: Redefine bindings to overcome shortcomings Date: Thu, 7 May 2015 23:18:42 +0100 Message-ID: <20150507221842.GW22845@sirena.org.uk> References: <20150504121209.GM15510@sirena.org.uk> <20150505105714.GA22845@sirena.org.uk> <20150505171252.GI22845@sirena.org.uk> <20150507055231.GB32399@codeaurora.org> <20150507110233.GR15510@sirena.org.uk> <20150507211855.GA2455@codeaurora.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="+E8tZd0NUIRz5HKc" Return-path: Content-Disposition: inline In-Reply-To: <20150507211855.GA2455@codeaurora.org> Sender: linux-pm-owner@vger.kernel.org To: Stephen Boyd Cc: Viresh Kumar , Rafael Wysocki , Rob Herring , Arnd Bergmann , Nishanth Menon , Mike Turquette , Linaro Kernel Mailman List , "linux-pm@vger.kernel.org" , Grant Likely , "olof@lixom.net" , Sudeep Holla , "devicetree@vger.kernel.org" , Viswanath Puttagunta , Lucas Stach , Thomas Petazzoni , "linux-arm-kernel@lists.infradead.org" , Thomas Abraham , Abhilash Kesavan , Kevin Hilman , santosh List-Id: devicetree@vger.kernel.org --+E8tZd0NUIRz5HKc Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, May 07, 2015 at 02:18:55PM -0700, Stephen Boyd wrote: > On 05/07, Mark Brown wrote: > > I'm not looking for anyone to explain this to me in e-mail, what I'm > > looking for is for the binding document to be clear so someone can tell > > what the binding means by reading the documentation for the binding. > Ok. Perhaps the simplest thing to do then is to reuse wording > from the regulator_set_load() API documentation? That's the only > usage of this value I'm aware of. Something like: > The current load of the device when using this OPP. Used > to set the most efficient regulator operating mode. > We don't need any sort of min/max for this property either, so a > single value should be all that's required. That makes sense to me. Perhaps "current drawn by the device" rather than "current load of the device" since current is sadly overloaded :/ --+E8tZd0NUIRz5HKc Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJVS+TBAAoJECTWi3JdVIfQA/AH/ApKBMqEOU22yQpC5elWPn0r sb+leNW3TK3PU/ieHDH4veGJE5DkWLvV0tnl8BjjyYXDfkEo3G2V5Sgx1RTq4413 jwlKqP8FUrPfHZ5dKfsV38EQZOF/sD5Ocanv+5qFZt6DUsAtCUc37MEDKAFDBfNr TpEjAMBczgxnzJG+eJHl4Wcis9VaDYFfR2znZUf8CoNTH015BdZijdcX6ul6ERdK kDl4ovxgvVRhHDtXndxKAqLl8Ti46RtJkqXlH1HQT/epybTBe4OtkFrHcAdjOfwp Zf0KBwDn2icyD1Vp86VdtXYjFVZeBn82hfIh0TS/E6rEWaeXG/ElBL3i2YdzRys= =bi+q -----END PGP SIGNATURE----- --+E8tZd0NUIRz5HKc-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: broonie@kernel.org (Mark Brown) Date: Thu, 7 May 2015 23:18:42 +0100 Subject: [PATCH V4 1/3] OPP: Redefine bindings to overcome shortcomings In-Reply-To: <20150507211855.GA2455@codeaurora.org> References: <20150504121209.GM15510@sirena.org.uk> <20150505105714.GA22845@sirena.org.uk> <20150505171252.GI22845@sirena.org.uk> <20150507055231.GB32399@codeaurora.org> <20150507110233.GR15510@sirena.org.uk> <20150507211855.GA2455@codeaurora.org> Message-ID: <20150507221842.GW22845@sirena.org.uk> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, May 07, 2015 at 02:18:55PM -0700, Stephen Boyd wrote: > On 05/07, Mark Brown wrote: > > I'm not looking for anyone to explain this to me in e-mail, what I'm > > looking for is for the binding document to be clear so someone can tell > > what the binding means by reading the documentation for the binding. > Ok. Perhaps the simplest thing to do then is to reuse wording > from the regulator_set_load() API documentation? That's the only > usage of this value I'm aware of. Something like: > The current load of the device when using this OPP. Used > to set the most efficient regulator operating mode. > We don't need any sort of min/max for this property either, so a > single value should be all that's required. That makes sense to me. Perhaps "current drawn by the device" rather than "current load of the device" since current is sadly overloaded :/ -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 473 bytes Desc: Digital signature URL: