From mboxrd@z Thu Jan 1 00:00:00 1970 From: Baruch Siach Subject: Re: [PATCH v3 01/11] dt-bindings: thermal: Describe Armada AP806 and CP110 Date: Fri, 15 Dec 2017 10:27:59 +0200 Message-ID: <20171215082759.7t24ka3lpkulcb7r@tarshish> References: <20171214103011.24713-1-miquel.raynal@free-electrons.com> <20171214103011.24713-2-miquel.raynal@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20171214103011.24713-2-miquel.raynal-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Miquel Raynal Cc: Zhang Rui , Eduardo Valentin , Rob Herring , Mark Rutland , Jason Cooper , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Catalin Marinas , Will Deacon , linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Thomas Petazzoni , Antoine Tenart , Nadav Haklai , David Sniatkiwicz List-Id: devicetree@vger.kernel.org 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'. baruch -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org - tel: +972.52.368.4656, http://www.tkos.co.il - -- 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