From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory CLEMENT Subject: Re: [PATCH v3 01/11] dt-bindings: thermal: Describe Armada AP806 and CP110 Date: Fri, 15 Dec 2017 09:44:19 +0100 Message-ID: <87po7gmlcs.fsf@free-electrons.com> References: <20171214103011.24713-1-miquel.raynal@free-electrons.com> <20171214103011.24713-2-miquel.raynal@free-electrons.com> <20171215082759.7t24ka3lpkulcb7r@tarshish> <20171215093203.74b7a59c@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20171215093203.74b7a59c@xps13> (Miquel RAYNAL's message of "Fri, 15 Dec 2017 09:32:03 +0100") Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Miquel RAYNAL Cc: Baruch Siach , Mark Rutland , Andrew Lunn , Jason Cooper , Nadav Haklai , linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Catalin Marinas , Antoine Tenart , Will Deacon , David Sniatkiwicz , Eduardo Valentin , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Rob Herring , Zhang Rui , Thomas Petazzoni , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org Hi Miquel, On ven., déc. 15 2017, Miquel RAYNAL wrote: > Hello Baruch, > > On Fri, 15 Dec 2017 10:27:59 +0200 > Baruch Siach wrote: > >> Hi Miquel >> >> On Thu, Dec 14, 2017 at 11:30:01AM +0100, Miquel Raynal wrote: >> > +- marvell,thermal-zone-name: The name to identify the thermal zone >> > + within the sysfs, useful when multiple >> > + thermal zones are registered (AP, >> > CPx...). >> >> I don't think that would be acceptable. DT is about describing the >> hardware. sysfs is a Linux implementation detail which is not tied to >> any specific hardware. If this is accepted, the property should be >> named 'linux,thermal-zone-name'. > > You are right the sysfs mention should not appear in the description. > > Otherwise for the naming I'm not sure "linux," is a valid prefix in > that case. Actually the choice between linux or marvell make me realize that there is something wrong. Having a name associated to a device is something pretty usual with the device tree, however it is as the class device level, such as clock-names, line-name, or regulator-name. So in my opinion if we want to support naming from device tree it would be done for all the thermal device not just for the Marvell one. However I don't think we need it. For example for the clocks we created the name dynamically using of the base address of the register to keep them unique. Gregory > > Miquèl > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html