[v2,0/7] Rename DTB overlay source files

Message ID 20221024173434.32518-1-afd@ti.com
Headers
Series Rename DTB overlay source files |

Message

Andrew Davis Oct. 24, 2022, 5:34 p.m. UTC
  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,
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%)
  

Comments

Frank Rowand Oct. 24, 2022, 10:04 p.m. UTC | #1
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%)
>
  
Rob Herring Oct. 26, 2022, 2:11 p.m. UTC | #2
On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> 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

I've applied patches 1-3 and 7. I'll send a PR for the branch to the
platform maintainers after a few days in linux-next.

Rob
  
Andy Shevchenko Nov. 11, 2022, 7:39 p.m. UTC | #3
On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > 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
> 
> I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> platform maintainers after a few days in linux-next.

The patch

commit 941214a512d8c80d47e720c17ec17e8539175e93
Author: Andrew Davis <afd@ti.com>
Date:   Mon Oct 24 12:34:29 2022 -0500

    kbuild: Allow DTB overlays to built into .dtbo.S files

broke the build reproducibility / no-op builds.

Before:
  2+ execution of `make` on non-changed tree did nothing

Now:
  Each run of `make` (even without a single bit changed) restarts vmlinux
  rebuild.

Please, revert or fix.
  
Andy Shevchenko Nov. 11, 2022, 7:40 p.m. UTC | #4
On Fri, Nov 11, 2022 at 09:39:57PM +0200, Andy Shevchenko wrote:
> On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> > On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > > 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
> > 
> > I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> > platform maintainers after a few days in linux-next.
> 
> The patch
> 
> commit 941214a512d8c80d47e720c17ec17e8539175e93
> Author: Andrew Davis <afd@ti.com>
> Date:   Mon Oct 24 12:34:29 2022 -0500
> 
>     kbuild: Allow DTB overlays to built into .dtbo.S files
> 
> broke the build reproducibility / no-op builds.
> 
> Before:
>   2+ execution of `make` on non-changed tree did nothing
> 
> Now:
>   Each run of `make` (even without a single bit changed) restarts vmlinux
>   rebuild.
> 
> Please, revert or fix.

git bisect start
# good: [f0c4d9fc9cc9462659728d168387191387e903cc] Linux 6.1-rc4
git bisect good f0c4d9fc9cc9462659728d168387191387e903cc
# bad: [f8f60f322f0640c8edda2942ca5f84b7a27c417a] Add linux-next specific files for 20221111
git bisect bad f8f60f322f0640c8edda2942ca5f84b7a27c417a
# good: [66500d0e2e5c9b64fc5d3f4879e2140572e1b386] Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git
git bisect good 66500d0e2e5c9b64fc5d3f4879e2140572e1b386
# bad: [080c1179d77ad36d6e51469b68801ef341ec292a] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git
git bisect bad 080c1179d77ad36d6e51469b68801ef341ec292a
# good: [7f5ff6593cc1b1ec2442be30eaf043cf09fa9282] Merge branch 'for-next' of https://gitlab.freedesktop.org/drm/tegra.git
git bisect good 7f5ff6593cc1b1ec2442be30eaf043cf09fa9282
# good: [c16d93a496889347965a860dc4fe493b642f8aff] Merge branch 'for-next' of git://git.kernel.dk/linux-block.git
git bisect good c16d93a496889347965a860dc4fe493b642f8aff
# good: [c9ef15a7221d84edcf03b422ef0a4e43affda226] Merge branch 'next-integrity' of git://git.kernel.org/pub/scm/linux/kernel/git/zohar/linux-integrity
git bisect good c9ef15a7221d84edcf03b422ef0a4e43affda226
# good: [736894df52110a9e2e106708d848ab08516e5910] Merge branch 'next/dt' into for-next
git bisect good 736894df52110a9e2e106708d848ab08516e5910
# good: [d19d44e32c93cfa815b22ee2254cd1f541a20116] Merge branch 'next' of git://git.kernel.org/pub/scm/linux/kernel/git/pcmoore/audit.git
git bisect good d19d44e32c93cfa815b22ee2254cd1f541a20116
# good: [93cc2559d3fdcd28b1a7972ab519a6cd8ba20f9d] spi: Remove the obsolte u64_stats_fetch_*_irq() users.
git bisect good 93cc2559d3fdcd28b1a7972ab519a6cd8ba20f9d
# bad: [26c9134a370ace32cda7a3f9efaf4ca85e57ca8d] Merge branch 'dt/dtbo-rename' into dt/next
git bisect bad 26c9134a370ace32cda7a3f9efaf4ca85e57ca8d
# good: [c9adc3bd1180d2911d2b1b886dbec916805be998] dt-bindings: Remove "status" from schema examples, again
git bisect good c9adc3bd1180d2911d2b1b886dbec916805be998
# bad: [fa9665ef77f5d8e861e2ed7563ebdbddddc6f82b] staging: pi433: overlay: Rename overlay source file from .dts to .dtso
git bisect bad fa9665ef77f5d8e861e2ed7563ebdbddddc6f82b
# bad: [941214a512d8c80d47e720c17ec17e8539175e93] kbuild: Allow DTB overlays to built into .dtbo.S files
git bisect bad 941214a512d8c80d47e720c17ec17e8539175e93
# good: [363547d2191cbc32ca954ba75d72908712398ff2] kbuild: Allow DTB overlays to built from .dtso named source files
git bisect good 363547d2191cbc32ca954ba75d72908712398ff2
# first bad commit: [941214a512d8c80d47e720c17ec17e8539175e93] kbuild: Allow DTB overlays to built into .dtbo.S files
  
