[PATCH 07/14][RFC] arm64: renesas: salvator-x: enable I2C

[PATCH 07/14][RFC] arm64: renesas: salvator-x: enable I2C

[Kuninori Morimoto]

From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index f522fda..0599b2b 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -45,6 +45,11 @@
 		renesas,groups = "scif2_data_a";
 		renesas,function = "scif2";
 	};
+
+	i2c2_pins: i2c2 {
+		renesas,groups = "i2c2_a";
+		renesas,function = "i2c2";
+	};
 };
 
 &scif1 {
@@ -60,3 +65,10 @@
 
 	status = "okay";
 };
+
+&i2c2 {
+	pinctrl-0 = <&i2c2_pins>;
+	pinctrl-names = "default";
+
+	status = "okay";
+};
-- 
1.9.1

Re: [PATCH 07/14][RFC] arm64: renesas: salvator-x: enable I2C

[Geert Uytterhoeven]

On Thu, Sep 10, 2015 at 9:16 AM, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Acked-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 07/14][RFC] arm64: renesas: salvator-x: enable I2C

[Wolfram Sang]

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


> +&i2c2 {
> +	pinctrl-0 = <&i2c2_pins>;
> +	pinctrl-names = "default";
> +
> +	status = "okay";
> +};

I think we should add the bus speed here explicitly and not rely on a
default:

	clock-frequency = <100000>;

Maybe 400000 if all connected devices support that.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 07/14][RFC] arm64: renesas: salvator-x: enable I2C

[Kuninori Morimoto]

Hi Wolfram

> > +&i2c2 {
> > +	pinctrl-0 = <&i2c2_pins>;
> > +	pinctrl-names = "default";
> > +
> > +	status = "okay";
> > +};
> 
> I think we should add the bus speed here explicitly and not rely on a
> default:
> 
> 	clock-frequency = <100000>;
> 
> Maybe 400000 if all connected devices support that.

Thanks, actually I used clock-frequency = <100000> in my test.
Then, removed it for posting.
will add it again in v2