From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v4 6/6] ARM: dts: STiH407: Add B2120 board support Date: Tue, 20 May 2014 08:43:53 +0100 Message-ID: <20140520074353.GD20874@lee--X1> References: <1394614210-15698-1-git-send-email-maxime.coquelin@st.com> <1394614210-15698-7-git-send-email-maxime.coquelin@st.com> <20140520072048.GC20874@lee--X1> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Olof Johansson Cc: Maxime COQUELIN , Rob Landley , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Srinivas Kandagatla , Stuart Menefy , Linus Walleij , Giuseppe Cavallaro , "linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org" List-Id: devicetree@vger.kernel.org > >> > + soc { > >> > + sbc_serial0: serial@9530000 { > >> > + status =3D "okay"; > >> > + }; > >> > >> You might want to consider reference-based syntax here instead, so= you > >> don't have to mimic the hierarchy. That'd be (at the root level of= the > >> file, below this secion: > >> > >> &sbc_serial0: { > >> status =3D "okay"; > >> }; > > > > I'm personally not keen on this scheme. It's sometimes helpful to = know > > the hierarchy and I don't think it's a large overhead to format the > > subordinate DTS files in this way. > > > > Please consider not enforcing this. >=20 > Definitely not enforcing it, and I didn't use to like it either but i= t > has some real upsides. >=20 > In particular, it saves a lot of grief when you're changing something > like the unit-id of a node in .dtsi and forget to do the same update > in the dts. I'm not entirely sure what a unit-id is, but I can see that there would be benefits to using the referenced-based syntax as you call it. If any of those benefits hold true here I won't push back, but I would personally like to see us default to the hierarchical scheme. --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html