From: Frank Rowand <frowand.list@gmail.com>
To: Andrew Davis <afd@ti.com>, Shawn Guo <shawnguo@kernel.org>,
Li Yang <leoyang.li@nxp.com>,
Sascha Hauer <s.hauer@pengutronix.de>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>,
Masahiro Yamada <masahiroy@kernel.org>,
Michal Marek <michal.lkml@markovi.net>,
Nick Desaulniers <ndesaulniers@google.com>,
Geert Uytterhoeven <geert@linux-m68k.org>
Cc: devicetree@vger.kernel.org, linux-kbuild@vger.kernel.org,
linux-renesas-soc@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/7] Rename DTB overlay source files
Date: Mon, 24 Oct 2022 17:04:25 -0500 [thread overview]
Message-ID: <51477b80-478f-d2cb-d98d-477431057fd4@gmail.com> (raw)
In-Reply-To: <20221024173434.32518-1-afd@ti.com>
Hi Andrew,
On 10/24/22 12:34, Andrew Davis wrote:
> Hello all,
>
> This is a series based on my patch here[0]. As suggested by Rob
> I've resurrected Frank's patch and appended it to mine as a series.
>
> First patch here is my original patch, 3rd is Frank's patch but with
> the unittest changes pulled out into the 2nd patch. That was re-worked
> moving the source building macro into scripts/Makefile.lib.
>
> Patches 4, 5, and 6 are an attempt at renaming all the existing DTB
> overlays. Split out by platform so they could be taken by platform
> maintainers or if easier ACK'd here and taken all together.
>
> This should cover all the DTB overlays so we can remove the old .dts
> rule for overlays and make .dtso the only supported way, let me know
> if we want that this cycle and I can post that too.
Thanks for picking this up and moving it forward. I've reviewed patches
1 - 3, and took a quick look at 4 - 7 (which also look fine at a glance).
-Frank
>
> Thanks,
> Andrew
>
> Changes from v1[1]:
> - Added patch to rename pi433 overlay.
> - Cleaned wording on patch 4-6.
> - Collected some ACKs
>
> [0] https://www.spinics.net/lists/kernel/msg4548509.html
> [1] https://www.spinics.net/lists/arm-kernel/msg1020165.html
>
> Andrew Davis (6):
> kbuild: Allow DTB overlays to built from .dtso named source files
> kbuild: Allow DTB overlays to built into .dtso.S files
> arm64: dts: freescale: Rename DTB overlay source files from .dts to
> .dtso
> arm64: dts: renesas: Rename DTB overlay source files from .dts to
> .dtso
> arm64: dts: xilinx: Rename DTB overlay source files from .dts to .dtso
> staging: pi433: overlay: Rename overlay source file from .dts to .dtso
>
> Frank Rowand (1):
> of: overlay: rename overlay source files from .dts to .dtso
>
> ...qds-13bb.dts => fsl-ls1028a-qds-13bb.dtso} | 0
> ...qds-65bb.dts => fsl-ls1028a-qds-65bb.dtso} | 0
> ...qds-7777.dts => fsl-ls1028a-qds-7777.dtso} | 0
> ...qds-85bb.dts => fsl-ls1028a-qds-85bb.dtso} | 0
> ...qds-899b.dts => fsl-ls1028a-qds-899b.dtso} | 0
> ...qds-9999.dts => fsl-ls1028a-qds-9999.dtso} | 0
> ...ts => imx8mm-venice-gw72xx-0x-imx219.dtso} | 0
> ...=> imx8mm-venice-gw72xx-0x-rs232-rts.dtso} | 0
> ...dts => imx8mm-venice-gw72xx-0x-rs422.dtso} | 0
> ...dts => imx8mm-venice-gw72xx-0x-rs485.dtso} | 0
> ...ts => imx8mm-venice-gw73xx-0x-imx219.dtso} | 0
> ...=> imx8mm-venice-gw73xx-0x-rs232-rts.dtso} | 0
> ...dts => imx8mm-venice-gw73xx-0x-rs422.dtso} | 0
> ...dts => imx8mm-venice-gw73xx-0x-rs485.dtso} | 0
> ...2.dts => draak-ebisu-panel-aa104xd12.dtso} | 0
> ...xd12.dts => salvator-panel-aa104xd12.dtso} | 0
> ...v-g-revA.dts => zynqmp-sck-kv-g-revA.dtso} | 0
> ...v-g-revB.dts => zynqmp-sck-kv-g-revB.dtso} | 0
> drivers/of/unittest-data/Makefile | 66 +++++++++----------
> .../{overlay.dts => overlay.dtso} | 0
> .../{overlay_0.dts => overlay_0.dtso} | 0
> .../{overlay_1.dts => overlay_1.dtso} | 0
> .../{overlay_10.dts => overlay_10.dtso} | 0
> .../{overlay_11.dts => overlay_11.dtso} | 0
> .../{overlay_12.dts => overlay_12.dtso} | 0
> .../{overlay_13.dts => overlay_13.dtso} | 0
> .../{overlay_15.dts => overlay_15.dtso} | 0
> .../{overlay_16.dts => overlay_16.dtso} | 0
> .../{overlay_17.dts => overlay_17.dtso} | 0
> .../{overlay_18.dts => overlay_18.dtso} | 0
> .../{overlay_19.dts => overlay_19.dtso} | 0
> .../{overlay_2.dts => overlay_2.dtso} | 0
> .../{overlay_20.dts => overlay_20.dtso} | 0
> .../{overlay_3.dts => overlay_3.dtso} | 0
> .../{overlay_4.dts => overlay_4.dtso} | 0
> .../{overlay_5.dts => overlay_5.dtso} | 0
> .../{overlay_6.dts => overlay_6.dtso} | 0
> .../{overlay_7.dts => overlay_7.dtso} | 0
> .../{overlay_8.dts => overlay_8.dtso} | 0
> .../{overlay_9.dts => overlay_9.dtso} | 0
> ...node.dts => overlay_bad_add_dup_node.dtso} | 0
> ...prop.dts => overlay_bad_add_dup_prop.dtso} | 0
> ...d_phandle.dts => overlay_bad_phandle.dtso} | 0
> ...bad_symbol.dts => overlay_bad_symbol.dtso} | 0
> .../{overlay_base.dts => overlay_base.dtso} | 0
> ...erlay_gpio_01.dts => overlay_gpio_01.dtso} | 0
> ...lay_gpio_02a.dts => overlay_gpio_02a.dtso} | 0
> ...lay_gpio_02b.dts => overlay_gpio_02b.dtso} | 0
> ...erlay_gpio_03.dts => overlay_gpio_03.dtso} | 0
> ...lay_gpio_04a.dts => overlay_gpio_04a.dtso} | 0
> ...lay_gpio_04b.dts => overlay_gpio_04b.dtso} | 0
> .../{testcases.dts => testcases.dtso} | 0
> drivers/of/unittest.c | 48 +++++++-------
> .../{pi433-overlay.dts => pi433-overlay.dtso} | 0
> .../pi433/Documentation/devicetree/pi433.txt | 6 +-
> scripts/Makefile.lib | 23 ++++++-
> 56 files changed, 82 insertions(+), 61 deletions(-)
> rename arch/arm64/boot/dts/freescale/{fsl-ls1028a-qds-13bb.dts => fsl-ls1028a-qds-13bb.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{fsl-ls1028a-qds-65bb.dts => fsl-ls1028a-qds-65bb.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{fsl-ls1028a-qds-7777.dts => fsl-ls1028a-qds-7777.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{fsl-ls1028a-qds-85bb.dts => fsl-ls1028a-qds-85bb.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{fsl-ls1028a-qds-899b.dts => fsl-ls1028a-qds-899b.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{fsl-ls1028a-qds-9999.dts => fsl-ls1028a-qds-9999.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw72xx-0x-imx219.dts => imx8mm-venice-gw72xx-0x-imx219.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw72xx-0x-rs232-rts.dts => imx8mm-venice-gw72xx-0x-rs232-rts.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw72xx-0x-rs422.dts => imx8mm-venice-gw72xx-0x-rs422.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw72xx-0x-rs485.dts => imx8mm-venice-gw72xx-0x-rs485.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw73xx-0x-imx219.dts => imx8mm-venice-gw73xx-0x-imx219.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw73xx-0x-rs232-rts.dts => imx8mm-venice-gw73xx-0x-rs232-rts.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw73xx-0x-rs422.dts => imx8mm-venice-gw73xx-0x-rs422.dtso} (100%)
> rename arch/arm64/boot/dts/freescale/{imx8mm-venice-gw73xx-0x-rs485.dts => imx8mm-venice-gw73xx-0x-rs485.dtso} (100%)
> rename arch/arm64/boot/dts/renesas/{draak-ebisu-panel-aa104xd12.dts => draak-ebisu-panel-aa104xd12.dtso} (100%)
> rename arch/arm64/boot/dts/renesas/{salvator-panel-aa104xd12.dts => salvator-panel-aa104xd12.dtso} (100%)
> rename arch/arm64/boot/dts/xilinx/{zynqmp-sck-kv-g-revA.dts => zynqmp-sck-kv-g-revA.dtso} (100%)
> rename arch/arm64/boot/dts/xilinx/{zynqmp-sck-kv-g-revB.dts => zynqmp-sck-kv-g-revB.dtso} (100%)
> rename drivers/of/unittest-data/{overlay.dts => overlay.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_0.dts => overlay_0.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_1.dts => overlay_1.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_10.dts => overlay_10.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_11.dts => overlay_11.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_12.dts => overlay_12.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_13.dts => overlay_13.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_15.dts => overlay_15.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_16.dts => overlay_16.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_17.dts => overlay_17.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_18.dts => overlay_18.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_19.dts => overlay_19.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_2.dts => overlay_2.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_20.dts => overlay_20.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_3.dts => overlay_3.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_4.dts => overlay_4.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_5.dts => overlay_5.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_6.dts => overlay_6.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_7.dts => overlay_7.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_8.dts => overlay_8.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_9.dts => overlay_9.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_bad_add_dup_node.dts => overlay_bad_add_dup_node.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_bad_add_dup_prop.dts => overlay_bad_add_dup_prop.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_bad_phandle.dts => overlay_bad_phandle.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_bad_symbol.dts => overlay_bad_symbol.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_base.dts => overlay_base.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_gpio_01.dts => overlay_gpio_01.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_gpio_02a.dts => overlay_gpio_02a.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_gpio_02b.dts => overlay_gpio_02b.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_gpio_03.dts => overlay_gpio_03.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_gpio_04a.dts => overlay_gpio_04a.dtso} (100%)
> rename drivers/of/unittest-data/{overlay_gpio_04b.dts => overlay_gpio_04b.dtso} (100%)
> rename drivers/of/unittest-data/{testcases.dts => testcases.dtso} (100%)
> rename drivers/staging/pi433/Documentation/devicetree/{pi433-overlay.dts => pi433-overlay.dtso} (100%)
>
next prev parent reply other threads:[~2022-10-24 23:47 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-24 17:34 [PATCH v2 0/7] Rename DTB overlay source files Andrew Davis
2022-10-24 17:34 ` [PATCH v2 1/7] kbuild: Allow DTB overlays to built from .dtso named " Andrew Davis
2022-10-24 22:01 ` Frank Rowand
2022-10-24 17:34 ` [PATCH v2 2/7] kbuild: Allow DTB overlays to built into .dtso.S files Andrew Davis
2022-10-24 22:01 ` Frank Rowand
2022-10-24 17:34 ` [PATCH v2 3/7] of: overlay: rename overlay source files from .dts to .dtso Andrew Davis
2022-10-24 22:01 ` Frank Rowand
2022-10-24 17:34 ` [PATCH v2 4/7] arm64: dts: freescale: Rename DTB " Andrew Davis
2022-11-19 6:32 ` Shawn Guo
2022-10-24 17:34 ` [PATCH v2 5/7] arm64: dts: renesas: " Andrew Davis
2022-10-24 17:34 ` [PATCH v2 6/7] arm64: dts: xilinx: " Andrew Davis
2022-11-30 16:50 ` Michal Simek
2022-10-24 17:34 ` [PATCH v2 7/7] staging: pi433: overlay: Rename overlay source file " Andrew Davis
2022-10-25 6:57 ` Geert Uytterhoeven
2022-10-24 22:04 ` Frank Rowand [this message]
2022-10-26 14:11 ` [PATCH v2 0/7] Rename DTB overlay source files Rob Herring
2022-11-11 19:39 ` Andy Shevchenko
2022-11-11 19:40 ` Andy Shevchenko
2022-11-11 21:05 ` Andrew Davis
2022-11-13 12:20 ` Andy Shevchenko
2022-11-13 12:36 ` Andy Shevchenko
2022-11-14 16:06 ` Andy Shevchenko
2022-11-14 20:43 ` Andrew Davis
2022-11-15 11:29 ` Andy Shevchenko
2022-11-14 14:39 ` Andy Shevchenko
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=51477b80-478f-d2cb-d98d-477431057fd4@gmail.com \
--to=frowand.list@gmail.com \
--cc=afd@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=geert@linux-m68k.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=leoyang.li@nxp.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=masahiroy@kernel.org \
--cc=michal.lkml@markovi.net \
--cc=ndesaulniers@google.com \
--cc=nm@ti.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=vigneshr@ti.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).