[PATCH] ARM: shmobile: Lager: Correct I2C bus for VDD MPU regulator

[PATCH] ARM: shmobile: Lager: Correct I2C bus for VDD MPU regulator

[Khiem Nguyen]

I2C bus for VDD MPU regulator is IIC3, not I2C3.

Signed-off-by: Khiem Nguyen <khiem.nguyen.xt@renesas.com>
---
  arch/arm/boot/dts/r8a7790-lager.dts |   10 +++++-----
  1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7790-lager.dts 
b/arch/arm/boot/dts/r8a7790-lager.dts
index 0890d47..a8f32d7 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -215,9 +215,9 @@
          renesas,function = "i2c2";
      };

-    i2c3_pins: i2c3 {
-        renesas,groups = "i2c3";
-        renesas,function = "i2c3";
+    iic3_pins: iic3 {
+        renesas,groups = "iic3";
+        renesas,function = "iic3";
      };
  };

@@ -353,9 +353,9 @@
      pinctrl-names = "default";
  };

-&i2c3 {
+&iic3 {
      pinctrl-names = "default";
-    pinctrl-0 = <&i2c3_pins>;
+    pinctrl-0 = <&iic3_pins>;
      status = "okay";

      vdd_dvfs: regulator@68 {
-- 
1.7.9.5

Re: [PATCH] ARM: shmobile: Lager: Correct I2C bus for VDD MPU regulator

[Magnus Damm]

On Thu, Jul 3, 2014 at 3:05 AM, Khiem Nguyen
<khiem.nguyen.xt@renesas.com> wrote:
> I2C bus for VDD MPU regulator is IIC3, not I2C3.
>
> Signed-off-by: Khiem Nguyen <khiem.nguyen.xt@renesas.com>
> ---
>  arch/arm/boot/dts/r8a7790-lager.dts |   10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)

Hi Khiem-san,

Thanks for your patch. Can you please extend the commit message to
include the reason behind this patch?

From my side this looks like a software policy change. Unless I'm
mistaken both I2C3 and IIC3 share the same pins on the SoC, and
because of that it should be possible to access that particular I2C
bus already without any modification.

So please explain why you want to change this.

Cheers,

/ magnus