[PATCH 0/5] Add RZ/G2L I2C support

STALE1858d

Revision v1 of 3 in this series.

9 messages, 3 authors, 2021-06-14 · open the first message on its own page

[PATCH 0/5] Add RZ/G2L I2C support

From: Biju Das <biju.das.jz@bp.renesas.com>
Date: 2021-06-11 16:56:34

This patch series aims to support I2C on RZ/G2L based platforms.

This patchset is based on master branch [1]

[1] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/

Biju Das (5):
  dt-bindings: i2c: renesas,riic: Document RZ/G2L I2C controller
  drivers: clk: renesas: r9a07g044-cpg: Add I2C Clocks
  i2c: riic: Add RZ/G2L support
  arm64: dts: renesas: r9a07g044: Add I2C nodes
  arm64: defconfig: Enable RIIC

 .../devicetree/bindings/i2c/renesas,riic.yaml | 19 ++++-
 arch/arm64/boot/dts/renesas/r9a07g044.dtsi    | 80 +++++++++++++++++++
 arch/arm64/configs/defconfig                  |  1 +
 drivers/clk/renesas/r9a07g044-cpg.c           | 12 +++
 drivers/i2c/busses/Kconfig                    |  1 +
 drivers/i2c/busses/i2c-riic.c                 | 21 +++++
 6 files changed, 130 insertions(+), 4 deletions(-)

-- 
2.17.1

[PATCH 1/5] dt-bindings: i2c: renesas,riic: Document RZ/G2L I2C controller

From: Biju Das <biju.das.jz@bp.renesas.com>
Date: 2021-06-11 16:56:36

Document RZ/G2L I2C controller bindings.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 .../devicetree/bindings/i2c/renesas,riic.yaml | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/i2c/renesas,riic.yaml b/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
index 2d6378164958..52d92ec7ec0b 100644
--- a/Documentation/devicetree/bindings/i2c/renesas,riic.yaml
+++ b/Documentation/devicetree/bindings/i2c/renesas,riic.yaml

@@ -4,7 +4,7 @@
 $id: http://devicetree.org/schemas/i2c/renesas,riic.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
-title: Renesas RZ/A I2C Bus Interface (RIIC)
+title: Renesas RZ/A and RZ/G2L I2C Bus Interface (RIIC)
 
 maintainers:
   - Chris Brandt <chris.brandt@renesas.com>

@@ -17,9 +17,10 @@ properties:
   compatible:
     items:
       - enum:
-          - renesas,riic-r7s72100 # RZ/A1H
-          - renesas,riic-r7s9210  # RZ/A2M
-      - const: renesas,riic-rz    # RZ/A
+          - renesas,riic-r7s72100   # RZ/A1H
+          - renesas,riic-r7s9210    # RZ/A2M
+          - renesas,riic-r9a07g044  # RZ/G2{L,LC}
+      - const: renesas,riic-rz      # RZ/A or RZ/G2L
 
   reg:
     maxItems: 1

@@ -56,6 +57,16 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+if:
+  properties:
+    compatible:
+      contains:
+        enum:
+          - renesas,riic-r9a07g044
+then:
+  required:
+    - resets
+
 unevaluatedProperties: false
 
 examples:

-- 
2.17.1

[PATCH 3/5] i2c: riic: Add RZ/G2L support

From: Biju Das <biju.das.jz@bp.renesas.com>
Date: 2021-06-11 16:56:43

RZ/G2L i2c controller is compatible with RZ/A i2c controller.
By default IP is in reset state, so need to perform release
reset before accessing any register.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/i2c/busses/Kconfig    |  1 +
 drivers/i2c/busses/i2c-riic.c | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 281a65d9b44b..5da09288b461 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig

@@ -941,6 +941,7 @@ config I2C_QUP
 config I2C_RIIC
 	tristate "Renesas RIIC adapter"
 	depends on ARCH_RENESAS || COMPILE_TEST
+	select RESET_CONTROLLER
 	help
 	  If you say yes to this option, support will be included for the
 	  Renesas RIIC I2C interface.

