Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753119AbbETMkS (ORCPT ); Wed, 20 May 2015 08:40:18 -0400 Received: from mail-pa0-f52.google.com ([209.85.220.52]:33098 "EHLO mail-pa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751818AbbETMkO (ORCPT ); Wed, 20 May 2015 08:40:14 -0400 Date: Wed, 20 May 2015 14:40:09 +0200 From: Thierry Reding To: Alexandre Courbot Cc: Krzysztof =?utf-8?Q?Koz=C5=82owski?= , Stephen Warren , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-tegra@vger.kernel.org" , Linux Kernel Mailing List Subject: Re: [RESEND PATCH 2/2] ARM: dts: tegra: Use labels for overriding nodes in Tegra114 boards Message-ID: <20150520124008.GB24016@ulmo.nvidia.com> References: <1432036279-6318-1-git-send-email-k.kozlowski.k@gmail.com> <1432036279-6318-2-git-send-email-k.kozlowski.k@gmail.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="O3RTKUHj+75w1tg5" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2811 Lines: 67 --O3RTKUHj+75w1tg5 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, May 20, 2015 at 02:05:38PM +0900, Alexandre Courbot wrote: > On Wed, May 20, 2015 at 2:03 PM, Krzysztof Koz=C5=82owski > wrote: > > 2015-05-20 13:05 GMT+09:00 Alexandre Courbot : > >> On Tue, May 19, 2015 at 8:51 PM, Krzysztof Kozlowski > >> wrote: > >>> Usage of labels instead of full paths reduces possible mistakes when > >>> overriding nodes. > >>> > >>> Signed-off-by: Krzysztof Kozlowski > >> > >> Indentation seems to be off by one tab in the added code (hence the > >> huge size of this patch ; most lines should not change), can you > >> check? > > > > It is correct. This change in indentation is an effect of moving nodes > > out of first bracket ("\ {"). That is the convention. > > > > It us up to you guys if this is worth the effort. For exynos we use > > label-convention and now I am converting old DTS to it. I think the > > label-convention is less error-prone when extending or overriding > > nodes. Also it removes duplicated addresses. >=20 > I don't really have a strong opinion on this - Stephen and Thierry > have worked with DT much more, let's see what they think... I agree with Stephen that this is unnecessary churn. I understand the reason why people prefer to use labels, but I don't think it's enough of an issue to warrent rewriting all of the DTS files. I personally don't like the convention very much because it makes the otherwise very neatly structured DTS files hard to read. Thierry --O3RTKUHj+75w1tg5 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVXICnAAoJEN0jrNd/PrOhIeYP/3UMvKeGd3+ny+UUBW8BpMz+ 5Kciba9sb0E+q6f62mbB6h6ITWPFKdrGfvkRqmCFGJy7uMyeluZGET44rLMuNdt5 PSYRlxZq7mLO7/MZ9t1dHI6du8jP5ghGjrxGXqTvQ4l7AwY/4nFZdG95XdZeSSL2 AIoTzwHajsi5i3fFQV3YUfDIIqUvodAUEm6JtW6uHEvmOxY0Ll41DcpAwCi+xho7 2iwGonOwjyJr67s26SIccrOOIBqx1fpZneWa0hx4KVsxbtbc83MYXqXHDvYyFEW8 xEQjy3VDzRAf0Uww9O4iocmUyslDBIVDD18zoFgiUhbcRdwCr90wiGc/DgMyK2gM MgJWuc34vU1cWb1BNy+ZPwGTexcrP3Ku6EQ0jbNXWo1ZJQQ/urQFO5fIpsyp4GeB VK3pYM1A8actaZfP9976cdH8PcMYPfVTRLSD76GgmTbdV1kS7QNoTxmr5IbuusJT 4WntVd46619KEaAQm/aM8pQIs1pKw28LPEiqidT2Wa1dQPhSBtOazcJNi9rBaJo3 qNkjbpSvdb85yejXzyBisIxTkOeq3m4zWElY5dFgK0AdOAPpHubBUYfBdG71Y2GF tNTHg/Nz57y948VBIAS3hpTERBTiQTh0EkwwmWDkFKr6D6zYJCCCgH+XVLXMcXBm jKmvfobFKq9ibNqD5F+1 =krdU -----END PGP SIGNATURE----- --O3RTKUHj+75w1tg5-- -- 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/