Re: [PATCH v7 1/5] dt-bindings: arm: fsl: Add GOcontroll Moduline IV/Mini

[Maud Spierings]

On 2/10/26 09:45, Maud Spierings via B4 Relay wrote:
> From: Maud Spierings <maudspierings@xxxxxxxxxxxxxx>
>
> Document the compatible strings for the Moduline IV and Mini.
> Also add a compatible for the Ka-Ro Electronics TX8M-1610 SoM that they
> are based on.
>
> Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> Signed-off-by: Maud Spierings <maudspierings@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 5716d701292c..c76c1731b4f7 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -996,6 +996,14 @@ properties:
>             - const: engicam,icore-mx8mm             # i.MX8MM Engicam i.Core MX8M Mini SoM
>             - const: fsl,imx8mm
>   
> +      - description: Ka-Ro Electronics TX8M-1610 based boards
> +        items:
> +          - enum:
> +              - gocontroll,moduline-iv-306-d
> +              - gocontroll,moduline-mini-111
> +          - const: karo,imx8mm-tx8m-1610

I just realised that I did karo,tx8p-ml81 for the imx8mp module, 
doubting if I should change this to karo,tx8m-1610.

This inconsistency is now bothering me.

Kind regards,
Maud

> +          - const: fsl,imx8mm
> +
>         - description: Kontron BL i.MX8MM (N801X S) Board
>           items:
>             - const: kontron,imx8mm-bl