[PATCH 0/3] Add RZ/G3E RIIC support

[PATCH 0/3] Add RZ/G3E RIIC support

[Biju Das]

Add support for the I2C Bus Interface (RIIC) found in the Renesas
RZ/G3E (R9A09G047) SoC. This IP is compatible with Renesas RZ/V2H
(R9A09G057) RIIC IP.

Note:
This patch series depend on [1] for clk and [2] for soc dtsi

[1] https://lore.kernel.org/all/20241213123550.289193-1-biju.das.jz@bp.renesas.com/
[2] https://lore.kernel.org/all/20241213173901.599226-7-biju.das.jz@bp.renesas.com/#t

Biju Das (3):
  dt-bindings: i2c: renesas,riic: Document the R9A09G047 support
  clk: renesas: r9a09g047: Add I2C clocks/resets
  arm64: dts: renesas: r9a09g047: Add I2C nodes

 .../devicetree/bindings/i2c/renesas,riic.yaml |   4 +-
 arch/arm64/boot/dts/renesas/r9a09g047.dtsi    | 189 ++++++++++++++++++
 drivers/clk/renesas/r9a09g047-cpg.c           |  32 +++
 3 files changed, 224 insertions(+), 1 deletion(-)

-- 
2.43.0

[PATCH 1/3] dt-bindings: i2c: renesas,riic: Document the R9A09G047 support

[Biju Das]

Document support for the I2C Bus Interface (RIIC) found in the Renesas
RZ/G3E (R9A09G047) SoC. This IP is compatible with Renesas RZ/V2H
(R9A09G057) RIIC IP.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
 Documentation/devicetree/bindings/i2c/renesas,riic.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/i2c/renesas,riic.yaml b/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
index 505a8ec92266..830d5baa32c8 100644
--- a/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
+++ b/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
@@ -26,7 +26,9 @@ properties:
           - const: renesas,riic-rz      # RZ/A or RZ/G2L
 
       - items:
-          - const: renesas,riic-r9a08g045   # RZ/G3S
+          - enum:
+              - renesas,riic-r9a08g045   # RZ/G3S
+              - renesas,riic-r9a09g047   # RZ/G3E
           - const: renesas,riic-r9a09g057   # RZ/V2H(P)
 
       - const: renesas,riic-r9a09g057   # RZ/V2H(P)
-- 
2.43.0

Re: [PATCH 1/3] dt-bindings: i2c: renesas,riic: Document the R9A09G047 support

[Geert Uytterhoeven]

On Mon, Dec 16, 2024 at 1:01 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Document support for the I2C Bus Interface (RIIC) found in the Renesas
> RZ/G3E (R9A09G047) SoC. This IP is compatible with Renesas RZ/V2H
> (R9A09G057) RIIC IP.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH 1/3] dt-bindings: i2c: renesas,riic: Document the R9A09G047 support

[Geert Uytterhoeven]

Hi Biju,

On Mon, Dec 16, 2024 at 1:01 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Document support for the I2C Bus Interface (RIIC) found in the Renesas
> RZ/G3E (R9A09G047) SoC. This IP is compatible with Renesas RZ/V2H
> (R9A09G057) RIIC IP.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>

> --- a/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
> +++ b/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
> @@ -26,7 +26,9 @@ properties:
>            - const: renesas,riic-rz      # RZ/A or RZ/G2L
>
>        - items:
> -          - const: renesas,riic-r9a08g045   # RZ/G3S
> +          - enum:
> +              - renesas,riic-r9a08g045   # RZ/G3S
> +              - renesas,riic-r9a09g047   # RZ/G3E
>            - const: renesas,riic-r9a09g057   # RZ/V2H(P)
>
>        - const: renesas,riic-r9a09g057   # RZ/V2H(P)

BTW, I think you want to make resets required for renesas,riic-r9a09g057, too.
Or perhaps make it required unless compatible with
renesas,riic-r7s72100 or renesas,riic-r7s9210.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH 1/3] dt-bindings: i2c: renesas,riic: Document the R9A09G047 support

[Conor Dooley]

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

On Mon, Dec 16, 2024 at 12:00:23PM +0000, Biju Das wrote:
> Document support for the I2C Bus Interface (RIIC) found in the Renesas
> RZ/G3E (R9A09G047) SoC. This IP is compatible with Renesas RZ/V2H
> (R9A09G057) RIIC IP.
> 
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>

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

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

RE: [PATCH 1/3] dt-bindings: i2c: renesas,riic: Document the R9A09G047 support

[Biju Das]

Hi Geert,

> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 16 December 2024 16:34
> Subject: Re: [PATCH 1/3] dt-bindings: i2c: renesas,riic: Document the R9A09G047 support
> 
> Hi Biju,
> 
> On Mon, Dec 16, 2024 at 1:01 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> > Document support for the I2C Bus Interface (RIIC) found in the Renesas
> > RZ/G3E (R9A09G047) SoC. This IP is compatible with Renesas RZ/V2H
> > (R9A09G057) RIIC IP.
> >
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> 
> > --- a/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
> > +++ b/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
> > @@ -26,7 +26,9 @@ properties:
> >            - const: renesas,riic-rz      # RZ/A or RZ/G2L
> >
> >        - items:
> > -          - const: renesas,riic-r9a08g045   # RZ/G3S
> > +          - enum:
> > +              - renesas,riic-r9a08g045   # RZ/G3S
> > +              - renesas,riic-r9a09g047   # RZ/G3E
> >            - const: renesas,riic-r9a09g057   # RZ/V2H(P)
> >
> >        - const: renesas,riic-r9a09g057   # RZ/V2H(P)
> 
> BTW, I think you want to make resets required for renesas,riic-r9a09g057, too.
> Or perhaps make it required unless compatible with
> renesas,riic-r7s72100 or renesas,riic-r7s9210.

Thanks for pointing out. I will make reset required property for
renesas,riic-r9a09g057 compatible devices.

Cheers,
Biju