Andrew Davis Nov. 11, 2022, 9:05 p.m. UTC | #5
On 11/11/22 1:39 PM, Andy Shevchenko wrote:
> On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
>> On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
>>> 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
>>
>> I've applied patches 1-3 and 7. I'll send a PR for the branch to the
>> platform maintainers after a few days in linux-next.
> 
> The patch
> 
> commit 941214a512d8c80d47e720c17ec17e8539175e93
> Author: Andrew Davis <afd@ti.com>
> Date:   Mon Oct 24 12:34:29 2022 -0500
> 
>      kbuild: Allow DTB overlays to built into .dtbo.S files
> 
> broke the build reproducibility / no-op builds.
> 
> Before:
>    2+ execution of `make` on non-changed tree did nothing
> 
> Now:
>    Each run of `make` (even without a single bit changed) restarts vmlinux
>    rebuild.
> 
> Please, revert or fix.
> 

I do not see this behavior. What config are you using?

Not sure how this patch could be the root cause, it only adds
a build target/rule, but doesn't actually use it anywhere yet..

Andrew
  
Andy Shevchenko Nov. 13, 2022, 12:20 p.m. UTC | #6
On Fri, Nov 11, 2022 at 03:05:20PM -0600, Andrew Davis wrote:
> On 11/11/22 1:39 PM, Andy Shevchenko wrote:
> > On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> > > On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > > > 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
> > > 
> > > I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> > > platform maintainers after a few days in linux-next.
> > 
> > The patch
> > 
> > commit 941214a512d8c80d47e720c17ec17e8539175e93
> > Author: Andrew Davis <afd@ti.com>
> > Date:   Mon Oct 24 12:34:29 2022 -0500
> > 
> >      kbuild: Allow DTB overlays to built into .dtbo.S files
> > 
> > broke the build reproducibility / no-op builds.
> > 
> > Before:
> >    2+ execution of `make` on non-changed tree did nothing
> > 
> > Now:
> >    Each run of `make` (even without a single bit changed) restarts vmlinux
> >    rebuild.
> > 
> > Please, revert or fix.
> > 
> 
> I do not see this behavior. What config are you using?
> 
> Not sure how this patch could be the root cause, it only adds
> a build target/rule, but doesn't actually use it anywhere yet..

For your reference I started with this one [1].

When I bisected, I just answered with defaults on whatever `make` told me at
the configuration stage.

The actual `make` command I used:

	make O=/path/to/the/result W=1 C=1 CF=-D__CHECK_ENDIAN__ -j64

