From: Shawn Guo <shawnguo@kernel.org>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Stefan Wahren <stefan.wahren@i2se.com>,
devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
linux-imx@nxp.com,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Fabio Estevam <festevam@gmail.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/9] ARM: dts: imx28: Clean up older DTS files
Date: Mon, 6 Mar 2023 10:58:26 +0800 [thread overview]
Message-ID: <20230306025826.GE143566@dragon> (raw)
In-Reply-To: <20230203131303.GY13319@pengutronix.de>
On Fri, Feb 03, 2023 at 02:13:03PM +0100, Sascha Hauer wrote:
> On Fri, Feb 03, 2023 at 10:44:49AM +0100, Sébastien Szymanski wrote:
> > Hello,
> >
> > On 2/1/23 21:33, Stefan Wahren wrote:
> > > Since the imx28 belongs to the early ARM platforms which has been
> > > adapted to DT, a lot of these DTS files are not in the best shape.
> > > So this series tries to address this by using label references
> > > and SPDX tags.
> > >
> > > Theses patches doesn't include functional changes. The resulting
> > > DTB files has been verified with dtdiff.
> > >
> > > Stefan Wahren (9):
> > > ARM: dts: imx28-apf28: Convert to use label references
> > > ARM: dts: imx28-m28/sps1: Convert to use label references
> > > ARM: dts: imx28-apx4devkit: Convert to use label references
> > > ARM: dts: imx28-cfa10036: Convert to use label references
> > > ARM: dts: imx28-duckbill: Convert to use label references
> > > ARM: dts: imx28-duckbill-2: Include base board
> > > ARM: dts: imx28-evk: Convert to use label references
> > > ARM: dts: imx28-ts4600: Convert to use label references
> > > ARM: dts: imx28-tx28: add SPDX-License-Identifier
> > >
> > > arch/arm/boot/dts/imx28-apf28.dts | 96 ++--
> > > arch/arm/boot/dts/imx28-apf28dev.dts | 312 ++++++------
> > > arch/arm/boot/dts/imx28-apx4devkit.dts | 380 +++++++-------
> > > arch/arm/boot/dts/imx28-cfa10036.dts | 193 ++++----
> > > arch/arm/boot/dts/imx28-cfa10049.dts | 454 +++++++++--------
> > > arch/arm/boot/dts/imx28-cfa10055.dts | 224 +++++----
> > > arch/arm/boot/dts/imx28-cfa10056.dts | 146 +++---
> > > arch/arm/boot/dts/imx28-cfa10057.dts | 252 +++++-----
> > > arch/arm/boot/dts/imx28-cfa10058.dts | 186 ++++---
> > > arch/arm/boot/dts/imx28-duckbill-2-485.dts | 174 +------
> > > .../arm/boot/dts/imx28-duckbill-2-enocean.dts | 198 +-------
> > > arch/arm/boot/dts/imx28-duckbill-2-spi.dts | 211 ++------
> > > arch/arm/boot/dts/imx28-duckbill-2.dts | 256 +++++-----
> > > arch/arm/boot/dts/imx28-duckbill.dts | 196 ++++----
> > > arch/arm/boot/dts/imx28-evk.dts | 462 +++++++++---------
> > > arch/arm/boot/dts/imx28-m28.dtsi | 44 +-
> > > arch/arm/boot/dts/imx28-m28cu3.dts | 354 +++++++-------
> > > arch/arm/boot/dts/imx28-m28evk.dts | 420 ++++++++--------
> > > arch/arm/boot/dts/imx28-sps1.dts | 201 ++++----
> > > arch/arm/boot/dts/imx28-ts4600.dts | 80 ++-
> > > arch/arm/boot/dts/imx28-tx28.dts | 38 +-
> > > 21 files changed, 2142 insertions(+), 2735 deletions(-)
> > >
> >
> > FYI, such changes have been rejected years ago:
> >
> > https://lore.kernel.org/linux-arm-kernel/20140512141814.GE8330@dragon/
>
> I hope the attitude has changed here in the last years. Device trees
> with label references are so much easier to read and to maintain.
Ok, I'm flexible on this. Considering Krzysztof has been doing the same
for other platforms, I guess arm-soc folks changed their thoughts.
Shawn
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-03-06 2:59 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-01 20:33 [PATCH 0/9] ARM: dts: imx28: Clean up older DTS files Stefan Wahren
2023-02-01 20:33 ` [PATCH 1/9] ARM: dts: imx28-apf28: Convert to use label references Stefan Wahren
2023-02-13 8:22 ` Sébastien Szymanski
2023-02-13 14:50 ` Stefan Wahren
2023-02-01 20:33 ` [PATCH 2/9] ARM: dts: imx28-m28/sps1: " Stefan Wahren
2023-02-01 21:03 ` Marek Vasut
2023-02-01 20:33 ` [PATCH 3/9] ARM: dts: imx28-apx4devkit: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 4/9] ARM: dts: imx28-cfa10036: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 5/9] ARM: dts: imx28-duckbill: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 6/9] ARM: dts: imx28-duckbill-2: Include base board Stefan Wahren
2023-02-01 20:33 ` [PATCH 7/9] ARM: dts: imx28-evk: Convert to use label references Stefan Wahren
2023-02-01 20:33 ` [PATCH 8/9] ARM: dts: imx28-ts4600: " Stefan Wahren
2023-02-01 20:33 ` [PATCH 9/9] ARM: dts: imx28-tx28: add SPDX-License-Identifier Stefan Wahren
2023-02-02 6:16 ` Lothar Waßmann
2023-02-03 9:44 ` [PATCH 0/9] ARM: dts: imx28: Clean up older DTS files Sébastien Szymanski
2023-02-03 10:16 ` Krzysztof Kozlowski
2023-02-03 13:13 ` Sascha Hauer
2023-03-06 2:58 ` Shawn Guo [this message]
2023-03-07 9:18 ` Krzysztof Kozlowski
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=20230306025826.GE143566@dragon \
--to=shawnguo@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=stefan.wahren@i2se.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).