Re: [PATCH v2 0/7] Rename DTB overlay source files
From: Frank Rowand
Date: Mon Oct 24 2022 - 19:47:26 EST
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%)
>