diff mbox series

[1/2] dt-bindings: arm: sunxi: Add Sipeed Longan Module 3H and Longan Pi 3H

Message ID 20231228145647.1470-2-jszhang@kernel.org
State Changes Requested
Headers show
Series arm64: dts: allwinner: h618: Add Sipeed Longan SoM 3H and Pi 3H board | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Jisheng Zhang Dec. 28, 2023, 2:56 p.m. UTC
Add name & compatible for the Sipeed Longan Module 3H and Longan PI 3H
board.

Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
---
 Documentation/devicetree/bindings/arm/sunxi.yaml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Andre Przywara Dec. 29, 2023, 11:56 p.m. UTC | #1
On Thu, 28 Dec 2023 22:56:46 +0800
Jisheng Zhang <jszhang@kernel.org> wrote:

Hi Jisheng,

thanks for sending this, seems to be on time with the release of the
hardware. Don't see that very often!

> Add name & compatible for the Sipeed Longan Module 3H and Longan PI 3H
> board.
> 
> Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/sunxi.yaml | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
> index 11c5ce941dd7..34d8ae484e01 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi.yaml
> +++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
> @@ -835,6 +835,13 @@ properties:
>            - const: sinlinx,sina33
>            - const: allwinner,sun8i-a33
>  
> +      - description: Sipeed Longan Pi 3H board for the Sipeed Longan Module 3H
> +        items:
> +          - enum:
> +              - sipeed,longan-pi-3h

The picture of the SoM suggests that it cannot be used on its own, so
for anything usable we will always need both the SoM and some board.
Definitely we do for this board. So please follow the example for the
SoPine (or other SoM based boards), and just name all three names as
"const" items.

Cheers,
Andre

> +          - const: sipeed,longan-module-3h
> +          - const: allwinner,sun50i-h618
> +
>        - description: SourceParts PopStick v1.1
>          items:
>            - const: sourceparts,popstick-v1.1
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
index 11c5ce941dd7..34d8ae484e01 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
@@ -835,6 +835,13 @@  properties:
           - const: sinlinx,sina33
           - const: allwinner,sun8i-a33
 
+      - description: Sipeed Longan Pi 3H board for the Sipeed Longan Module 3H
+        items:
+          - enum:
+              - sipeed,longan-pi-3h
+          - const: sipeed,longan-module-3h
+          - const: allwinner,sun50i-h618
+
       - description: SourceParts PopStick v1.1
         items:
           - const: sourceparts,popstick-v1.1