Re: [PATCH v2 1/2] dt-bindings: arm: amlogic: add amediatech x96-max bindings
From: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Date: 2018-12-22 16:40:11
Also in:
linux-amlogic, linux-arm-kernel
Hi Kevin, On Thu, Dec 20, 2018 at 12:35 AM Kevin Hilman [off-list ref] wrote:
Add new vendor for amediatech, and initial board: x96-max
as we have seen there is no "amediatech" branding anywhere on the board itself however, FCC gave us a clue about the manufacturer: [0] can you please add this info to the patch description when re-sending?
Signed-off-by: Kevin Hilman <khilman@baylibre.com>
Acked-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> (with the two comments below)
Documentation/devicetree/bindings/arm/amlogic.txt | 1 + Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
if Rob is fine with vendor-prefixes.txt being updated together with adding a new board then I'm as well :)
quoted hunk ↗ jump to hunk
2 files changed, 2 insertions(+)diff --git a/Documentation/devicetree/bindings/arm/amlogic.txt b/Documentation/devicetree/bindings/arm/amlogic.txt index 8dbc259081e4..c50b084b6c59 100644 --- a/Documentation/devicetree/bindings/arm/amlogic.txt +++ b/Documentation/devicetree/bindings/arm/amlogic.txt@@ -99,6 +99,7 @@ Board compatible values (alphabetically, grouped by SoC): - "amlogic,p281" (Meson gxl s905w) - "oranth,tx3-mini" (Meson gxl s905w) + - "amediatech,x96-max" (Meson gxm s912)
I assume that you'll not only fix the SoC here (G12A) as reported by Chris Moore but also the ordering Regards Martin [0] https://fccid.io/2AI6D-X96MAX