RE: [PATCH] dt-bindings: arm: fsl: Add DHCOM PicoITX and DHCOM DRC02 boards [Klartext]

RE: [PATCH] dt-bindings: arm: fsl: Add DHCOM PicoITX and DHCOM DRC02 boards [Klartext]

[Christoph Niedermaier]

From: Fabio Estevam <festevam@gmail.com>
Sent: Monday, May 24, 2021 3:03 PM
> 
> Hi Christoph,
>

Hi Fabio,

> 
> On Mon, May 24, 2021 at 9:17 AM Christoph Niedermaier
> <cniedermaier@dh-electronics.com> wrote:
> 
>> +      - description: i.MX6S DHCOM DRC02 Board
>> +        items:
>> +          - const: dh,imx6s-dhcom-drc02
>> +          - const: dh,imx6s-dhcom-som
>> +          - const: fsl,imx6s
> 
> We don't have an fsl,imx6s compatible for the i.MX6 Solo variant. We
> just use fsl,imx6dl instead.

My thought was to be future proof. If there is no match with the Solo now,
it will fall back to the i.MX6 DualLite. That is why I added both fsl,imx6s
and fsl,imx6dl in this order.

Should I remove the line with fsl,imx6s?

Regards
Christoph

Re: [PATCH] dt-bindings: arm: fsl: Add DHCOM PicoITX and DHCOM DRC02 boards [Klartext]

[Fabio Estevam]

Hi Christoph,

On Mon, May 24, 2021 at 7:39 PM Christoph Niedermaier
<cniedermaier@dh-electronics.com> wrote:

> My thought was to be future proof. If there is no match with the Solo now,
> it will fall back to the i.MX6 DualLite. That is why I added both fsl,imx6s
> and fsl,imx6dl in this order.
>
> Should I remove the line with fsl,imx6s?

Yes, please.

Thanks

RE: [PATCH] dt-bindings: arm: fsl: Add DHCOM PicoITX and DHCOM DRC02 boards [Klartext]

[Christoph Niedermaier]

From: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Sent: Monday, May 24, 2021 2:04 PM
> 
> Add DH electronics DHCOM PicoITX and DHCOM DRC02 boards.
> 
> Signed-off-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-kernel@vger.kernel.org
> Cc: robh+dt@kernel.org
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: kernel@dh-electronics.com
> To: devicetree@vger.kernel.org
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml
> b/Documentation/devicetree/bindings/arm/fsl.yaml
> index fce2a8670b49..afbdea078abb 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -407,6 +407,12 @@ properties:
>            - const: dfi,fs700e-m60
>            - const: fsl,imx6dl
> 
> +      - description: i.MX6DL DHCOM PicoITX Board
> +        items:
> +          - const: dh,imx6dl-dhcom-picoitx
> +          - const: dh,imx6dl-dhcom-som
> +          - const: fsl,imx6dl
> +
>        - description: i.MX6DL Gateworks Ventana Boards
>          items:
>            - enum:
> @@ -458,6 +464,13 @@ properties:
>            - const: toradex,colibri_imx6dl          # Colibri iMX6 Module
>            - const: fsl,imx6dl
> 
> +      - description: i.MX6S DHCOM DRC02 Board
> +        items:
> +          - const: dh,imx6s-dhcom-drc02
> +          - const: dh,imx6s-dhcom-som
> +          - const: fsl,imx6s
> +          - const: fsl,imx6dl
> +
>        - description: i.MX6SL based Boards
>          items:
>            - enum:
> --
> 2.11.0

Ignore this one, because the recipient is wrong!

Regards
Christoph