But there is nothing that can affect the described issue.

[1]: https://p.defau.lt/?ZSOdGnNxF9v9AQtrfDo_KQ
  
Andy Shevchenko Nov. 13, 2022, 12:36 p.m. UTC | #7
On Sun, Nov 13, 2022 at 02:20:48PM +0200, Andy Shevchenko wrote:
> On Fri, Nov 11, 2022 at 03:05:20PM -0600, Andrew Davis wrote:
> > On 11/11/22 1:39 PM, Andy Shevchenko wrote:
> > > On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> > > > On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > > > > 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
> > > > 
> > > > I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> > > > platform maintainers after a few days in linux-next.
> > > 
> > > The patch
> > > 
> > > commit 941214a512d8c80d47e720c17ec17e8539175e93
> > > Author: Andrew Davis <afd@ti.com>
> > > Date:   Mon Oct 24 12:34:29 2022 -0500
> > > 
> > >      kbuild: Allow DTB overlays to built into .dtbo.S files
> > > 
> > > broke the build reproducibility / no-op builds.
> > > 
> > > Before:
> > >    2+ execution of `make` on non-changed tree did nothing
> > > 
> > > Now:
> > >    Each run of `make` (even without a single bit changed) restarts vmlinux
> > >    rebuild.
> > > 
> > > Please, revert or fix.
> > > 
> > 
> > I do not see this behavior. What config are you using?
> > 
> > Not sure how this patch could be the root cause, it only adds
> > a build target/rule, but doesn't actually use it anywhere yet..
> 
> For your reference I started with this one [1].
> 
> When I bisected, I just answered with defaults on whatever `make` told me at
> the configuration stage.
> 
> The actual `make` command I used:
> 
> 	make O=/path/to/the/result W=1 C=1 CF=-D__CHECK_ENDIAN__ -j64
> 
> But there is nothing that can affect the described issue.

Actually, O= might affect which Makefile is used and how.
The C=, CF= are sparse flags, W= is just warning level.

> [1]: https://p.defau.lt/?ZSOdGnNxF9v9AQtrfDo_KQ
  
Andy Shevchenko Nov. 14, 2022, 2:39 p.m. UTC | #8
On Fri, Nov 11, 2022 at 03:05:20PM -0600, Andrew Davis wrote:
> On 11/11/22 1:39 PM, Andy Shevchenko wrote:
> > On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> > > On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > > > 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
> > > 
> > > I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> > > platform maintainers after a few days in linux-next.
> > 
> > The patch
> > 
> > commit 941214a512d8c80d47e720c17ec17e8539175e93
> > Author: Andrew Davis <afd@ti.com>
> > Date:   Mon Oct 24 12:34:29 2022 -0500
> > 
> >      kbuild: Allow DTB overlays to built into .dtbo.S files
> > 
> > broke the build reproducibility / no-op builds.
> > 
> > Before:
> >    2+ execution of `make` on non-changed tree did nothing
> > 
> > Now:
> >    Each run of `make` (even without a single bit changed) restarts vmlinux
> >    rebuild.
> > 
> > Please, revert or fix.
> > 
> 
> I do not see this behavior. What config are you using?
> 
> Not sure how this patch could be the root cause, it only adds
> a build target/rule, but doesn't actually use it anywhere yet..

Just to confirm, I reverted the patch and everything went fine again,
so simple revert helps!

