From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH V6 03/12] phy: st-miphy40lp: Add binding information Date: Tue, 18 Feb 2014 15:58:14 +0100 Message-ID: <201402181558.14663.arnd@arndb.de> References: <2CC2A0A4A178534D93D5159BF3BCB66189FD2CAFB1@EAPEX1MAIL1.st.com> <20140218122324.GA23267@e106331-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140218122324.GA23267-NuALmloUBlrZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mark Rutland Cc: Mohit KUMAR DCG , Pratyush ANAND , Viresh Kumar , Kishon Vijay Abraham I , spear-devel , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On Tuesday 18 February 2014, Mark Rutland wrote: > > > > - These are Spear SoC specific miscellaneous registers. Here these are used for > > to configure sata/pcie aux clock. > > > > > > > +- phy-id: Instance id of the phy. > > > > +- #phy-cells : from the generic PHY bindings, must be 1. > > > > + - 1st cell: phandle to the phy node. > > > > + - 2nd cell: 0 if phy (in 1st cell) is to be used for SATA, 1 for PCIe > > > > + and 2 for Super Speed USB. > > > > > > One cell or two? > > > > - No of cells are two, is this the question? > > The description of #phy-cells says it must be 1. Presumably it must be 2. The description here is counting the phandle, while we normally only count the arguments following the phandle. #phy-cells=<1> is correct here, but the description should be written a bit clearer. Arnd -- 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