diff --git a/drivers/i2c/busses/i2c-riic.c b/drivers/i2c/busses/i2c-riic.c
index 4eccc0f69861..a3860631c6db 100644
--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c

@@ -42,8 +42,10 @@
 #include <linux/io.h>
 #include <linux/module.h>
 #include <linux/of.h>
+#include <linux/of_device.h>
 #include <linux/platform_device.h>
 #include <linux/pm_runtime.h>
+#include <linux/reset.h>
 
 #define RIIC_ICCR1	0x00
 #define RIIC_ICCR2	0x04

@@ -86,6 +88,11 @@
 
 #define RIIC_INIT_MSG	-1
 
+enum riic_type {
+	RIIC_RZ_A,
+	RIIC_RZ_G2L,
+};
+
 struct riic_dev {
 	void __iomem *base;
 	u8 *buf;

@@ -395,7 +402,9 @@ static int riic_i2c_probe(struct platform_device *pdev)
 	struct i2c_adapter *adap;
 	struct resource *res;
 	struct i2c_timings i2c_t;
+	struct reset_control *rstc;
 	int i, ret;
+	enum riic_type type;
 
 	riic = devm_kzalloc(&pdev->dev, sizeof(*riic), GFP_KERNEL);
 	if (!riic)

@@ -412,6 +421,17 @@ static int riic_i2c_probe(struct platform_device *pdev)
 		return PTR_ERR(riic->clk);
 	}
 
+	type = (enum riic_type)of_device_get_match_data(&pdev->dev);
+	if (type == RIIC_RZ_G2L) {
+		rstc = devm_reset_control_get(&pdev->dev, NULL);
+		if (IS_ERR(rstc)) {
+			dev_err(&pdev->dev, "Error: missing reset ctrl\n");
+			return PTR_ERR(rstc);
+		}
+
+		reset_control_deassert(rstc);
+	}
+
 	for (i = 0; i < ARRAY_SIZE(riic_irqs); i++) {
 		res = platform_get_resource(pdev, IORESOURCE_IRQ, riic_irqs[i].res_num);
 		if (!res)

@@ -472,6 +492,7 @@ static int riic_i2c_remove(struct platform_device *pdev)
 }
 
 static const struct of_device_id riic_i2c_dt_ids[] = {
+	{ .compatible = "renesas,riic-r9a07g044", .data = (void *)RIIC_RZ_G2L },
 	{ .compatible = "renesas,riic-rz" },
 	{ /* Sentinel */ },
 };

-- 
2.17.1

Re: [PATCH 3/5] i2c: riic: Add RZ/G2L support

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2021-06-14 12:38:40

Hi Biju,

On Fri, Jun 11, 2021 at 6:56 PM Biju Das [off-list ref] wrote:

RZ/G2L i2c controller is compatible with RZ/A i2c controller.
By default IP is in reset state, so need to perform release
reset before accessing any register.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Thanks for your patch!

quoted hunk

--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig

@@ -941,6 +941,7 @@ config I2C_QUP
 config I2C_RIIC
        tristate "Renesas RIIC adapter"
        depends on ARCH_RENESAS || COMPILE_TEST
+       select RESET_CONTROLLER

if ARCH_R9A07G044?

quoted hunk

--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c

_kzalloc(&pdev->dev, sizeof(*riic), GFP_KERNEL);

quoted hunk

        if (!riic)

@@ -412,6 +421,17 @@ static int riic_i2c_probe(struct platform_device *pdev)
                return PTR_ERR(riic->clk);
        }

