diff mbox series

[1/2] dt-bindings: arm: rockchip: add Firefly CORE PX30 JD4

Message ID 20240716-rockchip-px30-firefly-v1-1-60cdad3023a3@collabora.com (mailing list archive)
State New, archived
Headers show
Series Add support for Firefly CORE PX30 JD4 | expand

Commit Message

Christopher Obbard July 16, 2024, 3:51 p.m. UTC
The Firefly CORE PX30 JD4 board is a SOM and motherboard bundle from
Firefly. Add devicetree binding documentation for it.

Signed-off-by: Christopher Obbard <chris.obbard@collabora.com>
---
 Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Conor Dooley July 16, 2024, 4:26 p.m. UTC | #1
On Tue, Jul 16, 2024 at 04:51:04PM +0100, Christopher Obbard wrote:
> The Firefly CORE PX30 JD4 board is a SOM and motherboard bundle from
> Firefly. Add devicetree binding documentation for it.
> 
> Signed-off-by: Christopher Obbard <chris.obbard@collabora.com>
> ---
>  Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
> index e04c213a0dee4..19e06e1253e15 100644
> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
> @@ -148,6 +148,11 @@ properties:
>            - const: engicam,px30-core
>            - const: rockchip,px30
>  
> +      - description: Firefly Core PX30 JD4
> +        items:
> +          - const: firefly,core-px30-jd4
> +          - const: rockchip,px30

Not having individual compatibles for the carrier and som seems odd to,
given there's no requirement to use the som with this particular
carrier.

> +
>        - description: Firefly Firefly-RK3288
>          items:
>            - enum:
> 
> -- 
> 2.45.2
>
Christopher Obbard July 16, 2024, 4:36 p.m. UTC | #2
On Tue, 2024-07-16 at 17:26 +0100, Conor Dooley wrote:
> On Tue, Jul 16, 2024 at 04:51:04PM +0100, Christopher Obbard wrote:
> > The Firefly CORE PX30 JD4 board is a SOM and motherboard bundle from
> > Firefly. Add devicetree binding documentation for it.
> > 
> > Signed-off-by: Christopher Obbard <chris.obbard@collabora.com>
> > ---
> >  Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml
> > b/Documentation/devicetree/bindings/arm/rockchip.yaml
> > index e04c213a0dee4..19e06e1253e15 100644
> > --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
> > +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
> > @@ -148,6 +148,11 @@ properties:
> >            - const: engicam,px30-core
> >            - const: rockchip,px30
> >  
> > +      - description: Firefly Core PX30 JD4
> > +        items:
> > +          - const: firefly,core-px30-jd4
> > +          - const: rockchip,px30
> 
> Not having individual compatibles for the carrier and som seems odd to,
> given there's no requirement to use the som with this particular
> carrier.

Thanks, I will fix that up in V2.

> 
> > +
> >        - description: Firefly Firefly-RK3288
> >          items:
> >            - enum:
> > 
> > -- 
> > 2.45.2
> >
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
index e04c213a0dee4..19e06e1253e15 100644
--- a/Documentation/devicetree/bindings/arm/rockchip.yaml
+++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
@@ -148,6 +148,11 @@  properties:
           - const: engicam,px30-core
           - const: rockchip,px30
 
+      - description: Firefly Core PX30 JD4
+        items:
+          - const: firefly,core-px30-jd4
+          - const: rockchip,px30
+
       - description: Firefly Firefly-RK3288
         items:
           - enum: