[PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support

Prabhakar posted 1 patch 4 weeks ago
There is a newer version of this series
.../devicetree/bindings/i3c/renesas,i3c.yaml     | 16 +++++++++++-----
1 file changed, 11 insertions(+), 5 deletions(-)
[PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Prabhakar 4 weeks ago
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Add device tree binding support for the I3C Bus Interface on Renesas
RZ/V2H(P) and RZ/V2N SoCs. The I3C IP on these SoCs is identical to
that found on the RZ/G3E SoC.

Add new compatible strings "renesas,r9a09g056-i3c" for RZ/V2N and
"renesas,r9a09g057-i3c" for RZ/V2H(P). Both variants use
"renesas,r9a09g047-i3c" as a fallback compatible to indicate hardware
compatibility with the RZ/G3E implementation.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 .../devicetree/bindings/i3c/renesas,i3c.yaml     | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml b/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml
index fe2e9633c46f..e41ba3ba4b58 100644
--- a/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml
+++ b/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml
@@ -4,7 +4,7 @@
 $id: http://devicetree.org/schemas/i3c/renesas,i3c.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
-title: Renesas RZ/G3S and RZ/G3E I3C Bus Interface
+title: Renesas RZ/G3S, RZ/G3E, RZ/V2H(P) and RZ/V2N I3C Bus Interface
 
 maintainers:
   - Wolfram Sang <wsa+renesas@sang-engineering.com>
@@ -12,10 +12,16 @@ maintainers:
 
 properties:
   compatible:
-    items:
-      - enum:
-          - renesas,r9a08g045-i3c # RZ/G3S
-          - renesas,r9a09g047-i3c # RZ/G3E
+    oneOf:
+      - items:
+          - enum:
+              - renesas,r9a08g045-i3c # RZ/G3S
+              - renesas,r9a09g047-i3c # RZ/G3E
+      - items:
+          - enum:
+              - renesas,r9a09g056-i3c # RZ/V2N
+              - renesas,r9a09g057-i3c # RZ/V2H(P)
+          - const: renesas,r9a09g047-i3c
 
   reg:
     maxItems: 1
-- 
2.51.0
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Tommaso Merciai 3 weeks, 6 days ago
Hi Prabhakar,
Thank you for your patch!

On Thu, Sep 04, 2025 at 05:03:05PM +0100, Prabhakar wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> 
> Add device tree binding support for the I3C Bus Interface on Renesas
> RZ/V2H(P) and RZ/V2N SoCs. The I3C IP on these SoCs is identical to
> that found on the RZ/G3E SoC.
> 
> Add new compatible strings "renesas,r9a09g056-i3c" for RZ/V2N and
> "renesas,r9a09g057-i3c" for RZ/V2H(P). Both variants use
> "renesas,r9a09g047-i3c" as a fallback compatible to indicate hardware
> compatibility with the RZ/G3E implementation.
> 
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> ---
>  .../devicetree/bindings/i3c/renesas,i3c.yaml     | 16 +++++++++++-----
>  1 file changed, 11 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml b/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml
> index fe2e9633c46f..e41ba3ba4b58 100644
> --- a/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml
> +++ b/Documentation/devicetree/bindings/i3c/renesas,i3c.yaml
> @@ -4,7 +4,7 @@
>  $id: http://devicetree.org/schemas/i3c/renesas,i3c.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>  
> -title: Renesas RZ/G3S and RZ/G3E I3C Bus Interface
> +title: Renesas RZ/G3S, RZ/G3E, RZ/V2H(P) and RZ/V2N I3C Bus Interface
>  
>  maintainers:
>    - Wolfram Sang <wsa+renesas@sang-engineering.com>
> @@ -12,10 +12,16 @@ maintainers:
>  
>  properties:
>    compatible:
> -    items:
> -      - enum:
> -          - renesas,r9a08g045-i3c # RZ/G3S
> -          - renesas,r9a09g047-i3c # RZ/G3E
> +    oneOf:
> +      - items:
> +          - enum:
> +              - renesas,r9a08g045-i3c # RZ/G3S
> +              - renesas,r9a09g047-i3c # RZ/G3E
> +      - items:
> +          - enum:
> +              - renesas,r9a09g056-i3c # RZ/V2N
> +              - renesas,r9a09g057-i3c # RZ/V2H(P)
> +          - const: renesas,r9a09g047-i3c
>  
>    reg:
>      maxItems: 1
> -- 
> 2.51.0
> 

Looks good to me.

Reviewed-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>


Kind Regards,
Tommaso
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Wolfram Sang 4 weeks ago
Hi,

> Add device tree binding support for the I3C Bus Interface on Renesas
> RZ/V2H(P) and RZ/V2N SoCs. The I3C IP on these SoCs is identical to
> that found on the RZ/G3E SoC.

Cool, has it been tested with devices already?

> -title: Renesas RZ/G3S and RZ/G3E I3C Bus Interface
> +title: Renesas RZ/G3S, RZ/G3E, RZ/V2H(P) and RZ/V2N I3C Bus Interface

I suggest "Renesas I3C Bus Interface". The above is not going to scale.

Rest looks good from a glimpse!
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Tommaso Merciai 3 weeks, 6 days ago
Hi Wolfram, Prabhakar,
Thanks for your work!

On Thu, Sep 04, 2025 at 06:08:49PM +0200, Wolfram Sang wrote:
> Hi,
> 
> > Add device tree binding support for the I3C Bus Interface on Renesas
> > RZ/V2H(P) and RZ/V2N SoCs. The I3C IP on these SoCs is identical to
> > that found on the RZ/G3E SoC.
> 
> Cool, has it been tested with devices already?
> 
> > -title: Renesas RZ/G3S and RZ/G3E I3C Bus Interface
> > +title: Renesas RZ/G3S, RZ/G3E, RZ/V2H(P) and RZ/V2N I3C Bus Interface
> 
> I suggest "Renesas I3C Bus Interface". The above is not going to scale.

Agreed, thank 
Maybe "Renesas RZ I3C Bus Interface"

?

> 
> Rest looks good from a glimpse!
> 

Kind Regards,
Tommaso
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Wolfram Sang 3 weeks, 6 days ago
> Maybe "Renesas RZ I3C Bus Interface"
> 
> ?

It is on R-Car Gen5 as well, so... no :)
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Tommaso Merciai 3 weeks, 6 days ago

On 9/5/25 13:16, Wolfram Sang wrote:
> 
>> Maybe "Renesas RZ I3C Bus Interface"
>>
>> ?
> 
> It is on R-Car Gen5 as well, so... no :)
> 
Ouch I miss that :'(
Please kindly ignore my comment :)

