Thread (2 messages) 2 messages, 1 author, 2016-08-29
STALE3615d

[PATCH v2] ARM64: dts: amlogic: Add basic support for Amlogic S905X

From: khilman@baylibre.com (Kevin Hilman)
Date: 2016-08-29 17:46:55
Also in: linux-arm-kernel

Carlo Caione [off-list ref] writes:
From: Carlo Caione <redacted>

This patch introduces the basic support for the Amlogic S905X (Meson
GXL) and for the Amlogic evaluation board P212.
No documentation has been released yet for this SoC, so for now only the
bare minimum has been added in the DT.

Acked-by: Rob Herring <robh@kernel.org>
Reviewed-by: Andreas F?rber <afaerber@suse.de>
Signed-off-by: Carlo Caione <redacted>
---

Hey Kevin,
Can you take this one?

Changelog:
v2:
  - Added acked-by and reviewed-by

---
 Documentation/devicetree/bindings/arm/amlogic.txt |   5 +
 arch/arm64/boot/dts/amlogic/Makefile              |   1 +
 arch/arm64/boot/dts/amlogic/meson-gxl-p212.dts    |  70 +++++++++
 arch/arm64/boot/dts/amlogic/meson-gxl.dtsi        | 179 ++++++++++++++++++++++
It seems to me that meson-gxl.dtsi is almost identicial to
meson-gxbb.dtsi.  I think we should create another .dtsi for all the
common parts.  Something like meson-gx-common.dtsi, which is included
by meson-gxbb.dtsi and meson-gxl.dtsi

Also, the GXL family has both the S905x and S905D SoCs, so we may need
meson-gxl-s905x.dtsi and meson-gxl-s905d.dtsi but I'm not sure of the
full set of differences between those, so we can save that for later.

But, I do think the board files should probably have the SoC name.  So
they would look something like: meson-gxl-s905x-p212.dts (which you
have) and meson-gxl-s905d-p230.dts (which I have.)

Kevin
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help