diff mbox series

[1/7] dt-bindings: arm: rockchip: Add meaningful comments for Radxa CM3

Message ID 20230117105504.18296-1-jagan@amarulasolutions.com (mailing list archive)
State New, archived
Headers show
Series [1/7] dt-bindings: arm: rockchip: Add meaningful comments for Radxa CM3 | expand

Commit Message

Jagan Teki Jan. 17, 2023, 10:54 a.m. UTC
Radxa produced several versions of the RK356x-based Compute modules
called Radxa Compute Module 3(CM3) for consumer and industrial use
with a variety of form factors.

So, adding meaningful comments on dt-bindings would help to understand
more about specific module relevance.

This patch is trying to add comments on existing CM3 bindings and
expecting the subsequent modules are following similar comments while
adding dt-bindings to it.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 Documentation/devicetree/bindings/arm/rockchip.yaml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Krzysztof Kozlowski Jan. 17, 2023, 11:05 a.m. UTC | #1
On 17/01/2023 11:54, Jagan Teki wrote:
> Radxa produced several versions of the RK356x-based Compute modules
> called Radxa Compute Module 3(CM3) for consumer and industrial use
> with a variety of form factors.
> 
> So, adding meaningful comments on dt-bindings would help to understand
> more about specific module relevance.
> 
> This patch is trying to add comments on existing CM3 bindings and
> expecting the subsequent modules are following similar comments while
> adding dt-bindings to it.


Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
index 98afd7c6b8e6..3d3c59624483 100644
--- a/Documentation/devicetree/bindings/arm/rockchip.yaml
+++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
@@ -621,11 +621,11 @@  properties:
           - const: pine64,soquartz
           - const: rockchip,rk3566
 
-      - description: Radxa Compute Module 3(CM3)
+      - description: Radxa Compute Module 3(CM3) SoM based boards
         items:
           - enum:
-              - radxa,radxa-cm3-io
-          - const: radxa,radxa-cm3
+              - radxa,radxa-cm3-io   # Radxa Compute Module 3(CM3) IO Board
+          - const: radxa,radxa-cm3   # Radxa Compute Module 3(CM3)
           - const: rockchip,rk3566
 
       - description: Radxa CM3 Industrial