[PATCH v2 2/3] dt-bindings: arm: fsl: Enumerate Debix Model A Board

[Dan Scally]

Hi Kieran

On 10/10/2022 14:56, Kieran Bingham wrote:
> Quoting Daniel Scally (2022-10-10 14:31:45)
>> Add an entry to the list of imx8mp boards denoting the Debix Model A
>> board from Polyhex Technology Co.
>>
>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
>> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>> Signed-off-by: Daniel Scally <dan.scally at ideasonboard.com>
>> ---
>> Changes in v2:
>>
>>      - Dropped 2gb suffix, added hyphens for readability (Laurent)
>>
>>   Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
>> index 7431579ab0e8..fc0c8eb15114 100644
>> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
>> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
>> @@ -931,6 +931,7 @@ properties:
>>                 - dh,imx8mp-dhcom-pdk2      # i.MX8MP DHCOM SoM on PDK2 board
>>                 - fsl,imx8mp-evk            # i.MX8MP EVK Board
>>                 - gateworks,imx8mp-gw74xx   # i.MX8MP Gateworks Board
>> +              - polyhex,imx8mp-debix-model-a # Polyhex Debix Model A (2GB) Board
> If we've dropped the 2gb suffix from the compatible string, does that
> mean we should drop it from the description too? I presume the
> implication is that we can support different sizes from the same compat
> string.
Argh! Yes, my bad - I'll fix that.
>
>>                 - toradex,verdin-imx8mp     # Verdin iMX8M Plus Modules
>>                 - toradex,verdin-imx8mp-nonwifi  # Verdin iMX8M Plus Modules without Wi-Fi / BT
>>                 - toradex,verdin-imx8mp-wifi  # Verdin iMX8M Plus Wi-Fi / BT Modules
>> -- 
>> 2.34.1
>>