Re: [PATCH v5 2/4] dt-bindings: arm64: marvell: add solidrun cn9132 CEX-7 evaluation board

[Conor Dooley]

On Thu, May 09, 2024 at 10:49:13AM +0000, Josua Mayer wrote:
> Am 09.05.24 um 12:46 schrieb Josua Mayer:
> > Add bindings for the SolidRun CN9132 COM-Express Type 7 evaluation board.
> > The CEX is based on CN9130 SoC and includes two southbridges.
> >
> > Because CN9132 and 9131 are just names for different designs around the
> > same SoC, no soc compatibles beside marvell,cn9130 are needed.
> >
> > Signed-off-by: Josua Mayer <josua@xxxxxxxxxxxxx>
> > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> > Reviewed-by: Andrew Lunn <andrew@xxxxxxx>
> > ---
> >  Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> > index 74d935ea279c..538d91be8857 100644
> > --- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml
> > @@ -92,4 +92,12 @@ properties:
> >            - const: solidrun,cn9130-sr-som
> >            - const: marvell,cn9130
> >  
> > +      - description:
> > +          SolidRun CN9132 COM-Express Type 7 based single-board computers
> > +        items:
> > +          - enum:
> > +              - solidrun,cn9132-clearfog
> > +          - const: solidrun,cn9132-sr-cex7
> > +          - const: marvell,cn9130
> > +
> >  additionalProperties: true
> >
> It appears I will not be able to submit actual device-tree for this
> board. Therefore when applying this patch-set, it may be skipped.
> 
> I am not sure about the policy in this case,
> if it is better to pick or skip.

What do you mean my "not be able to"? Does the device exist? If it does
then, I at least, have no objection to documenting a compatible for it.

Cheers,
Conor.

Attachment: signature.asc
Description: PGP signature