Re: [PATCH v1 3/3] arch/arm: dts: introduce meson-a1 device tree
From: Krzysztof Kozlowski <hidden>
Date: 2023-02-23 09:09:38
Also in:
linux-amlogic, linux-devicetree, lkml
From: Krzysztof Kozlowski <hidden>
Date: 2023-02-23 09:09:38
Also in:
linux-amlogic, linux-devicetree, lkml
On 23/02/2023 10:08, Krzysztof Kozlowski wrote:
On 22/02/2023 12:50, Alexey Romanov wrote:quoted
Add basic support for the 32-bit Amlogic A1. This device treeUse subject prefixes matching the subsystem (which you can get for example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory your patch is touching).quoted
describes following compontents: CPU, GIC, IRQ, Timer, UART, PIN controller. It's capable of booting up into the serial console. This is based on arm64 version of meson-a1.dtsi. Signed-off-by: Alexey Romanov <redacted> --- arch/arm/boot/dts/meson-a1.dtsi | 151 ++++++++++++++++++++++++++++++++There is such file and there is such DTS/hardware support. I don't see any reason why entire DTSI should be duplicated. What's more, your commit does not explain it - does not justify duplication.
One more comment - I think you just added dead code. It's uncompilable/untestable. Otherwise, please share how to build this DTSI without DTS. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel