+ Few TI folks to review and test. On 10/08/2013 19:05, Javier Martinez Canillas wrote:
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 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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
