Thread (10 messages) 10 messages, 3 authors, 2026-01-14

Re: [PATCH v2 1/2] dt-bindings: arm: amlogic: add Khadas VIM1S binding

From: Nick Xie <hidden>
Date: 2026-01-14 11:25:47
Also in: linux-amlogic, linux-devicetree, lkml

Krzysztof Kozlowski [off-list ref] 于2026年1月14日周三 18:33写道:
On 14/01/2026 11:07, Nick Xie wrote:
quoted
Krzysztof Kozlowski [off-list ref] 于2026年1月14日周三 16:51写道:
quoted
On Wed, Jan 14, 2026 at 02:25:48PM +0800, Nick Xie wrote:
quoted
Add devicetree binding for Khadas VIM1S board based on
Amlogic S4 S905Y4 SoC.

Signed-off-by: Nick Xie <redacted>
You still have the same From/SoB mismatch.
OK,  I will fix it in next version.
quoted
quoted
---
 Documentation/devicetree/bindings/arm/amlogic.yaml | 6 ++++++
 1 file changed, 6 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
index 08d9963fe9259..55663e0f7f915 100644
--- a/Documentation/devicetree/bindings/arm/amlogic.yaml
+++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
@@ -247,6 +247,12 @@ properties:
               - amlogic,aq222
           - const: amlogic,s4
Look here
quoted
quoted
quoted
+      - description: Boards with the Amlogic Meson S4 S905Y4 SoC
+        items:
+          - enum:
Yopu already have exactly such enum with such description, so your
compatible goes there. Look - it is called "Boards" - plural.
The exist board aq222 is based on SoC S905X2, but the new board vim1s
is baed on S905Y4,
they are different SoC, so we need to put S905Y4 under S805X2 ?
What is the point of having two separate lists with enums ending the
same SoC compatible?
Actually S805X2 and S905Y4 are different SoC. So maybe like this ?

S805X2:
- description: Boards with the Amlogic Meson S4 S805X2 SoC
  items:
    - enum:
        - amlogic,aq222
    - const: amlogic,s805x2
    - const: amlogic,s4

S905Y4:
- description: Boards with the Amlogic Meson S4 S905Y4 SoC
  items:
    - enum:
        - khadas,vim1s
    - const: amlogic,s905y4
    - const: amlogic,s4

Same as Amlogic A311D & S922X

S922X:
- const: amlogic,s922x
- const: amlogic,g12b

A311D:
- const: amlogic,a311d
- const: amlogic,g12b
quoted
quoted
quoted
+              - khadas,vim1s
+          - const: amlogic,s4
And here.
quoted
quoted
Best regards,
Krzysztof

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