diff mbox

[2/9] ARM: lager: add i2c1, i2c2 pins

Message ID 1402862194-17743-3-git-send-email-ben.dooks@codethink.co.uk (mailing list archive)
State Superseded, archived
Headers

Commit Message

Ben Dooks June 15, 2014, 7:56 p.m. UTC 
  Add pinctrl definitions for i2c1 and i2c2 busses on the Lager board
to ensure these are setup correctly at initialisation time. The i2c0
and i2c3 busses are connected to single function pins.

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 arch/arm/boot/dts/r8a7790-lager.dts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Sergei Shtylyov June 16, 2014, 12:33 p.m. UTC | #1 
Hello.

On 06/15/2014 11:56 PM, Ben Dooks wrote:

> Add pinctrl definitions for i2c1 and i2c2 busses on the Lager board
> to ensure these are setup correctly at initialisation time. The i2c0
> and i2c3 busses are connected to single function pins.

> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>

    Likewise, this as been already merged by Simon.

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ben Dooks June 18, 2014, 7:27 a.m. UTC | #2 
On 16/06/14 13:33, Sergei Shtylyov wrote:
> Hello.
> 
> On 06/15/2014 11:56 PM, Ben Dooks wrote:
> 
>> Add pinctrl definitions for i2c1 and i2c2 busses on the Lager board
>> to ensure these are setup correctly at initialisation time. The i2c0
>> and i2c3 busses are connected to single function pins.
> 
>> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
> 
>    Likewise, this as been already merged by Simon.

Ah, they had not been merged when I took the branch for this around
-rc8 time. I will look at changing the necessary bits for the vin
in the DT and re-sub them as a new series for Simon to look at merging.
Simon Horman June 18, 2014, 7:31 a.m. UTC | #3 
On Wed, Jun 18, 2014 at 08:27:37AM +0100, Ben Dooks wrote:
> On 16/06/14 13:33, Sergei Shtylyov wrote:
> > Hello.
> > 
> > On 06/15/2014 11:56 PM, Ben Dooks wrote:
> > 
> >> Add pinctrl definitions for i2c1 and i2c2 busses on the Lager board
> >> to ensure these are setup correctly at initialisation time. The i2c0
> >> and i2c3 busses are connected to single function pins.
> > 
> >> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
> > 
> >    Likewise, this as been already merged by Simon.
> 
> Ah, they had not been merged when I took the branch for this around
> -rc8 time. I will look at changing the necessary bits for the vin
> in the DT and re-sub them as a new series for Simon to look at merging.

Thanks, Ben.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index 8617755..4805c9f 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -204,6 +204,16 @@ 
 				 "msiof1_tx";
 		renesas,function = "msiof1";
 	};
+
+	i2c1_pins: i2c1 {
+		renesas,groups = "i2c1";
+		renesas,function = "i2c1";
+	};
+
+	i2c2_pins: i2c2 {
+		renesas,groups = "i2c2";
+		renesas,function = "i2c2";
+	};
 };
 
 &ether {
@@ -324,10 +334,14 @@ 
 
 &i2c1	{
 	status = "ok";
+	pinctrl-0 = <&i2c1_pins>;
+	pinctrl-names = "default";
 };
 
 &i2c2	{
 	status = "ok";
+	pinctrl-0 = <&i2c2_pins>;
+	pinctrl-names = "default";
 };
 
 &i2c3	{