Skip to content

ARM: dts: AM33XX: node definition cleanups

Benoit Cousson linux-arm-kernel linux-omap 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 2 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 2 patches the cover letter announces are in this archive.

Reviews trailers per patch

PatchRajendra Nayak
SeriesT-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/2] ARM: dts: AM33XX: node definition cleanups

+ Few TI folks to review and test.

Quoted 19 lines · Javier Martinez Canillas

Hi,

While reviewing the submission of a DT file for a new board based on the am3352 SoC, I noticed that am33xx*.dts files needed some cleanups. All am33xx based board files redefine device nodes that are already defined in the included am33xx.dtsi file.

This not only makes the DTS harder to maintain without any benefit but also leaks SoC internal details to board files such as how the OCP interconnect is modeled (currently just a flat representation) and the addresses of some IP modules that are part of the SoC and are not going to change from board to board.

This patch-set cleans these board files and is composed of the following patches:

[PATCH 1/2] ARM: dts: AM33XX: use pinmux node defined in included [PATCH 2/2] ARM: dts: AM33XX: don't redefine OCP bus and device

Quoted 7 lines

NOTE: I don't own any of these am33xx based boards so I was only able to build test the changes. However I compared the generated dtb before and after the changes and the disassembled DTS with dtc -I dtb -O dts foo.dtb > bar.dts are identical.

Even though the binary dtb is the same and there shouldn't be any visible changes, boot testing will be highly appreciated.

Thanks it looks good to me.

Regards,
Benoit

Read the cover letter in its thread