From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966105AbbJVXgb (ORCPT ); Thu, 22 Oct 2015 19:36:31 -0400 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:60137 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966015AbbJVXg2 (ORCPT ); Thu, 22 Oct 2015 19:36:28 -0400 Date: Fri, 23 Oct 2015 01:30:52 +0900 From: Mark Brown To: Viresh Kumar Cc: Rafael Wysocki , nm@ti.com, sboyd@codeaurora.org, linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, rob.herring@linaro.org, lee.jones@linaro.org, devicetree@vger.kernel.org, Ian Campbell , Kumar Gala , open list , Mark Rutland , Pawel Moll , "Rafael J. Wysocki" , Rob Herring Message-ID: <20151022163052.GW8232@sirena.org.uk> References: <2b87b162eabd1570ae2311e1ef8655acda72f678.1441972771.git.viresh.kumar@linaro.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="RMorpr61ug7CvG36" Content-Disposition: inline In-Reply-To: <2b87b162eabd1570ae2311e1ef8655acda72f678.1441972771.git.viresh.kumar@linaro.org> X-Cookie: Does not include installation. User-Agent: Mutt/1.5.24 (2015-08-30) X-SA-Exim-Connect-IP: 211.36.156.43 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH 01/16] PM / OPP: Add 'supply-names' binding X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --RMorpr61ug7CvG36 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Sep 11, 2015 at 05:31:57PM +0530, Viresh Kumar wrote: > +- supply-names: This is a required property, only if multiple supplies are > + available for the device. Otherwise it is optional. > + > + This list is used to pass names of all the device supplies. The order of names > + present here is important, as that should match the order in which values are > + present in 'opp-microvolt' and 'opp-microamp' properties. If we were going to add something like this (which I'm really not that thrilled about due to their encouragement of bad practice as previously discussed) it seems wrong to add it at the level of a specific binding rather than as a general facility. I think I'm not entirely sold on this use case though, I'll follow up to a later message in this thread... --RMorpr61ug7CvG36 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJWKQ87AAoJECTWi3JdVIfQWe4H/RQDcctlBFVAi9U5euO3pHTv ofFDIZ9GBKzFJH/QX2xElyvp5cfhJCA5b6x5uII/x/hRI4rtSqvKjSSEVmBrjqu6 Dvf+2stIM9bVyn50F4GgVp2yBamXP4FgIq/SyoU53jcLz8Lv9OupRDLvamcxLUao vksK67PCALyKYTfGAuakpooryc3AARoy2Trw5HfqtdzKvkZ0y/dfthJrH/h7f3MV RrabmJJ21+aCm/NMNUou5lNAG7gTmNXc9EqCdqNahO8Q5+G0uqHEg00tX6dMYKCZ ELFLlvqTQyEvx61EX9cJQ6Ob7BWzOJBTnRcexv5BkF49AtC0kLWJNVWzmSfpwp4= =hRCS -----END PGP SIGNATURE----- --RMorpr61ug7CvG36--