[PATCH V1 resend 0/6] Document r8a774a3 SoC bindings

[PATCH V1 resend 0/6] Document r8a774a3 SoC bindings

[Oliver Rhodes]

Hi all,

This patch series updates the renesas DT bindings documentation for core
components (SoC, SYSC, reset, clk, pinctrl, scif) on the RZ/G2M v3.0 (R8A774A3) SoC.
This SoC is similar to R-CAR M3-W+ (R8A77961) SoC.

v1->v1 resend:
* Fixed commit header for patch #4.

Oliver Rhodes (6):
  dt-bindings: soc: renesas: Document RZ/G2M v3.0 (r8a774a3) SoC
  dt-bindings: power: renesas: Document RZ/G2M v3.0 (r8a774a3) SYSC
    binding
  dt-bindings: clock: renesas: Document RZ/G2M v3.0 (r8a774a3) clock
  dt-bindings: reset: renesas: Document RZ/G2M v3.0 (r8a774a3) reset
    module
  dt-bindings: serial: renesas: Document RZ/G2M v3.0 (r8a774a3) scif
  dt-bindings: pinctrl: renesas: Document RZ/G2M v3.0 (r8a774a3) PFC
    support

 .../devicetree/bindings/clock/renesas,cpg-mssr.yaml  |  1 +
 .../devicetree/bindings/pinctrl/renesas,pfc.yaml     |  1 +
 .../devicetree/bindings/power/renesas,rcar-sysc.yaml |  1 +
 .../devicetree/bindings/reset/renesas,rst.yaml       |  1 +
 .../devicetree/bindings/serial/renesas,scif.yaml     |  1 +
 .../devicetree/bindings/soc/renesas/renesas.yaml     | 12 ++++++++++++
 6 files changed, 17 insertions(+)

-- 
2.34.1

[PATCH V1 resend 1/6] dt-bindings: soc: renesas: Document RZ/G2M v3.0 (r8a774a3) SoC

[Oliver Rhodes]

Add SoC documentation for Renesas RZ/G2M v3.0 (a.k.a r8a774a3) SoC.

Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
---
v1->v1 resend: 
* No change.
---
 .../devicetree/bindings/soc/renesas/renesas.yaml     | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
index 09d3ce97efa2..d582992aaf0e 100644
--- a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
+++ b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
@@ -127,6 +127,18 @@ properties:
           - const: hoperun,hihope-rzg2m
           - const: renesas,r8a774a1
 
+      - description: RZ/G2M v3.0 (R8A774A3)
+        items:
+          - enum:
+              - hoperun,hihope-rzg2m # HopeRun HiHope RZ/G2M platform
+          - const: renesas,r8a774a3
+
+      - items:
+          - enum:
+              - hoperun,hihope-rzg2-ex # HopeRun expansion board for HiHope RZ/G2 platforms
+          - const: hoperun,hihope-rzg2m
+          - const: renesas,r8a774a3
+
       - description: RZ/G2N (R8A774B1)
         items:
           - enum:
-- 
2.34.1

[PATCH V1 resend 2/6] dt-bindings: power: renesas: Document RZ/G2M v3.0 (r8a774a3) SYSC binding

[Oliver Rhodes]

Document bindings for the Renesas RZ/G2M v3.0 (a.k.a r8a774a3) SYSC block.

Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
---
v1->v1 resend:
* No change.
---
 Documentation/devicetree/bindings/power/renesas,rcar-sysc.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/power/renesas,rcar-sysc.yaml b/Documentation/devicetree/bindings/power/renesas,rcar-sysc.yaml
index e76fb273490f..347571e2545a 100644
--- a/Documentation/devicetree/bindings/power/renesas,rcar-sysc.yaml
+++ b/Documentation/devicetree/bindings/power/renesas,rcar-sysc.yaml
@@ -25,6 +25,7 @@ properties:
       - renesas,r8a7745-sysc  # RZ/G1E
       - renesas,r8a77470-sysc # RZ/G1C
       - renesas,r8a774a1-sysc # RZ/G2M
