mbox series

[0/3] arm64: dts: renesas: falcon: Restructuring

Message ID 20210316154705.2433528-1-geert+renesas@glider.be (mailing list archive)
Headers show
Series arm64: dts: renesas: falcon: Restructuring | expand

Message

Geert Uytterhoeven March 16, 2021, 3:47 p.m. UTC 
Hi all,

This patch series restructures the devices nodes in the Renesas Falcon
DTS board stack to better match the actual board stack structure.
The DTB generated after this series is the same as before, according to
scripts/dtc/dtx_diff.

I plan to queue this in renesas-devel for v5.13.

Geert Uytterhoeven (3):
  arm64: dts: renesas: falcon: Move console config to CPU board DTS
  arm64: dts: renesas: falcon: Move watchdog config to CPU board DTS
  arm64: dts: renesas: falcon: Move AVB0 to main DTS

 .../boot/dts/renesas/r8a779a0-falcon-cpu.dtsi | 47 +++++--------------
 .../boot/dts/renesas/r8a779a0-falcon.dts      | 39 ++++++++++++---
 2 files changed, 46 insertions(+), 40 deletions(-)

Comments

Wolfram Sang March 17, 2021, 8:37 a.m. UTC | #1 
On Tue, Mar 16, 2021 at 04:47:02PM +0100, Geert Uytterhoeven wrote:
> 	Hi all,
> 
> This patch series restructures the devices nodes in the Renesas Falcon
> DTS board stack to better match the actual board stack structure.
> The DTB generated after this series is the same as before, according to
> scripts/dtc/dtx_diff.
> 
> I plan to queue this in renesas-devel for v5.13.
> 
> Geert Uytterhoeven (3):
>   arm64: dts: renesas: falcon: Move console config to CPU board DTS
>   arm64: dts: renesas: falcon: Move watchdog config to CPU board DTS
>   arm64: dts: renesas: falcon: Move AVB0 to main DTS

Yes, it is better this way:

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Console, watchdog and eth0 still work:

Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Niklas Söderlund March 17, 2021, 9:03 a.m. UTC | #2 
Hi Geert,

Thanks for your work.

On 2021-03-16 16:47:02 +0100, Geert Uytterhoeven wrote:
> 	Hi all,
> 
> This patch series restructures the devices nodes in the Renesas Falcon
> DTS board stack to better match the actual board stack structure.
> The DTB generated after this series is the same as before, according to
> scripts/dtc/dtx_diff.
> 
> I plan to queue this in renesas-devel for v5.13.
> 
> Geert Uytterhoeven (3):
>   arm64: dts: renesas: falcon: Move console config to CPU board DTS
>   arm64: dts: renesas: falcon: Move watchdog config to CPU board DTS
>   arm64: dts: renesas: falcon: Move AVB0 to main DTS

These looks good to me,

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

> 
>  .../boot/dts/renesas/r8a779a0-falcon-cpu.dtsi | 47 +++++--------------
>  .../boot/dts/renesas/r8a779a0-falcon.dts      | 39 ++++++++++++---
>  2 files changed, 46 insertions(+), 40 deletions(-)
> 
> -- 
> 2.25.1
> 
> Gr{oetje,eeting}s,
> 
> 						Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> 							    -- Linus Torvalds