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

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

Hello.

On 06/05/2014 04:32 AM, Simon Horman 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>
>> ---
>>   arch/arm/boot/dts/r8a7790-lager.dts | 14 ++++++++++++++
>>   1 file changed, 14 insertions(+)

> I have queued up the following.

> There was some fuzz when applying this patch, please check that the
> following is correct.

> From: Ben Dooks <ben.dooks@codethink.co.uk>

> [PATCH] ARM: shmobile: lager: add i2c1, i2c2 pins

> 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>
> [horms+renesas@verge.net.au: Added shmobile to patch title]
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>   arch/arm/boot/dts/r8a7790-lager.dts | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)

> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
> index ee80788..e33ed0d 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -139,6 +139,16 @@
>   		states = <3300000 1
>   			  1800000 0>;
>   	};
> +
> +	i2c1_pins: i2c1 {
> +		renesas,groups = "i2c1";
> +		renesas,function = "i2c1";
> +	};
> +
> +	i2c2_pins: i2c2 {
> +		renesas,groups = "i2c2";
> +		renesas,function = "i2c2";
> +	};

    These subnodes ended up added to a wrong node, they should have been added 
to the PFC node below.

>   };
>
>   &extal_clk {

WBR, Sergei