+      - renesas,r8a774a3-sysc # RZ/G2M v3.0
       - renesas,r8a774b1-sysc # RZ/G2N
       - renesas,r8a774c0-sysc # RZ/G2E
       - renesas,r8a774e1-sysc # RZ/G2H
-- 
2.34.1

[PATCH V1 resend 3/6] dt-bindings: clock: renesas: Document RZ/G2M v3.0 (r8a774a3) clock

[Oliver Rhodes]

Add binding documentation for Renesas RZ/G2M v3.0 (a.k.a r8a774a3) Clock
Pulse Generator driver.

Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
---
v1->v1 resend:
* No change.
---
 Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml b/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
index 084259d30232..77ce3615c65a 100644
--- a/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
+++ b/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
@@ -31,6 +31,7 @@ properties:
       - renesas,r8a7745-cpg-mssr  # RZ/G1E
       - renesas,r8a77470-cpg-mssr # RZ/G1C
       - renesas,r8a774a1-cpg-mssr # RZ/G2M
+      - renesas,r8a774a3-cpg-mssr # RZ/G2M v3.0
       - renesas,r8a774b1-cpg-mssr # RZ/G2N
       - renesas,r8a774c0-cpg-mssr # RZ/G2E
       - renesas,r8a774e1-cpg-mssr # RZ/G2H
-- 
2.34.1

[PATCH V1 resend 4/6] dt-bindings: reset: renesas: Document RZ/G2M v3.0 (r8a774a3) reset module

[Oliver Rhodes]

Document bindings for the Renesas RZ/G2M v3.0 (a.k.a r8a774a3) reset
module.

Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
---
v1->v1 resend:
* Fixed the commit header.
---
 Documentation/devicetree/bindings/reset/renesas,rst.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/reset/renesas,rst.yaml b/Documentation/devicetree/bindings/reset/renesas,rst.yaml
index 58b4a45d3380..7a81491379b0 100644
--- a/Documentation/devicetree/bindings/reset/renesas,rst.yaml
+++ b/Documentation/devicetree/bindings/reset/renesas,rst.yaml
@@ -29,6 +29,7 @@ properties:
       - renesas,r8a7745-rst       # RZ/G1E
       - renesas,r8a77470-rst      # RZ/G1C
       - renesas,r8a774a1-rst      # RZ/G2M
+      - renesas,r8a774a3-rst      # RZ/G2M v3.0
       - renesas,r8a774b1-rst      # RZ/G2N
       - renesas,r8a774c0-rst      # RZ/G2E
       - renesas,r8a774e1-rst      # RZ/G2H
-- 
2.34.1

[PATCH V1 resend 5/6] dt-bindings: serial: renesas: Document RZ/G2M v3.0 (r8a774a3) scif

[Oliver Rhodes]

Document scif bindings for the Renesas RZ/G2M v3.0 (a.k.a r8a774a3) SoC.

Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
---
v1->v1 resend:
* No change.
---
 Documentation/devicetree/bindings/serial/renesas,scif.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/serial/renesas,scif.yaml b/Documentation/devicetree/bindings/serial/renesas,scif.yaml
index afc7c05898a1..51d9fb0f4763 100644
--- a/Documentation/devicetree/bindings/serial/renesas,scif.yaml
+++ b/Documentation/devicetree/bindings/serial/renesas,scif.yaml
@@ -46,6 +46,7 @@ properties:
       - items:
           - enum:
               - renesas,scif-r8a774a1     # RZ/G2M
+              - renesas,scif-r8a774a3     # RZ/G2M v3.0
               - renesas,scif-r8a774b1     # RZ/G2N
               - renesas,scif-r8a774c0     # RZ/G2E
               - renesas,scif-r8a774e1     # RZ/G2H
-- 
2.34.1

[PATCH V1 resend 6/6] dt-bindings: pinctrl: renesas: Document RZ/G2M v3.0 (r8a774a3) PFC support

[Oliver Rhodes]

Document PFC support for Renesas RZ/G2M v3.0 (a.k.a r8a774a3) SoC.

Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
---
v1->v1 resend:
* No change.
---
 Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
index 5d84364d1358..cfe004573366 100644
--- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
@@ -25,6 +25,7 @@ properties:
       - renesas,pfc-r8a7745     # RZ/G1E
       - renesas,pfc-r8a77470    # RZ/G1C
       - renesas,pfc-r8a774a1    # RZ/G2M
