From: "Andreas Färber" <afaerber@suse.de>
To: Rob Herring <robh@kernel.org>
Cc: linux-amlogic@lists.infradead.org,
Carlo Caione <carlo@caione.org>,
Kevin Hilman <khilman@baylibre.com>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Mark Rutland <mark.rutland@arm.com>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will.deacon@arm.com>,
devicetree@vger.kernel.org
Subject: Re: [PATCH 10/18] ARM64: dts: meson-gxbb: Reorder nodes
Date: Mon, 23 Jan 2017 17:56:24 +0100 [thread overview]
Message-ID: <d58f8343-8369-95ac-3c21-ba8f35e7ac89@suse.de> (raw)
In-Reply-To: <20170123165208.4ujwm6rjz2b3dom5@rob-hp-laptop>
Am 23.01.2017 um 17:52 schrieb Rob Herring:
> On Fri, Jan 20, 2017 at 11:21:10PM +0100, Andreas Färber wrote:
>> Sort nodes referenced by label alphabetically.
>
> Seems to be pointless churn.
We are constantly adding new nodes, so this avoids duplicates, mismerges
and uncertainties of where things should be added. Thought that would be
obvious, as I've been asked to do it elsewhere.
It also helps with diff -u'ing .dts files to see actual differences.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
WARNING: multiple messages have this Message-ID (diff)
From: "Andreas Färber" <afaerber-l3A5Bk7waGM@public.gmane.org>
To: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Carlo Caione <carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org>,
Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>,
Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 10/18] ARM64: dts: meson-gxbb: Reorder nodes
Date: Mon, 23 Jan 2017 17:56:24 +0100 [thread overview]
Message-ID: <d58f8343-8369-95ac-3c21-ba8f35e7ac89@suse.de> (raw)
In-Reply-To: <20170123165208.4ujwm6rjz2b3dom5@rob-hp-laptop>
Am 23.01.2017 um 17:52 schrieb Rob Herring:
> On Fri, Jan 20, 2017 at 11:21:10PM +0100, Andreas Färber wrote:
>> Sort nodes referenced by label alphabetically.
>
> Seems to be pointless churn.
We are constantly adding new nodes, so this avoids duplicates, mismerges
and uncertainties of where things should be added. Thought that would be
obvious, as I've been asked to do it elsewhere.
It also helps with diff -u'ing .dts files to see actual differences.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
--
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
WARNING: multiple messages have this Message-ID (diff)
From: afaerber@suse.de (Andreas Färber)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 10/18] ARM64: dts: meson-gxbb: Reorder nodes
Date: Mon, 23 Jan 2017 17:56:24 +0100 [thread overview]
Message-ID: <d58f8343-8369-95ac-3c21-ba8f35e7ac89@suse.de> (raw)
In-Reply-To: <20170123165208.4ujwm6rjz2b3dom5@rob-hp-laptop>
Am 23.01.2017 um 17:52 schrieb Rob Herring:
> On Fri, Jan 20, 2017 at 11:21:10PM +0100, Andreas F?rber wrote:
>> Sort nodes referenced by label alphabetically.
>
> Seems to be pointless churn.
We are constantly adding new nodes, so this avoids duplicates, mismerges
and uncertainties of where things should be added. Thought that would be
obvious, as I've been asked to do it elsewhere.
It also helps with diff -u'ing .dts files to see actual differences.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N?rnberg)
WARNING: multiple messages have this Message-ID (diff)
From: afaerber@suse.de (Andreas Färber)
To: linus-amlogic@lists.infradead.org
Subject: [PATCH 10/18] ARM64: dts: meson-gxbb: Reorder nodes
Date: Mon, 23 Jan 2017 17:56:24 +0100 [thread overview]
Message-ID: <d58f8343-8369-95ac-3c21-ba8f35e7ac89@suse.de> (raw)
In-Reply-To: <20170123165208.4ujwm6rjz2b3dom5@rob-hp-laptop>
Am 23.01.2017 um 17:52 schrieb Rob Herring:
> On Fri, Jan 20, 2017 at 11:21:10PM +0100, Andreas F?rber wrote:
>> Sort nodes referenced by label alphabetically.
>
> Seems to be pointless churn.
We are constantly adding new nodes, so this avoids duplicates, mismerges
and uncertainties of where things should be added. Thought that would be
obvious, as I've been asked to do it elsewhere.
It also helps with diff -u'ing .dts files to see actual differences.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N?rnberg)
next prev parent reply other threads:[~2017-01-23 16:56 UTC|newest]
Thread overview: 122+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-20 22:21 [PATCH 00/18] ARM64: meson: DT cleanups Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 01/18] Documentation: devicetree: amlogic: Reorder boards Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 16:48 ` Rob Herring
2017-01-23 16:48 ` Rob Herring
2017-01-23 16:48 ` Rob Herring
2017-01-23 16:48 ` Rob Herring
2017-01-23 18:36 ` Andreas Färber
2017-01-23 18:36 ` Andreas Färber
2017-01-23 18:36 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 02/18] ARM64: dts: amlogic: Sort Makefile Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 03/18] ARM64: dts: meson-gxbb-vega-s95: Fix Wifi node name Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 04/18] ARM64: dts: meson-gxbb-p20x: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 16:50 ` Rob Herring
2017-01-23 16:50 ` Rob Herring
2017-01-23 16:50 ` Rob Herring
2017-01-23 16:57 ` Andreas Färber
2017-01-23 16:57 ` Andreas Färber
2017-01-23 16:57 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 05/18] ARM64: dts: meson-gx-p23x-q20x: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 06/18] ARM64: dts: meson-gxm-nexbox-a1: Drop UART comment Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-23 8:55 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 07/18] ARM64: dts: meson-gxl-s905x: Comment typo fix Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:56 ` Neil Armstrong
2017-01-23 8:56 ` Neil Armstrong
2017-01-23 8:56 ` Neil Armstrong
2017-01-23 8:56 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 08/18] ARM64: dts: meson-gxm: Drop trailing white line Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 09/18] ARM64: dts: meson-gxbb-wetek-hub: Reference CVBS node by label Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:56 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 10/18] ARM64: dts: meson-gxbb: Reorder nodes Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 16:52 ` Rob Herring
2017-01-23 16:52 ` Rob Herring
2017-01-23 16:52 ` Rob Herring
2017-01-23 16:56 ` Andreas Färber [this message]
2017-01-23 16:56 ` Andreas Färber
2017-01-23 16:56 ` Andreas Färber
2017-01-23 16:56 ` Andreas Färber
2017-01-30 14:59 ` Russell King - ARM Linux
2017-01-30 14:59 ` Russell King - ARM Linux
2017-01-30 14:59 ` Russell King - ARM Linux
2017-01-20 22:21 ` [PATCH 11/18] ARM64: dts: meson-gxbb-odroidc2: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 12/18] ARM64: dts: meson-gxbb-vega-s95: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 13/18] ARM64: dts: meson-gxbb-nexbox-a95x: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:57 ` Neil Armstrong
2017-01-23 8:57 ` Neil Armstrong
2017-01-23 8:57 ` Neil Armstrong
2017-01-23 8:57 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 14/18] ARM64: dts: meson-gxbb-p20x: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` [PATCH 15/18] ARM64: dts: meson-gxl: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:57 ` Neil Armstrong
2017-01-23 8:57 ` Neil Armstrong
2017-01-23 8:57 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 16/18] ARM64: dts: meson-gxl-s905x-nexbox-a95x: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 8:58 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 17/18] ARM64: dts: meson-gxm-nexbox-a1: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 8:58 ` Neil Armstrong
2017-01-20 22:21 ` [PATCH 18/18] ARM64: dts: meson-gx-p23x-q20x: " Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-20 22:21 ` Andreas Färber
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 8:58 ` Neil Armstrong
2017-01-23 18:46 ` [PATCH 00/18] ARM64: meson: DT cleanups Kevin Hilman
2017-01-23 18:46 ` Kevin Hilman
2017-01-23 18:46 ` Kevin Hilman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d58f8343-8369-95ac-3c21-ba8f35e7ac89@suse.de \
--to=afaerber@suse.de \
--cc=carlo@caione.org \
--cc=catalin.marinas@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=khilman@baylibre.com \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=robh@kernel.org \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.