(Tested on today's Linux Next)
  
Andy Shevchenko Nov. 14, 2022, 4:06 p.m. UTC | #9
On Sun, Nov 13, 2022 at 02:36:33PM +0200, Andy Shevchenko wrote:
> On Sun, Nov 13, 2022 at 02:20:48PM +0200, Andy Shevchenko wrote:
> > On Fri, Nov 11, 2022 at 03:05:20PM -0600, Andrew Davis wrote:
> > > On 11/11/22 1:39 PM, Andy Shevchenko wrote:
> > > > On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> > > > > On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > > > > > 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
> > > > > 
> > > > > I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> > > > > platform maintainers after a few days in linux-next.
> > > > 
> > > > The patch
> > > > 
> > > > commit 941214a512d8c80d47e720c17ec17e8539175e93
> > > > Author: Andrew Davis <afd@ti.com>
> > > > Date:   Mon Oct 24 12:34:29 2022 -0500
> > > > 
> > > >      kbuild: Allow DTB overlays to built into .dtbo.S files
> > > > 
> > > > broke the build reproducibility / no-op builds.
> > > > 
> > > > Before:
> > > >    2+ execution of `make` on non-changed tree did nothing
> > > > 
> > > > Now:
> > > >    Each run of `make` (even without a single bit changed) restarts vmlinux
> > > >    rebuild.
> > > > 
> > > > Please, revert or fix.
> > > > 
> > > 
> > > I do not see this behavior. What config are you using?
> > > 
> > > Not sure how this patch could be the root cause, it only adds
> > > a build target/rule, but doesn't actually use it anywhere yet..
> > 
> > For your reference I started with this one [1].
> > 
> > When I bisected, I just answered with defaults on whatever `make` told me at
> > the configuration stage.
> > 
> > The actual `make` command I used:
> > 
> > 	make O=/path/to/the/result W=1 C=1 CF=-D__CHECK_ENDIAN__ -j64
> > 
> > But there is nothing that can affect the described issue.
> 
> Actually, O= might affect which Makefile is used and how.
> The C=, CF= are sparse flags, W= is just warning level.

As far as I can tell right now it's the OF_UNITTEST on x86_64 that makes the
above mentioned patch to be a culprit. Not sure if on ARM / ARM64 you can
reproduce that. And it's really strange nobody reported this for a week+.

Whatever, I'm open for the suggestions and material to test.

> > [1]: https://p.defau.lt/?ZSOdGnNxF9v9AQtrfDo_KQ
  
Andrew Davis Nov. 14, 2022, 8:43 p.m. UTC | #10
On 11/14/22 10:06 AM, Andy Shevchenko wrote:
> On Sun, Nov 13, 2022 at 02:36:33PM +0200, Andy Shevchenko wrote:
>> On Sun, Nov 13, 2022 at 02:20:48PM +0200, Andy Shevchenko wrote:
>>> On Fri, Nov 11, 2022 at 03:05:20PM -0600, Andrew Davis wrote:
>>>> On 11/11/22 1:39 PM, Andy Shevchenko wrote:
>>>>> On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
>>>>>> On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
>>>>>>> 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
>>>>>>
>>>>>> I've applied patches 1-3 and 7. I'll send a PR for the branch to the
>>>>>> platform maintainers after a few days in linux-next.
>>>>>
>>>>> The patch
>>>>>
>>>>> commit 941214a512d8c80d47e720c17ec17e8539175e93
>>>>> Author: Andrew Davis <afd@ti.com>
>>>>> Date:   Mon Oct 24 12:34:29 2022 -0500
>>>>>
>>>>>       kbuild: Allow DTB overlays to built into .dtbo.S files
>>>>>
>>>>> broke the build reproducibility / no-op builds.
>>>>>
>>>>> Before:
>>>>>     2+ execution of `make` on non-changed tree did nothing
>>>>>
>>>>> Now:
>>>>>     Each run of `make` (even without a single bit changed) restarts vmlinux
>>>>>     rebuild.
>>>>>
>>>>> Please, revert or fix.
>>>>>
>>>>
>>>> I do not see this behavior. What config are you using?
>>>>
>>>> Not sure how this patch could be the root cause, it only adds
>>>> a build target/rule, but doesn't actually use it anywhere yet..
>>>
>>> For your reference I started with this one [1].
>>>
>>> When I bisected, I just answered with defaults on whatever `make` told me at
>>> the configuration stage.
>>>
>>> The actual `make` command I used:
>>>
>>> 	make O=/path/to/the/result W=1 C=1 CF=-D__CHECK_ENDIAN__ -j64
>>>
>>> But there is nothing that can affect the described issue.
>>
>> Actually, O= might affect which Makefile is used and how.
>> The C=, CF= are sparse flags, W= is just warning level.
> 
> As far as I can tell right now it's the OF_UNITTEST on x86_64 that makes the
> above mentioned patch to be a culprit. Not sure if on ARM / ARM64 you can
> reproduce that. And it's really strange nobody reported this for a week+.
> 
> Whatever, I'm open for the suggestions and material to test.
> 

I think I found the issue, we forgot to add the new dtbo.S/o files
to the list of preserved intermediate targets, so Make was
removing them after build.

Sending the fix now.

Thanks,
Andrew

>>> [1]: https://p.defau.lt/?ZSOdGnNxF9v9AQtrfDo_KQ
>
  
Andy Shevchenko Nov. 15, 2022, 11:29 a.m. UTC | #11
On Mon, Nov 14, 2022 at 02:43:51PM -0600, Andrew Davis wrote:
> On 11/14/22 10:06 AM, Andy Shevchenko wrote:
> > On Sun, Nov 13, 2022 at 02:36:33PM +0200, Andy Shevchenko wrote:
> > > On Sun, Nov 13, 2022 at 02:20:48PM +0200, Andy Shevchenko wrote:
> > > > On Fri, Nov 11, 2022 at 03:05:20PM -0600, Andrew Davis wrote:
> > > > > On 11/11/22 1:39 PM, Andy Shevchenko wrote:
> > > > > > On Wed, Oct 26, 2022 at 09:11:21AM -0500, Rob Herring wrote:
> > > > > > > On Mon, Oct 24, 2022 at 12:34 PM Andrew Davis <afd@ti.com> 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,
> > > > > > > > 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
> > > > > > > 
> > > > > > > I've applied patches 1-3 and 7. I'll send a PR for the branch to the
> > > > > > > platform maintainers after a few days in linux-next.
> > > > > > 
> > > > > > The patch
> > > > > > 
> > > > > > commit 941214a512d8c80d47e720c17ec17e8539175e93
> > > > > > Author: Andrew Davis <afd@ti.com>
> > > > > > Date:   Mon Oct 24 12:34:29 2022 -0500
> > > > > > 
> > > > > >       kbuild: Allow DTB overlays to built into .dtbo.S files
> > > > > > 
> > > > > > broke the build reproducibility / no-op builds.
> > > > > > 
> > > > > > Before:
> > > > > >     2+ execution of `make` on non-changed tree did nothing
> > > > > > 
> > > > > > Now:
> > > > > >     Each run of `make` (even without a single bit changed) restarts vmlinux
> > > > > >     rebuild.
> > > > > > 
> > > > > > Please, revert or fix.
> > > > > > 
> > > > > 
> > > > > I do not see this behavior. What config are you using?
> > > > > 
> > > > > Not sure how this patch could be the root cause, it only adds
> > > > > a build target/rule, but doesn't actually use it anywhere yet..
> > > > 
> > > > For your reference I started with this one [1].
> > > > 
> > > > When I bisected, I just answered with defaults on whatever `make` told me at
> > > > the configuration stage.
> > > > 
> > > > The actual `make` command I used:
> > > > 
> > > > 	make O=/path/to/the/result W=1 C=1 CF=-D__CHECK_ENDIAN__ -j64
> > > > 
> > > > But there is nothing that can affect the described issue.
> > > 
> > > Actually, O= might affect which Makefile is used and how.
> > > The C=, CF= are sparse flags, W= is just warning level.
> > 
> > As far as I can tell right now it's the OF_UNITTEST on x86_64 that makes the
> > above mentioned patch to be a culprit. Not sure if on ARM / ARM64 you can
> > reproduce that. And it's really strange nobody reported this for a week+.
> > 
> > Whatever, I'm open for the suggestions and material to test.
> 
> I think I found the issue, we forgot to add the new dtbo.S/o files
> to the list of preserved intermediate targets, so Make was
> removing them after build.
> 
> Sending the fix now.

I'll test it ASAP and reply there.
Thank you.

> > > > [1]: https://p.defau.lt/?ZSOdGnNxF9v9AQtrfDo_KQ