+      - renesas,pfc-r8a774a3    # RZ/G2M v3.0
       - renesas,pfc-r8a774b1    # RZ/G2N
       - renesas,pfc-r8a774c0    # RZ/G2E
       - renesas,pfc-r8a774e1    # RZ/G2H
-- 
2.34.1

Re: [PATCH V1 resend 3/6] dt-bindings: clock: renesas: Document RZ/G2M v3.0 (r8a774a3) clock

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 1193 bytes --]

On Tue, Jul 23, 2024 at 12:07:30PM +0100, Oliver Rhodes wrote:
> Add binding documentation for Renesas RZ/G2M v3.0 (a.k.a r8a774a3) Clock
> Pulse Generator driver.
> 
> Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
> ---
> v1->v1 resend:
> * No change.
> ---
>  Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml b/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> index 084259d30232..77ce3615c65a 100644
> --- a/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> +++ b/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> @@ -31,6 +31,7 @@ properties:
>        - renesas,r8a7745-cpg-mssr  # RZ/G1E
>        - renesas,r8a77470-cpg-mssr # RZ/G1C
>        - renesas,r8a774a1-cpg-mssr # RZ/G2M
> +      - renesas,r8a774a3-cpg-mssr # RZ/G2M v3.0

Please explain in your commit message why a fallback is not suitable
here.

>        - renesas,r8a774b1-cpg-mssr # RZ/G2N
>        - renesas,r8a774c0-cpg-mssr # RZ/G2E
>        - renesas,r8a774e1-cpg-mssr # RZ/G2H
> -- 
> 2.34.1
> 
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH V1 resend 0/6] Document r8a774a3 SoC bindings

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 438 bytes --]

On Tue, Jul 23, 2024 at 12:07:27PM +0100, Oliver Rhodes wrote:
> Hi all,
> 
> This patch series updates the renesas DT bindings documentation for core
> components (SoC, SYSC, reset, clk, pinctrl, scif) on the RZ/G2M v3.0 (R8A774A3) SoC.
> This SoC is similar to R-CAR M3-W+ (R8A77961) SoC.

Your commit messages for any binding without a fallback compatible need
to mention why this device is not compatible with the !v3.0 SoC.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

RE: [PATCH V1 resend 3/6] dt-bindings: clock: renesas: Document RZ/G2M v3.0 (r8a774a3) clock

[Oliver Rhodes]

Hi Conor,

Thank you for the feedback.

> -----Original Message-----
> From: Conor Dooley <conor@kernel.org>
> Sent: Tuesday, July 23, 2024 3:04 PM
> Subject: Re: [PATCH V1 resend 3/6] dt-bindings: clock: renesas: Document RZ/G2M v3.0 (r8a774a3) clock
>
> On Tue, Jul 23, 2024 at 12:07:30PM +0100, Oliver Rhodes wrote:
> > Add binding documentation for Renesas RZ/G2M v3.0 (a.k.a r8a774a3)
> > Clock Pulse Generator driver.
> >
> > Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
> > ---
> > v1->v1 resend:
> > * No change.
> > ---
> >  Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git
> > a/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> > b/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> > index 084259d30232..77ce3615c65a 100644
> > --- a/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> > +++ b/Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> > @@ -31,6 +31,7 @@ properties:
> >        - renesas,r8a7745-cpg-mssr  # RZ/G1E
> >        - renesas,r8a77470-cpg-mssr # RZ/G1C
> >        - renesas,r8a774a1-cpg-mssr # RZ/G2M
> > +      - renesas,r8a774a3-cpg-mssr # RZ/G2M v3.0
>
> Please explain in your commit message why a fallback is not suitable here.

The r8a774a1-cpg-mssr is almost identical to r8a774a3-cpg-mssr, but it lacks
clocks like FCPCI. To handle this difference, we need to use a separate
compatible. I will update the commit message in the next version.

Thanks,
Oliver
________________________________