Kind Regards,
Tommaso
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Lad, Prabhakar 4 weeks ago
Hi Wolfram,

Thank you for the review.

On Thu, Sep 4, 2025 at 5:08 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
>
> Hi,
>
> > Add device tree binding support for the I3C Bus Interface on Renesas
> > RZ/V2H(P) and RZ/V2N SoCs. The I3C IP on these SoCs is identical to
> > that found on the RZ/G3E SoC.
>
> Cool, has it been tested with devices already?
>
Yes with the P3T1085UK Arduino Shield Evaluation kit (logs can be found at [0]).

[0] https://lore.kernel.org/all/20250904165909.281131-1-prabhakar.mahadev-lad.rj@bp.renesas.com/

> > -title: Renesas RZ/G3S and RZ/G3E I3C Bus Interface
> > +title: Renesas RZ/G3S, RZ/G3E, RZ/V2H(P) and RZ/V2N I3C Bus Interface
>
> I suggest "Renesas I3C Bus Interface". The above is not going to scale.
>
OK, I'll update and re-spin the patch.

> Rest looks good from a glimpse!
>

Cheers,
Prabhakar
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Wolfram Sang 4 weeks ago
> Yes with the P3T1085UK Arduino Shield Evaluation kit (logs can be found at [0]).

Cool. Since it reports two temperatures, it must be my modified version
of the shield :)
Re: [PATCH] dt-bindings: i3c: renesas,i3c: Add RZ/V2H(P) and RZ/V2N support
Posted by Lad, Prabhakar 3 weeks, 6 days ago
Hi Wolfram,

On Thu, Sep 4, 2025 at 8:41 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
>
>
> > Yes with the P3T1085UK Arduino Shield Evaluation kit (logs can be found at [0]).
>
> Cool. Since it reports two temperatures, it must be my modified version
> of the shield :)
>
Yes it is ;-)

Cheers,
Prabhakar