[PATCH RESEND] ARM: dts: AM33XX: Rename I2C and GPIO nodes

AnilKumar, Chimata anilkumar at ti.com
Wed Jan 2 21:12:29 EST 2013


On Wed, Nov 21, 2012 at 17:22:17, AnilKumar, Chimata wrote:
> Rename I2C and GPIO nodes according to AM33XX TRM. According to
> AM33XX TRM device instances are starting from "0" like i2c0, i2c1
> and i2c3.
> 
> Signed-off-by: Pantelis Antoniou <panto at antoniou-consulting.com>
> [panto at antoniou-consulting.com: initial patch by pantelis's]
> Signed-off-by: AnilKumar Ch <anilkumar at ti.com>
> ---
> Changes from first version:
> 	- Updated pantelis's patch
> 	  * Modified based on linux-omap/master
> 	  * Added GPIO nodes renaming as well

Hi Tony/Benoit,

If there are no comments could you please pull this patch?

Thanks
AnilKumar
 
>  arch/arm/boot/dts/am335x-bone.dts  |   10 +++++-----
>  arch/arm/boot/dts/am335x-evm.dts   |   18 +++++++++---------
>  arch/arm/boot/dts/am335x-evmsk.dts |   18 +++++++++---------
>  arch/arm/boot/dts/am33xx.dtsi      |   14 +++++++-------
>  4 files changed, 30 insertions(+), 30 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/am335x-bone.dts b/arch/arm/boot/dts/am335x-bone.dts
> index 2c33888..77b4352 100644
> --- a/arch/arm/boot/dts/am335x-bone.dts
> +++ b/arch/arm/boot/dts/am335x-bone.dts
> @@ -43,7 +43,7 @@
>  			status = "okay";
>  		};
>  
> -		i2c1: i2c at 44e0b000 {
> +		i2c0: i2c at 44e0b000 {
>  			status = "okay";
>  			clock-frequency = <400000>;
>  
> @@ -59,27 +59,27 @@
>  
>  		led at 2 {
>  			label = "beaglebone:green:heartbeat";
> -			gpios = <&gpio2 21 0>;
> +			gpios = <&gpio1 21 0>;
>  			linux,default-trigger = "heartbeat";
>  			default-state = "off";
>  		};
>  
>  		led at 3 {
>  			label = "beaglebone:green:mmc0";
> -			gpios = <&gpio2 22 0>;
> +			gpios = <&gpio1 22 0>;
>  			linux,default-trigger = "mmc0";
>  			default-state = "off";
>  		};
>  
>  		led at 4 {
>  			label = "beaglebone:green:usr2";
> -			gpios = <&gpio2 23 0>;
> +			gpios = <&gpio1 23 0>;
>  			default-state = "off";
>  		};
>  
>  		led at 5 {
>  			label = "beaglebone:green:usr3";
> -			gpios = <&gpio2 24 0>;
> +			gpios = <&gpio1 24 0>;
>  			default-state = "off";
>  		};
>  	};
> diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
> index 9f65f17..705a9c5 100644
> --- a/arch/arm/boot/dts/am335x-evm.dts
> +++ b/arch/arm/boot/dts/am335x-evm.dts
> @@ -51,7 +51,7 @@
>  			status = "okay";
>  		};
>  
> -		i2c1: i2c at 44e0b000 {
> +		i2c0: i2c at 44e0b000 {
>  			status = "okay";
>  			clock-frequency = <400000>;
>  
> @@ -60,7 +60,7 @@
>  			};
>  		};
>  
> -		i2c2: i2c at 4802a000 {
> +		i2c1: i2c at 4802a000 {
>  			status = "okay";
>  			clock-frequency = <100000>;
>  
> @@ -123,12 +123,12 @@
>  		debounce-delay-ms = <5>;
>  		col-scan-delay-us = <2>;
>  
> -		row-gpios = <&gpio2 25 0	/* Bank1, pin25 */
> -			     &gpio2 26 0	/* Bank1, pin26 */
> -			     &gpio2 27 0>;	/* Bank1, pin27 */
> +		row-gpios = <&gpio1 25 0	/* Bank1, pin25 */
> +			     &gpio1 26 0	/* Bank1, pin26 */
> +			     &gpio1 27 0>;	/* Bank1, pin27 */
>  
> -		col-gpios = <&gpio2 21 0	/* Bank1, pin21 */
> -			     &gpio2 22 0>;	/* Bank1, pin22 */
> +		col-gpios = <&gpio1 21 0	/* Bank1, pin21 */
> +			     &gpio1 22 0>;	/* Bank1, pin22 */
>  
>  		linux,keymap = <0x0000008b	/* MENU */
>  				0x0100009e	/* BACK */
> @@ -147,14 +147,14 @@
>  		switch at 9 {
>  			label = "volume-up";
>  			linux,code = <115>;
> -			gpios = <&gpio1 2 1>;
> +			gpios = <&gpio0 2 1>;
>  			gpio-key,wakeup;
>  		};
>  
>  		switch at 10 {
>  			label = "volume-down";
>  			linux,code = <114>;
> -			gpios = <&gpio1 3 1>;
> +			gpios = <&gpio0 3 1>;
>  			gpio-key,wakeup;
>  		};
>  	};
> diff --git a/arch/arm/boot/dts/am335x-evmsk.dts b/arch/arm/boot/dts/am335x-evmsk.dts
> index f5a6162..f67c360 100644
> --- a/arch/arm/boot/dts/am335x-evmsk.dts
> +++ b/arch/arm/boot/dts/am335x-evmsk.dts
> @@ -58,7 +58,7 @@
>  			status = "okay";
>  		};
>  
> -		i2c1: i2c at 44e0b000 {
> +		i2c0: i2c at 44e0b000 {
>  			status = "okay";
>  			clock-frequency = <400000>;
>  
> @@ -115,26 +115,26 @@
>  
>  		led at 1 {
>  			label = "evmsk:green:usr0";
> -			gpios = <&gpio2 4 0>;
> +			gpios = <&gpio1 4 0>;
>  			default-state = "off";
>  		};
>  
>  		led at 2 {
>  			label = "evmsk:green:usr1";
> -			gpios = <&gpio2 5 0>;
> +			gpios = <&gpio1 5 0>;
>  			default-state = "off";
>  		};
>  
>  		led at 3 {
>  			label = "evmsk:green:mmc0";
> -			gpios = <&gpio2 6 0>;
> +			gpios = <&gpio1 6 0>;
>  			linux,default-trigger = "mmc0";
>  			default-state = "off";
>  		};
>  
>  		led at 4 {
>  			label = "evmsk:green:heartbeat";
> -			gpios = <&gpio2 7 0>;
> +			gpios = <&gpio1 7 0>;
>  			linux,default-trigger = "heartbeat";
>  			default-state = "off";
>  		};
> @@ -148,26 +148,26 @@
>  		switch at 1 {
>  			label = "button0";
>  			linux,code = <0x100>;
> -			gpios = <&gpio3 3 0>;
> +			gpios = <&gpio2 3 0>;
>  		};
>  
>  		switch at 2 {
>  			label = "button1";
>  			linux,code = <0x101>;
> -			gpios = <&gpio3 2 0>;
> +			gpios = <&gpio2 2 0>;
>  		};
>  
>  		switch at 3 {
>  			label = "button2";
>  			linux,code = <0x102>;
> -			gpios = <&gpio1 30 0>;
> +			gpios = <&gpio0 30 0>;
>  			gpio-key,wakeup;
>  		};
>  
>  		switch at 4 {
>  			label = "button3";
>  			linux,code = <0x103>;
> -			gpios = <&gpio3 5 0>;
> +			gpios = <&gpio2 5 0>;
>  		};
>  	};
>  };
> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> index 5dfd682..2ad01c7 100644
> --- a/arch/arm/boot/dts/am33xx.dtsi
> +++ b/arch/arm/boot/dts/am33xx.dtsi
> @@ -87,7 +87,7 @@
>  			reg = <0x48200000 0x1000>;
>  		};
>  
> -		gpio1: gpio at 44e07000 {
> +		gpio0: gpio at 44e07000 {
>  			compatible = "ti,omap4-gpio";
>  			ti,hwmods = "gpio1";
>  			gpio-controller;
> @@ -98,7 +98,7 @@
>  			interrupts = <96>;
>  		};
>  
> -		gpio2: gpio at 4804c000 {
> +		gpio1: gpio at 4804c000 {
>  			compatible = "ti,omap4-gpio";
>  			ti,hwmods = "gpio2";
>  			gpio-controller;
> @@ -109,7 +109,7 @@
>  			interrupts = <98>;
>  		};
>  
> -		gpio3: gpio at 481ac000 {
> +		gpio2: gpio at 481ac000 {
>  			compatible = "ti,omap4-gpio";
>  			ti,hwmods = "gpio3";
>  			gpio-controller;
> @@ -120,7 +120,7 @@
>  			interrupts = <32>;
>  		};
>  
> -		gpio4: gpio at 481ae000 {
> +		gpio3: gpio at 481ae000 {
>  			compatible = "ti,omap4-gpio";
>  			ti,hwmods = "gpio4";
>  			gpio-controller;
> @@ -185,7 +185,7 @@
>  			status = "disabled";
>  		};
>  
> -		i2c1: i2c at 44e0b000 {
> +		i2c0: i2c at 44e0b000 {
>  			compatible = "ti,omap4-i2c";
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> @@ -195,7 +195,7 @@
>  			status = "disabled";
>  		};
>  
> -		i2c2: i2c at 4802a000 {
> +		i2c1: i2c at 4802a000 {
>  			compatible = "ti,omap4-i2c";
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> @@ -205,7 +205,7 @@
>  			status = "disabled";
>  		};
>  
> -		i2c3: i2c at 4819c000 {
> +		i2c2: i2c at 4819c000 {
>  			compatible = "ti,omap4-i2c";
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> -- 
> 1.7.9.5
> 
> 



More information about the devicetree-discuss mailing list