Message ID | 53B4AC3D.30801@renesas.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
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 -- To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 {
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(-)