diff mbox series

[2/2] dt-bindings: arm: fsl: drop usage of VAR-SOM-MX8MM SoM compatible alone

Message ID 20240902075220.27971-2-krzysztof.kozlowski@linaro.org (mailing list archive)
State New
Headers show
Series [1/2] arm64: dts: imx8mm-var-som: drop unused top-level compatible | expand

Commit Message

Krzysztof Kozlowski Sept. 2, 2024, 7:52 a.m. UTC
The Variscite VAR-SOM-MX8MM System-on-Module cannot be used alone
without motherboard, so drop the usage of its compatible alone.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/arm/fsl.yaml | 1 -
 1 file changed, 1 deletion(-)

Comments

Peng Fan Sept. 2, 2024, 10:15 a.m. UTC | #1
> Subject: [PATCH 2/2] dt-bindings: arm: fsl: drop usage of VAR-SOM-
> MX8MM SoM compatible alone
> 
> The Variscite VAR-SOM-MX8MM System-on-Module cannot be used
> alone without motherboard, so drop the usage of its compatible alone.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml
> b/Documentation/devicetree/bindings/arm/fsl.yaml
> index 1abdeb804468..e5f982fa750b 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -953,7 +953,6 @@ properties:
>                - toradex,verdin-imx8mm     # Verdin iMX8M Mini Modules
>                - toradex,verdin-imx8mm-nonwifi  # Verdin iMX8M Mini
> Modules without Wi-Fi / BT
>                - toradex,verdin-imx8mm-wifi  # Verdin iMX8M Mini Wi-Fi /
> BT Modules
> -              - variscite,var-som-mx8mm   # i.MX8MM Variscite VAR-SOM-
> MX8MM module
>                - prt,prt8mm                # i.MX8MM Protonic PRT8MM Board
>            - const: fsl,imx8mm
> 

Line 1072
"const: variscite,var-som-mx8mm" should this one also need removed?

Regards,
Peng.

> --
> 2.43.0
>
Krzysztof Kozlowski Sept. 2, 2024, 10:20 a.m. UTC | #2
On 02/09/2024 12:15, Peng Fan wrote:
>> Subject: [PATCH 2/2] dt-bindings: arm: fsl: drop usage of VAR-SOM-
>> MX8MM SoM compatible alone
>>
>> The Variscite VAR-SOM-MX8MM System-on-Module cannot be used
>> alone without motherboard, so drop the usage of its compatible alone.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml
>> b/Documentation/devicetree/bindings/arm/fsl.yaml
>> index 1abdeb804468..e5f982fa750b 100644
>> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
>> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
>> @@ -953,7 +953,6 @@ properties:
>>                - toradex,verdin-imx8mm     # Verdin iMX8M Mini Modules
>>                - toradex,verdin-imx8mm-nonwifi  # Verdin iMX8M Mini
>> Modules without Wi-Fi / BT
>>                - toradex,verdin-imx8mm-wifi  # Verdin iMX8M Mini Wi-Fi /
>> BT Modules
>> -              - variscite,var-som-mx8mm   # i.MX8MM Variscite VAR-SOM-
>> MX8MM module
>>                - prt,prt8mm                # i.MX8MM Protonic PRT8MM Board
>>            - const: fsl,imx8mm
>>
> 
> Line 1072
> "const: variscite,var-som-mx8mm" should this one also need removed?

Of course not! How is this related to patch description?

Best regards,
Krzysztof
Peng Fan Sept. 2, 2024, 12:26 p.m. UTC | #3
> Subject: [PATCH 2/2] dt-bindings: arm: fsl: drop usage of VAR-SOM-
> MX8MM SoM compatible alone
> 
> The Variscite VAR-SOM-MX8MM System-on-Module cannot be used
> alone without motherboard, so drop the usage of its compatible alone.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---

I got a mistake in my previous comment, so

Reviewed-by: Peng Fan <peng.fan@nxp.com>
Rob Herring Sept. 3, 2024, 3:14 p.m. UTC | #4
On Mon, 02 Sep 2024 09:52:20 +0200, Krzysztof Kozlowski wrote:
> The Variscite VAR-SOM-MX8MM System-on-Module cannot be used alone
> without motherboard, so drop the usage of its compatible alone.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index 1abdeb804468..e5f982fa750b 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -953,7 +953,6 @@  properties:
               - toradex,verdin-imx8mm     # Verdin iMX8M Mini Modules
               - toradex,verdin-imx8mm-nonwifi  # Verdin iMX8M Mini Modules without Wi-Fi / BT
               - toradex,verdin-imx8mm-wifi  # Verdin iMX8M Mini Wi-Fi / BT Modules
-              - variscite,var-som-mx8mm   # i.MX8MM Variscite VAR-SOM-MX8MM module
               - prt,prt8mm                # i.MX8MM Protonic PRT8MM Board
           - const: fsl,imx8mm