Thread (25 messages) 25 messages, 7 authors, 2023-03-10

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

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 tree
Use 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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help