Re: [PATCH 2/3] dt-bindings: arm: Add Polyhex DEBIX SOM A based boards

[Krzysztof Kozlowski]

On 04/07/2023 20:41, Marco Felsch wrote:
> Add devicetree bindings for i.MX8MP based DEBIX SOM A and SOM A I/O
> baseboard:
>   - https://debix.io/hardware/debix-som-a.html
>   - https://debix.io/hardware/debix-som-a-io-board.html
> 
> Signed-off-by: Marco Felsch <m.felsch@xxxxxxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index d9e763ef932e5..6240485ba13b2 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -1059,6 +1059,14 @@ properties:
>            - const: polyhex,imx8mp-debix             # Polyhex Debix boards
>            - const: fsl,imx8mp
>  
> +      - description: Polyhex DEBIX i.MX8MP SOM A based boards
> +        items:
> +          - enum:
> +              - polyhex,imx8mp-debix-som-a-bmb-08   # Polyhex Debix SOM A on SOM A I/O board
> +          - const: polyhex,imx8mp-debix-som-a       # Polyhex Debix SOM A
> +          - const: polyhex,imx8mp-debix             # Polyhex Debix boards

A bit too generic, unless Debix boards are basically very similar. Since
your website links to "debix.io" this looks like compatible for all
vendor boards, thus way too generic.

Best regards,
Krzysztof