Renesas Electronics Europe GmbH
Registered Office: Arcadiastrasse 10
DE-40472 Duesseldorf
Commercial Registry: Duesseldorf, HRB 3708
Managing Director: Carsten Jauch
VAT-No.: DE 14978647
Tax-ID-No: 105/5839/1793

Legal Disclaimer: This e-mail communication (and any attachment/s) is confidential and contains proprietary information, some or all of which may be legally privileged. It is intended solely for the use of the individual or entity to which it is addressed. Access to this email by anyone else is unauthorized. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited and may be unlawful.

RE: [PATCH V1 resend 0/6] Document r8a774a3 SoC bindings

[Oliver Rhodes]

Hi Conor,

Thank you for the comment.

> -----Original Message-----
> From: Conor Dooley <conor@kernel.org>
> Sent: Tuesday, July 23, 2024 3:05 PM
> To: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
> Cc: Rob Herring <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>; Conor Dooley
> <conor+dt@kernel.org>; Geert Uytterhoeven <geert+renesas@glider.be>; Magnus Damm
> <magnus.damm@gmail.com>; linux-renesas-soc@vger.kernel.org; devicetree@vger.kernel.org
> Subject: Re: [PATCH V1 resend 0/6] Document r8a774a3 SoC bindings
>
> On Tue, Jul 23, 2024 at 12:07:27PM +0100, Oliver Rhodes wrote:
> > Hi all,
> >
> > This patch series updates the renesas DT bindings documentation for
> > core components (SoC, SYSC, reset, clk, pinctrl, scif) on the RZ/G2M v3.0 (R8A774A3) SoC.
> > This SoC is similar to R-CAR M3-W+ (R8A77961) SoC.
>
> Your commit messages for any binding without a fallback compatible need to mention why this device is
> not compatible with the !v3.0 SoC.

For SoC identification, it is a different SoC altogether, so we need to use a dedicated compatible.
For SYSC, it lacks the A2VC power area compared to the RZ/G2M v1.X so we cannot use family compatible.
Similarly for clock, it lacks some modules such as the FCPCI, so we need to use a dedicated compatible.

R-CAR M3-W (r8a77960) is similar to RZ/G2M (r8a774a1) and R-CAR M3-W+ (r8a77961) is similar to RZ/G2M v3.0(r8a774a3).
So, I referred R-CAR M3-W+ bindings and made similar changes for this SoC.

I will update the commit messages for SoC, SYSC and clk to reflect this.

Thank you,
Oliver
________________________________

Renesas Electronics Europe GmbH
Registered Office: Arcadiastrasse 10
DE-40472 Duesseldorf
Commercial Registry: Duesseldorf, HRB 3708
Managing Director: Carsten Jauch
VAT-No.: DE 14978647
Tax-ID-No: 105/5839/1793

Legal Disclaimer: This e-mail communication (and any attachment/s) is confidential and contains proprietary information, some or all of which may be legally privileged. It is intended solely for the use of the individual or entity to which it is addressed. Access to this email by anyone else is unauthorized. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited and may be unlawful.

Re: [PATCH V1 resend 1/6] dt-bindings: soc: renesas: Document RZ/G2M v3.0 (r8a774a3) SoC

[Rob Herring (Arm)]

On Tue, 23 Jul 2024 12:07:28 +0100, Oliver Rhodes wrote:
> Add SoC documentation for Renesas RZ/G2M v3.0 (a.k.a r8a774a3) SoC.
> 
> Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
> ---
> v1->v1 resend:
> * No change.
> ---
>  .../devicetree/bindings/soc/renesas/renesas.yaml     | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 

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

Re: [PATCH V1 resend 2/6] dt-bindings: power: renesas: Document RZ/G2M v3.0 (r8a774a3) SYSC binding

[Rob Herring (Arm)]

On Tue, 23 Jul 2024 12:07:29 +0100, Oliver Rhodes wrote:
> Document bindings for the Renesas RZ/G2M v3.0 (a.k.a r8a774a3) SYSC block.
> 
> Signed-off-by: Oliver Rhodes <oliver.rhodes.aj@renesas.com>
> ---
> v1->v1 resend:
> * No change.
> ---
>  Documentation/devicetree/bindings/power/renesas,rcar-sysc.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

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