+       type = (enum riic_type)of_device_get_match_data(&pdev->dev);
+       if (type == RIIC_RZ_G2L) {
+               rstc = devm_reset_control_get(&pdev->dev, NULL);
+               if (IS_ERR(rstc)) {
+                       dev_err(&pdev->dev, "Error: missing reset ctrl\n");
+                       return PTR_ERR(rstc);
+               }
+
+               reset_control_deassert(rstc);

Just wondering: does it harm if the driver is unloaded or unbounded,
and rebound while the I2C controller is not in reset state?

quoted hunk

+       }
+
        for (i = 0; i < ARRAY_SIZE(riic_irqs); i++) {
                res = platform_get_resource(pdev, IORESOURCE_IRQ, riic_irqs[i].res_num);
                if (!res)

@@ -472,6 +492,7 @@ static int riic_i2c_remove(struct platform_device *pdev)
 }

 static const struct of_device_id riic_i2c_dt_ids[] = {
+       { .compatible = "renesas,riic-r9a07g044", .data = (void *)RIIC_RZ_G2L },
        { .compatible = "renesas,riic-rz" },

Please fill in .data, to avoid relying implicitly on RIIC_RZ_A being zero.

        { /* Sentinel */ },
 };

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/5] dt-bindings: i2c: renesas,riic: Document RZ/G2L I2C controller

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2021-06-14 12:43:19

On Fri, Jun 11, 2021 at 6:56 PM Biju Das [off-list ref] wrote:

Document RZ/G2L I2C controller bindings.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@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 3/5] i2c: riic: Add RZ/G2L support

From: Philipp Zabel <p.zabel@pengutronix.de>
Date: 2021-06-14 13:27:50

Hi Biju,

On Fri, 2021-06-11 at 17:56 +0100, Biju Das wrote:

quoted hunk

RZ/G2L i2c controller is compatible with RZ/A i2c controller.
By default IP is in reset state, so need to perform release
reset before accessing any register.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/i2c/busses/Kconfig    |  1 +
 drivers/i2c/busses/i2c-riic.c | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 281a65d9b44b..5da09288b461 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig

@@ -941,6 +941,7 @@ config I2C_QUP
 config I2C_RIIC
 	tristate "Renesas RIIC adapter"
 	depends on ARCH_RENESAS || COMPILE_TEST
+	select RESET_CONTROLLER

There's no need for this. The reset API defines inline stubs so this can
be compiled without RESET_CONTROLLER enabled.

quoted hunk

 	help
 	  If you say yes to this option, support will be included for the
 	  Renesas RIIC I2C interface.

diff --git a/drivers/i2c/busses/i2c-riic.c b/drivers/i2c/busses/i2c-riic.c
index 4eccc0f69861..a3860631c6db 100644
--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c

@@ -42,8 +42,10 @@
 #include <linux/io.h>
 #include <linux/module.h>
 #include <linux/of.h>
+#include <linux/of_device.h>
 #include <linux/platform_device.h>
 #include <linux/pm_runtime.h>
+#include <linux/reset.h>
 
 #define RIIC_ICCR1	0x00
 #define RIIC_ICCR2	0x04

@@ -86,6 +88,11 @@
 
 #define RIIC_INIT_MSG	-1
 
+enum riic_type {
+	RIIC_RZ_A,
+	RIIC_RZ_G2L,
+};
+
 struct riic_dev {
 	void __iomem *base;
 	u8 *buf;

@@ -395,7 +402,9 @@ static int riic_i2c_probe(struct platform_device *pdev)
 	struct i2c_adapter *adap;
 	struct resource *res;
 	struct i2c_timings i2c_t;
+	struct reset_control *rstc;
 	int i, ret;
+	enum riic_type type;
 
 	riic = devm_kzalloc(&pdev->dev, sizeof(*riic), GFP_KERNEL);
 	if (!riic)

@@ -412,6 +421,17 @@ static int riic_i2c_probe(struct platform_device *pdev)
 		return PTR_ERR(riic->clk);
 	}
 
+	type = (enum riic_type)of_device_get_match_data(&pdev->dev);
+	if (type == RIIC_RZ_G2L) {
+		rstc = devm_reset_control_get(&pdev->dev, NULL);

Please use devm_reset_control_get_exclusive().

quoted hunk

+		if (IS_ERR(rstc)) {
+			dev_err(&pdev->dev, "Error: missing reset ctrl\n");
+			return PTR_ERR(rstc);
+		}
+
+		reset_control_deassert(rstc);
+	}
+
 	for (i = 0; i < ARRAY_SIZE(riic_irqs); i++) {
 		res = platform_get_resource(pdev, IORESOURCE_IRQ, riic_irqs[i].res_num);
 		if (!res)

@@ -472,6 +492,7 @@ static int riic_i2c_remove(struct platform_device *pdev)
 }
 
 static const struct of_device_id riic_i2c_dt_ids[] = {
+	{ .compatible = "renesas,riic-r9a07g044", .data = (void *)RIIC_RZ_G2L },
 	{ .compatible = "renesas,riic-rz" },
 	{ /* Sentinel */ },
 };

