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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <fb1e21a4a27948d08f960b24dc5dbe54@dh-electronics.com>
Date:   Mon, 7 Jun 2021 09:29:36 +0000
From:   Christoph Niedermaier <cniedermaier@...electronics.com>
To:     'Rob Herring' <robh@...nel.org>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Shawn Guo <shawnguo@...nel.org>,
        kernel <kernel@...electronics.com>,
        'Fabio Estevam' <festevam@...il.com>
Subject: RE: [PATCH V2] dt-bindings: arm: fsl: Add DHCOM PicoITX and DHCOM
 DRC02 boards [Klartext]

From: Rob Herring <robh@...nel.org>
Sent: Wednesday, June 2, 2021 9:50 PM

> On Tue, May 25, 2021 at 04:30:01PM +0200, Christoph Niedermaier wrote:
>> Add DH electronics DHCOM PicoITX and DHCOM DRC02 boards.
>>
>> Signed-off-by: Christoph Niedermaier <cniedermaier@...electronics.com>
>> Cc: linux-arm-kernel@...ts.infradead.org
>> Cc: linux-kernel@...r.kernel.org
>> Cc: robh+dt@...nel.org
>> Cc: Shawn Guo <shawnguo@...nel.org>
>> Cc: kernel@...electronics.com
>> To: devicetree@...r.kernel.org
>> ---
>> V2: Remove line with fsl,imx6s on the DRC02 Board
>> ---
>>  Documentation/devicetree/bindings/arm/fsl.yaml | 12 ++++++++++++
>>  1 file changed, 12 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
>> index fce2a8670b49..3c4ff79a3be7 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,12 @@ 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,imx6dl
> 
> fsl,imx6s?

In the first version I had here an additional line with "fsl,imx6s",
but currently the kernel isn't supporting that compatible. The i.MX6
Solo is currently supported by "fsl,imx6dl". So my idea was to add
both "fsl,imx6dl" and "fsl,imx6s" to match it maybe on a later kernel
version. If there is no match with the Solo now, it will fall back to
the i.MX6 DualLite. That is why I had both fsl,imx6s and fsl,imx6dl
in that order. On Fabio's advice, I removed the line with "fsl,imx6s"
in version 2.
Is this what you meant by your comment?

> 
>> +
>>        - description: i.MX6SL based Boards
>>          items:
>>            - enum:
>> --
>> 2.11.0

Regards
Christoph