diff mbox series

[06/15] dt-bindings: soc: renesas: Document R-Car V4M Gray Hawk Single

Message ID 3b1baf0eaf9f483308a6df1340dae51d4b88a337.1704726960.git.geert+renesas@glider.be
State Not Applicable, archived
Headers show
Series arm64: renesas: Add R-Car V4M and Gray Hawk Single support | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Geert Uytterhoeven Jan. 8, 2024, 3:33 p.m. UTC
Document the compatible values for the Renesas R-Car V4M (R8A779H0) SoC
and the Renesas Gray Hawk Single development board.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
The Gray Hawk Single board is an integrated variant of the (not yet
supported) Gray Hawk board stack, which is very similar to the White
Hawk board stack.

The schematics call it '"Gray Hawk"(1Board)'.
The Setup Manual calls it 'Gray Hawk Single Board'.

Alternative naming candidates would be 'Gray Hawk S' and/or
'renesas,gray-hawk-s'.
---
 Documentation/devicetree/bindings/soc/renesas/renesas.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Conor Dooley Jan. 8, 2024, 5:54 p.m. UTC | #1
On Mon, Jan 08, 2024 at 04:33:45PM +0100, Geert Uytterhoeven wrote:
> Document the compatible values for the Renesas R-Car V4M (R8A779H0) SoC
> and the Renesas Gray Hawk Single development board.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.

> ---
> The Gray Hawk Single board is an integrated variant of the (not yet
> supported) Gray Hawk board stack, which is very similar to the White
> Hawk board stack.
> 
> The schematics call it '"Gray Hawk"(1Board)'.
> The Setup Manual calls it 'Gray Hawk Single Board'.
> 
> Alternative naming candidates would be 'Gray Hawk S' and/or
> 'renesas,gray-hawk-s'.
> ---
>  Documentation/devicetree/bindings/soc/renesas/renesas.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> index 16ca3ff7b1aea146..1c8f5122bccdb5c8 100644
> --- a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> +++ b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> @@ -354,6 +354,12 @@ properties:
>            - const: renesas,white-hawk-cpu
>            - const: renesas,r8a779g0
>  
> +      - description: R-Car V4M (R8A779H0)
> +        items:
> +          - enum:
> +              - renesas,gray-hawk-single # Gray Hawk Single (RTP8A779H0ASKB0F10S)
> +          - const: renesas,r8a779h0
> +
>        - description: R-Car H3e (R8A779M0)
>          items:
>            - enum:
> -- 
> 2.34.1
>
Niklas Söderlund Jan. 10, 2024, 12:10 p.m. UTC | #2
Hi Geert,

Thanks for your work.

On 2024-01-08 16:33:45 +0100, Geert Uytterhoeven wrote:
> Document the compatible values for the Renesas R-Car V4M (R8A779H0) SoC
> and the Renesas Gray Hawk Single development board.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> The Gray Hawk Single board is an integrated variant of the (not yet
> supported) Gray Hawk board stack, which is very similar to the White
> Hawk board stack.
> 
> The schematics call it '"Gray Hawk"(1Board)'.
> The Setup Manual calls it 'Gray Hawk Single Board'.
> 
> Alternative naming candidates would be 'Gray Hawk S' and/or
> 'renesas,gray-hawk-s'.
> ---
>  Documentation/devicetree/bindings/soc/renesas/renesas.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> index 16ca3ff7b1aea146..1c8f5122bccdb5c8 100644
> --- a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> +++ b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> @@ -354,6 +354,12 @@ properties:
>            - const: renesas,white-hawk-cpu
>            - const: renesas,r8a779g0
>  
> +      - description: R-Car V4M (R8A779H0)
> +        items:
> +          - enum:
> +              - renesas,gray-hawk-single # Gray Hawk Single (RTP8A779H0ASKB0F10S)

nit: Should that not be 'Single Board' or '1Board'? I see both in the 
documentation but not just "Single".

> +          - const: renesas,r8a779h0
> +
>        - description: R-Car H3e (R8A779M0)
>          items:
>            - enum:
> -- 
> 2.34.1
> 
>
Wolfram Sang Jan. 24, 2024, 10:38 a.m. UTC | #3
On Mon, Jan 08, 2024 at 04:33:45PM +0100, Geert Uytterhoeven wrote:
> Document the compatible values for the Renesas R-Car V4M (R8A779H0) SoC
> and the Renesas Gray Hawk Single development board.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> The Gray Hawk Single board is an integrated variant of the (not yet
> supported) Gray Hawk board stack, which is very similar to the White
> Hawk board stack.
> 
> The schematics call it '"Gray Hawk"(1Board)'.
> The Setup Manual calls it 'Gray Hawk Single Board'.
> 
> Alternative naming candidates would be 'Gray Hawk S' and/or
> 'renesas,gray-hawk-s'.

I like this current version with the explicit and easy to understand
naming.

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
index 16ca3ff7b1aea146..1c8f5122bccdb5c8 100644
--- a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
+++ b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
@@ -354,6 +354,12 @@  properties:
           - const: renesas,white-hawk-cpu
           - const: renesas,r8a779g0
 
+      - description: R-Car V4M (R8A779H0)
+        items:
+          - enum:
+              - renesas,gray-hawk-single # Gray Hawk Single (RTP8A779H0ASKB0F10S)
+          - const: renesas,r8a779h0
+
       - description: R-Car H3e (R8A779M0)
         items:
           - enum: