Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754783Ab3IXVeu (ORCPT ); Tue, 24 Sep 2013 17:34:50 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:46725 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754062Ab3IXVes (ORCPT ); Tue, 24 Sep 2013 17:34:48 -0400 Message-ID: <5242051F.70609@ti.com> Date: Tue, 24 Sep 2013 17:33:19 -0400 From: Eduardo Valentin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 MIME-Version: 1.0 To: Mark Rutland CC: Eduardo Valentin , "swarren@wwwdotorg.org" , Pawel Moll , "ian.campbell@citrix.com" , "rob.herring@calxeda.com" , "linux@roeck-us.net" , "rui.zhang@intel.com" , "wni@nvidia.com" , "joe@perches.com" , "grant.likely@linaro.org" , "durgadoss.r@intel.com" , "lm-sensors@lm-sensors.org" , "linux-kernel@vger.kernel.org" , "linux-pm@vger.kernel.org" Subject: Re: [PATCHv6 02/16] drivers: thermal: introduce device tree parser References: <1379537849-28425-1-git-send-email-eduardo.valentin@ti.com> <1379540136-28378-1-git-send-email-eduardo.valentin@ti.com> <20130923104003.GA16069@e106331-lin.cambridge.arm.com> In-Reply-To: <20130923104003.GA16069@e106331-lin.cambridge.arm.com> X-Enigmail-Version: 1.5.2 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="9VuNC348GOpQBO6G88XXmFPgOq4Q8R8lS" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2597 Lines: 78 --9VuNC348GOpQBO6G88XXmFPgOq4Q8R8lS Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 23-09-2013 06:40, Mark Rutland wrote: >=20 > It would be nice to have a name for the cells after a phandle which > describe the cooling device's configuration. You've called them > parameters here, but it probably makes sense to call them a > cooling-specifier (following clock-specifier and interrupt-specifier). Maybe it was not very clear, and I am working on improving this, but what I am proposing is simply to have: cooling-device =3D <&cdev min max> where min and max are one cell unsigned values referring to minimum cooling level and maximum cooling level, for this reference. Note that 'cdev' may have 10 levels, but in this reference we may use only from 6 to 10: cooling-device =3D <&cdev 6 10>; I don't see a need to have a cooling-names for this case. And for now, I also don't see why we would use other specifiers. But we can leave it open for future extensions. It does make sense to have thermal-sensor-names (using thermal-sensor as per your suggestion). Because it makes clear where the sensor is in the case of using several sensors in one zone. Just like in the example I already gave: > +cpu-thermal: cpu-thermal { > + polling-delay-passive =3D <250>; /* milliseconds */ > + polling-delay =3D <1000>; /* milliseconds */ > + > + /* sensor ID */ > + thermal-sensors =3D <&bandgap0 0>, > + <&adc 0>; > + thermal-sensors-names =3D "cpu", "pcb north"; > + > + /* hotspot =3D 100 * bandgap - 120 * adc + 484 */ > + coefficients =3D <100 -120 484>; > + > + trips { > + ... > + }; > + > + cooling-attachments { > + ... > + }; > +}; --=20 You have got to be excited about what you are doing. (L. Lamport) Eduardo Valentin --9VuNC348GOpQBO6G88XXmFPgOq4Q8R8lS Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iF4EAREIAAYFAlJCBSsACgkQCXcVR3XQvP1s2gD6Ak+M7jZvhrKL6WWAYGh9j+XT JWpfJDtukecQzRmJ3JsA/2Gni/BCC7ay0GgvMwk1o7FqN8cFxJ6//g02cMMqppab =7JL8 -----END PGP SIGNATURE----- --9VuNC348GOpQBO6G88XXmFPgOq4Q8R8lS-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/