regards
Philipp

Re: [PATCH 3/5] i2c: riic: Add RZ/G2L support

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2021-06-14 13:39:00

Hi Philipp,

On Mon, Jun 14, 2021 at 3:27 PM Philipp Zabel [off-list ref] wrote:

On Fri, 2021-06-11 at 17:56 +0100, Biju Das wrote:

quoted

RZ/G2L i2c controller is compatible with RZ/A i2c controller.
By default IP is in reset state, so need to perform release
reset before accessing any register.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/i2c/busses/Kconfig    |  1 +
 drivers/i2c/busses/i2c-riic.c | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 281a65d9b44b..5da09288b461 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig

@@ -941,6 +941,7 @@ config I2C_QUP
 config I2C_RIIC
      tristate "Renesas RIIC adapter"
      depends on ARCH_RENESAS || COMPILE_TEST
+     select RESET_CONTROLLER

There's no need for this. The reset API defines inline stubs so this can
be compiled without RESET_CONTROLLER enabled.

AFAIK, the issue is that RIIC on RZ/G2L requires reset support,
so it must be enabled when building a kernel for RZ/G2L.
As RZ/A does not need or use it, and may run from SRAM, I'd like to
leave it disabled when building a kernel not including RZ/G2L support.

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 3/5] i2c: riic: Add RZ/G2L support

From: Biju Das <biju.das.jz@bp.renesas.com>
Date: 2021-06-14 16:05:00

Hi Geert,

Thanks for the feedback.

-----Original Message-----
Subject: Re: [PATCH 3/5] i2c: riic: Add RZ/G2L support

Hi Biju,

On Fri, Jun 11, 2021 at 6:56 PM Biju Das [off-list ref]
wrote:

quoted

RZ/G2L i2c controller is compatible with RZ/A i2c controller.
By default IP is in reset state, so need to perform release reset
before accessing any register.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Thanks for your patch!

quoted

--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig

@@ -941,6 +941,7 @@ config I2C_QUP
 config I2C_RIIC
        tristate "Renesas RIIC adapter"
        depends on ARCH_RENESAS || COMPILE_TEST
+       select RESET_CONTROLLER

if ARCH_R9A07G044?

OK,Agreed.

quoted

--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c

_kzalloc(&pdev->dev, sizeof(*riic), GFP_KERNEL);

quoted

        if (!riic)

@@ -412,6 +421,17 @@ static int riic_i2c_probe(struct platform_device

*pdev)

quoted

                return PTR_ERR(riic->clk);
        }

+       type = (enum riic_type)of_device_get_match_data(&pdev->dev);
+       if (type == RIIC_RZ_G2L) {
+               rstc = devm_reset_control_get(&pdev->dev, NULL);
+               if (IS_ERR(rstc)) {
+                       dev_err(&pdev->dev, "Error: missing reset

ctrl\n");

quoted

+                       return PTR_ERR(rstc);
+               }
+
+               reset_control_deassert(rstc);

Just wondering: does it harm if the driver is unloaded or unbounded, and
rebound while the I2C controller is not in reset state?

I have tested and it works after unloading and reloading it again.

Cheers,
Biju

quoted

+       }
+
        for (i = 0; i < ARRAY_SIZE(riic_irqs); i++) {
                res = platform_get_resource(pdev, IORESOURCE_IRQ,

riic_irqs[i].res_num);

quoted

                if (!res)

@@ -472,6 +492,7 @@ static int riic_i2c_remove(struct platform_device

*pdev)  }

 static const struct of_device_id riic_i2c_dt_ids[] = {
+       { .compatible = "renesas,riic-r9a07g044", .data = (void
+ *)RIIC_RZ_G2L },
        { .compatible = "renesas,riic-rz" },

Please fill in .data, to avoid relying implicitly on RIIC_RZ_A being zero.

quoted

        { /* Sentinel */ },
 };

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 3/5] i2c: riic: Add RZ/G2L support

From: Biju Das <biju.das.jz@bp.renesas.com>
Date: 2021-06-14 17:20:15

Hi Philipp,

Thanks for the review.

Subject: Re: [PATCH 3/5] i2c: riic: Add RZ/G2L support

Hi Biju,

On Fri, 2021-06-11 at 17:56 +0100, Biju Das wrote:

quoted

RZ/G2L i2c controller is compatible with RZ/A i2c controller.
By default IP is in reset state, so need to perform release reset
before accessing any register.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/i2c/busses/Kconfig    |  1 +
 drivers/i2c/busses/i2c-riic.c | 21 +++++++++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 281a65d9b44b..5da09288b461 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig

@@ -941,6 +941,7 @@ config I2C_QUP
 config I2C_RIIC
 	tristate "Renesas RIIC adapter"
 	depends on ARCH_RENESAS || COMPILE_TEST
+	select RESET_CONTROLLER

There's no need for this. The reset API defines inline stubs so this can
be compiled without RESET_CONTROLLER enabled.

Hope you are ok with Geert's suggestion, as RZ/A boards are resource constrained.
So it is not good to enable for them.

quoted

 	help
 	  If you say yes to this option, support will be included for the
 	  Renesas RIIC I2C interface.

diff --git a/drivers/i2c/busses/i2c-riic.c

b/drivers/i2c/busses/i2c-riic.c index 4eccc0f69861..a3860631c6db
100644

--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c

@@ -42,8 +42,10 @@
 #include <linux/io.h>
 #include <linux/module.h>
 #include <linux/of.h>
+#include <linux/of_device.h>
 #include <linux/platform_device.h>
 #include <linux/pm_runtime.h>
+#include <linux/reset.h>

 #define RIIC_ICCR1	0x00
 #define RIIC_ICCR2	0x04

@@ -86,6 +88,11 @@

 #define RIIC_INIT_MSG	-1

+enum riic_type {
+	RIIC_RZ_A,
+	RIIC_RZ_G2L,
+};
+
 struct riic_dev {
 	void __iomem *base;
 	u8 *buf;

@@ -395,7 +402,9 @@ static int riic_i2c_probe(struct platform_device

*pdev)

quoted

 	struct i2c_adapter *adap;
 	struct resource *res;
 	struct i2c_timings i2c_t;
+	struct reset_control *rstc;
 	int i, ret;
+	enum riic_type type;

 	riic = devm_kzalloc(&pdev->dev, sizeof(*riic), GFP_KERNEL);
 	if (!riic)

@@ -412,6 +421,17 @@ static int riic_i2c_probe(struct platform_device

*pdev)

quoted

 		return PTR_ERR(riic->clk);
 	}

+	type = (enum riic_type)of_device_get_match_data(&pdev->dev);
+	if (type == RIIC_RZ_G2L) {
+		rstc = devm_reset_control_get(&pdev->dev, NULL);

Please use devm_reset_control_get_exclusive().

OK. Will do.

Regards,
Biju

quoted

+		if (IS_ERR(rstc)) {
+			dev_err(&pdev->dev, "Error: missing reset ctrl\n");
+			return PTR_ERR(rstc);
+		}
+
+		reset_control_deassert(rstc);
+	}
+
 	for (i = 0; i < ARRAY_SIZE(riic_irqs); i++) {
 		res = platform_get_resource(pdev, IORESOURCE_IRQ,

riic_irqs[i].res_num);

quoted

 		if (!res)

@@ -472,6 +492,7 @@ static int riic_i2c_remove(struct platform_device

*pdev)  }

 static const struct of_device_id riic_i2c_dt_ids[] = {
+	{ .compatible = "renesas,riic-r9a07g044", .data = (void
+*)RIIC_RZ_G2L },
 	{ .compatible = "renesas,riic-rz" },
 	{ /* Sentinel */ },
 };

regards
Philipp

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help