* [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes
@ 2012-08-27 12:31 ` Vaibhav Hiremath
0 siblings, 0 replies; 12+ messages in thread
From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw)
To: linux-omap-u79uwXL29TY76Z2rM5mHXA
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
This series is trivial patch-series and should be considered as
preparation for the future where we supposed to get rid of
hwmod dependency.
1/2: Converts all hex numbers to lowercase, fixing inconsistency
2/2: Add reg and interrupt property to all device/module nodes inside
DTS files.
Although currently hwmod overwrites resources, I have validated this
patch series by changing the omap_device layer to respect DT resources
and boot Tested on BeagleBone platform.
I will be submitting the changes for omap_device layer as well, still
needs to fix on certain things.
Vaibhav Hiremath (2):
arm/dts: AM33XX: Convert all hex numbers to lower-case
arm/dts: AM33XX: Specify reg and interrupt property for all nodes
arch/arm/boot/dts/am335x-bone.dts | 2 +-
arch/arm/boot/dts/am335x-evm.dts | 2 +-
arch/arm/boot/dts/am33xx.dtsi | 62 +++++++++++++++++++++++++++++++------
3 files changed, 54 insertions(+), 12 deletions(-)
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes @ 2012-08-27 12:31 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-omap Cc: linux-arm-kernel, devicetree-discuss, linux-kernel, Vaibhav Hiremath This series is trivial patch-series and should be considered as preparation for the future where we supposed to get rid of hwmod dependency. 1/2: Converts all hex numbers to lowercase, fixing inconsistency 2/2: Add reg and interrupt property to all device/module nodes inside DTS files. Although currently hwmod overwrites resources, I have validated this patch series by changing the omap_device layer to respect DT resources and boot Tested on BeagleBone platform. I will be submitting the changes for omap_device layer as well, still needs to fix on certain things. Vaibhav Hiremath (2): arm/dts: AM33XX: Convert all hex numbers to lower-case arm/dts: AM33XX: Specify reg and interrupt property for all nodes arch/arm/boot/dts/am335x-bone.dts | 2 +- arch/arm/boot/dts/am335x-evm.dts | 2 +- arch/arm/boot/dts/am33xx.dtsi | 62 +++++++++++++++++++++++++++++++------ 3 files changed, 54 insertions(+), 12 deletions(-) ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes @ 2012-08-27 12:31 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-arm-kernel This series is trivial patch-series and should be considered as preparation for the future where we supposed to get rid of hwmod dependency. 1/2: Converts all hex numbers to lowercase, fixing inconsistency 2/2: Add reg and interrupt property to all device/module nodes inside DTS files. Although currently hwmod overwrites resources, I have validated this patch series by changing the omap_device layer to respect DT resources and boot Tested on BeagleBone platform. I will be submitting the changes for omap_device layer as well, still needs to fix on certain things. Vaibhav Hiremath (2): arm/dts: AM33XX: Convert all hex numbers to lower-case arm/dts: AM33XX: Specify reg and interrupt property for all nodes arch/arm/boot/dts/am335x-bone.dts | 2 +- arch/arm/boot/dts/am335x-evm.dts | 2 +- arch/arm/boot/dts/am33xx.dtsi | 62 +++++++++++++++++++++++++++++++------ 3 files changed, 54 insertions(+), 12 deletions(-) ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <1346070714-6122-1-git-send-email-hvaibhav-l0cyMroinI0@public.gmane.org>]
* [PATCH 1/2] arm/dts: AM33XX: Convert all hex numbers to lower-case 2012-08-27 12:31 ` Vaibhav Hiremath (?) @ 2012-08-27 12:31 ` Vaibhav Hiremath -1 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-omap-u79uwXL29TY76Z2rM5mHXA Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r To make it consistent, convert all hex number presentation to lower-case from all am33xx specific nodes. Signed-off-by: Vaibhav Hiremath <hvaibhav-l0cyMroinI0@public.gmane.org> Cc: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> --- arch/arm/boot/dts/am335x-bone.dts | 2 +- arch/arm/boot/dts/am335x-evm.dts | 2 +- arch/arm/boot/dts/am33xx.dtsi | 20 ++++++++++---------- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch/arm/boot/dts/am335x-bone.dts b/arch/arm/boot/dts/am335x-bone.dts index a7906cb..c64f7fa 100644 --- a/arch/arm/boot/dts/am335x-bone.dts +++ b/arch/arm/boot/dts/am335x-bone.dts @@ -19,7 +19,7 @@ }; ocp { - uart1: serial@44E09000 { + uart1: serial@44e09000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts index 5dd8a6b..8963f20 100644 --- a/arch/arm/boot/dts/am335x-evm.dts +++ b/arch/arm/boot/dts/am335x-evm.dts @@ -19,7 +19,7 @@ }; ocp { - uart1: serial@44E09000 { + uart1: serial@44e09000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index dde76f7..be43511 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -71,7 +71,7 @@ #interrupt-cells = <1>; }; - gpio2: gpio@4804C000 { + gpio2: gpio@4804c000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio2"; gpio-controller; @@ -80,7 +80,7 @@ #interrupt-cells = <1>; }; - gpio3: gpio@481AC000 { + gpio3: gpio@481ac000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio3"; gpio-controller; @@ -89,7 +89,7 @@ #interrupt-cells = <1>; }; - gpio4: gpio@481AE000 { + gpio4: gpio@481ae000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio4"; gpio-controller; @@ -98,7 +98,7 @@ #interrupt-cells = <1>; }; - uart1: serial@44E09000 { + uart1: serial@44e09000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart1"; clock-frequency = <48000000>; @@ -119,28 +119,28 @@ status = "disabled"; }; - uart4: serial@481A6000 { + uart4: serial@481a6000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart4"; clock-frequency = <48000000>; status = "disabled"; }; - uart5: serial@481A8000 { + uart5: serial@481a8000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart5"; clock-frequency = <48000000>; status = "disabled"; }; - uart6: serial@481AA000 { + uart6: serial@481aa000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart6"; clock-frequency = <48000000>; status = "disabled"; }; - i2c1: i2c@44E0B000 { + i2c1: i2c@44e0b000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; @@ -148,7 +148,7 @@ status = "disabled"; }; - i2c2: i2c@4802A000 { + i2c2: i2c@4802a000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; @@ -156,7 +156,7 @@ status = "disabled"; }; - i2c3: i2c@4819C000 { + i2c3: i2c@4819c000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 1/2] arm/dts: AM33XX: Convert all hex numbers to lower-case @ 2012-08-27 12:31 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-omap Cc: linux-arm-kernel, devicetree-discuss, linux-kernel, Vaibhav Hiremath, Tony Lindgren To make it consistent, convert all hex number presentation to lower-case from all am33xx specific nodes. Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> Cc: Tony Lindgren <tony@atomide.com> --- arch/arm/boot/dts/am335x-bone.dts | 2 +- arch/arm/boot/dts/am335x-evm.dts | 2 +- arch/arm/boot/dts/am33xx.dtsi | 20 ++++++++++---------- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch/arm/boot/dts/am335x-bone.dts b/arch/arm/boot/dts/am335x-bone.dts index a7906cb..c64f7fa 100644 --- a/arch/arm/boot/dts/am335x-bone.dts +++ b/arch/arm/boot/dts/am335x-bone.dts @@ -19,7 +19,7 @@ }; ocp { - uart1: serial@44E09000 { + uart1: serial@44e09000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts index 5dd8a6b..8963f20 100644 --- a/arch/arm/boot/dts/am335x-evm.dts +++ b/arch/arm/boot/dts/am335x-evm.dts @@ -19,7 +19,7 @@ }; ocp { - uart1: serial@44E09000 { + uart1: serial@44e09000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index dde76f7..be43511 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -71,7 +71,7 @@ #interrupt-cells = <1>; }; - gpio2: gpio@4804C000 { + gpio2: gpio@4804c000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio2"; gpio-controller; @@ -80,7 +80,7 @@ #interrupt-cells = <1>; }; - gpio3: gpio@481AC000 { + gpio3: gpio@481ac000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio3"; gpio-controller; @@ -89,7 +89,7 @@ #interrupt-cells = <1>; }; - gpio4: gpio@481AE000 { + gpio4: gpio@481ae000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio4"; gpio-controller; @@ -98,7 +98,7 @@ #interrupt-cells = <1>; }; - uart1: serial@44E09000 { + uart1: serial@44e09000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart1"; clock-frequency = <48000000>; @@ -119,28 +119,28 @@ status = "disabled"; }; - uart4: serial@481A6000 { + uart4: serial@481a6000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart4"; clock-frequency = <48000000>; status = "disabled"; }; - uart5: serial@481A8000 { + uart5: serial@481a8000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart5"; clock-frequency = <48000000>; status = "disabled"; }; - uart6: serial@481AA000 { + uart6: serial@481aa000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart6"; clock-frequency = <48000000>; status = "disabled"; }; - i2c1: i2c@44E0B000 { + i2c1: i2c@44e0b000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; @@ -148,7 +148,7 @@ status = "disabled"; }; - i2c2: i2c@4802A000 { + i2c2: i2c@4802a000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; @@ -156,7 +156,7 @@ status = "disabled"; }; - i2c3: i2c@4819C000 { + i2c3: i2c@4819c000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 1/2] arm/dts: AM33XX: Convert all hex numbers to lower-case @ 2012-08-27 12:31 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-arm-kernel To make it consistent, convert all hex number presentation to lower-case from all am33xx specific nodes. Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> Cc: Tony Lindgren <tony@atomide.com> --- arch/arm/boot/dts/am335x-bone.dts | 2 +- arch/arm/boot/dts/am335x-evm.dts | 2 +- arch/arm/boot/dts/am33xx.dtsi | 20 ++++++++++---------- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch/arm/boot/dts/am335x-bone.dts b/arch/arm/boot/dts/am335x-bone.dts index a7906cb..c64f7fa 100644 --- a/arch/arm/boot/dts/am335x-bone.dts +++ b/arch/arm/boot/dts/am335x-bone.dts @@ -19,7 +19,7 @@ }; ocp { - uart1: serial at 44E09000 { + uart1: serial at 44e09000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts index 5dd8a6b..8963f20 100644 --- a/arch/arm/boot/dts/am335x-evm.dts +++ b/arch/arm/boot/dts/am335x-evm.dts @@ -19,7 +19,7 @@ }; ocp { - uart1: serial at 44E09000 { + uart1: serial at 44e09000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index dde76f7..be43511 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -71,7 +71,7 @@ #interrupt-cells = <1>; }; - gpio2: gpio at 4804C000 { + gpio2: gpio at 4804c000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio2"; gpio-controller; @@ -80,7 +80,7 @@ #interrupt-cells = <1>; }; - gpio3: gpio at 481AC000 { + gpio3: gpio at 481ac000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio3"; gpio-controller; @@ -89,7 +89,7 @@ #interrupt-cells = <1>; }; - gpio4: gpio at 481AE000 { + gpio4: gpio at 481ae000 { compatible = "ti,omap4-gpio"; ti,hwmods = "gpio4"; gpio-controller; @@ -98,7 +98,7 @@ #interrupt-cells = <1>; }; - uart1: serial at 44E09000 { + uart1: serial at 44e09000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart1"; clock-frequency = <48000000>; @@ -119,28 +119,28 @@ status = "disabled"; }; - uart4: serial at 481A6000 { + uart4: serial at 481a6000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart4"; clock-frequency = <48000000>; status = "disabled"; }; - uart5: serial at 481A8000 { + uart5: serial at 481a8000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart5"; clock-frequency = <48000000>; status = "disabled"; }; - uart6: serial at 481AA000 { + uart6: serial at 481aa000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart6"; clock-frequency = <48000000>; status = "disabled"; }; - i2c1: i2c at 44E0B000 { + i2c1: i2c at 44e0b000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; @@ -148,7 +148,7 @@ status = "disabled"; }; - i2c2: i2c at 4802A000 { + i2c2: i2c at 4802a000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; @@ -156,7 +156,7 @@ status = "disabled"; }; - i2c3: i2c at 4819C000 { + i2c3: i2c at 4819c000 { compatible = "ti,omap4-i2c"; #address-cells = <1>; #size-cells = <0>; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] arm/dts: AM33XX: Specify reg and interrupt property for all nodes 2012-08-27 12:31 ` Vaibhav Hiremath (?) @ 2012-08-27 12:31 ` Vaibhav Hiremath -1 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-omap Cc: Benoit Cousson, Tony Lindgren, devicetree-discuss, linux-kernel, Vaibhav Hiremath, linux-arm-kernel The device/node resources (like, IORESOURCE_MEM and IORESOURCE_IRQ) are overwritten by hwmod resources, due to all known reasons but that should not be the reason for not providing all the information in the DTS blob. Ideally we should use DTS resource and use HWMOD framework wherever required and for only specific things. Newer platforms like, OMAP5 and AM33XX, we only support DT boot mode, so this patch is preparation for the future where we supposed to get rid of hwmod dependency anyway. Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> Cc: Tony Lindgren <tony@atomide.com> Cc: Benoit Cousson <b-cousson@ti.com> --- arch/arm/boot/dts/am33xx.dtsi | 42 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 42 insertions(+), 0 deletions(-) diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index be43511..f0642e1 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -69,6 +69,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x44e07000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <96>; }; gpio2: gpio@4804c000 { @@ -78,6 +81,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x4804c000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <98>; }; gpio3: gpio@481ac000 { @@ -87,6 +93,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x481ac000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <32>; }; gpio4: gpio@481ae000 { @@ -96,12 +105,18 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x481ae000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <62>; }; uart1: serial@44e09000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart1"; clock-frequency = <48000000>; + reg = <0x44e09000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <72>; status = "disabled"; }; @@ -109,6 +124,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart2"; clock-frequency = <48000000>; + reg = <0x48022000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <73>; status = "disabled"; }; @@ -116,6 +134,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart3"; clock-frequency = <48000000>; + reg = <0x48024000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <74>; status = "disabled"; }; @@ -123,6 +144,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart4"; clock-frequency = <48000000>; + reg = <0x481a6000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <44>; status = "disabled"; }; @@ -130,6 +154,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart5"; clock-frequency = <48000000>; + reg = <0x481a8000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <45>; status = "disabled"; }; @@ -137,6 +164,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart6"; clock-frequency = <48000000>; + reg = <0x481aa000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <46>; status = "disabled"; }; @@ -145,6 +175,9 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c1"; + reg = <0x44e0b000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <70>; status = "disabled"; }; @@ -153,6 +186,9 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c2"; + reg = <0x4802a000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <71>; status = "disabled"; }; @@ -161,12 +197,18 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c3"; + reg = <0x4819c000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <30>; status = "disabled"; }; wdt2: wdt@44e35000 { compatible = "ti,omap3-wdt"; ti,hwmods = "wd_timer2"; + reg = <0x44e35000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <91>; }; }; }; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] arm/dts: AM33XX: Specify reg and interrupt property for all nodes @ 2012-08-27 12:31 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-omap Cc: linux-arm-kernel, devicetree-discuss, linux-kernel, Vaibhav Hiremath, Tony Lindgren, Benoit Cousson The device/node resources (like, IORESOURCE_MEM and IORESOURCE_IRQ) are overwritten by hwmod resources, due to all known reasons but that should not be the reason for not providing all the information in the DTS blob. Ideally we should use DTS resource and use HWMOD framework wherever required and for only specific things. Newer platforms like, OMAP5 and AM33XX, we only support DT boot mode, so this patch is preparation for the future where we supposed to get rid of hwmod dependency anyway. Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> Cc: Tony Lindgren <tony@atomide.com> Cc: Benoit Cousson <b-cousson@ti.com> --- arch/arm/boot/dts/am33xx.dtsi | 42 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 42 insertions(+), 0 deletions(-) diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index be43511..f0642e1 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -69,6 +69,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x44e07000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <96>; }; gpio2: gpio@4804c000 { @@ -78,6 +81,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x4804c000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <98>; }; gpio3: gpio@481ac000 { @@ -87,6 +93,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x481ac000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <32>; }; gpio4: gpio@481ae000 { @@ -96,12 +105,18 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x481ae000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <62>; }; uart1: serial@44e09000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart1"; clock-frequency = <48000000>; + reg = <0x44e09000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <72>; status = "disabled"; }; @@ -109,6 +124,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart2"; clock-frequency = <48000000>; + reg = <0x48022000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <73>; status = "disabled"; }; @@ -116,6 +134,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart3"; clock-frequency = <48000000>; + reg = <0x48024000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <74>; status = "disabled"; }; @@ -123,6 +144,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart4"; clock-frequency = <48000000>; + reg = <0x481a6000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <44>; status = "disabled"; }; @@ -130,6 +154,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart5"; clock-frequency = <48000000>; + reg = <0x481a8000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <45>; status = "disabled"; }; @@ -137,6 +164,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart6"; clock-frequency = <48000000>; + reg = <0x481aa000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <46>; status = "disabled"; }; @@ -145,6 +175,9 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c1"; + reg = <0x44e0b000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <70>; status = "disabled"; }; @@ -153,6 +186,9 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c2"; + reg = <0x4802a000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <71>; status = "disabled"; }; @@ -161,12 +197,18 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c3"; + reg = <0x4819c000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <30>; status = "disabled"; }; wdt2: wdt@44e35000 { compatible = "ti,omap3-wdt"; ti,hwmods = "wd_timer2"; + reg = <0x44e35000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <91>; }; }; }; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] arm/dts: AM33XX: Specify reg and interrupt property for all nodes @ 2012-08-27 12:31 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-08-27 12:31 UTC (permalink / raw) To: linux-arm-kernel The device/node resources (like, IORESOURCE_MEM and IORESOURCE_IRQ) are overwritten by hwmod resources, due to all known reasons but that should not be the reason for not providing all the information in the DTS blob. Ideally we should use DTS resource and use HWMOD framework wherever required and for only specific things. Newer platforms like, OMAP5 and AM33XX, we only support DT boot mode, so this patch is preparation for the future where we supposed to get rid of hwmod dependency anyway. Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> Cc: Tony Lindgren <tony@atomide.com> Cc: Benoit Cousson <b-cousson@ti.com> --- arch/arm/boot/dts/am33xx.dtsi | 42 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 42 insertions(+), 0 deletions(-) diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index be43511..f0642e1 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -69,6 +69,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x44e07000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <96>; }; gpio2: gpio at 4804c000 { @@ -78,6 +81,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x4804c000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <98>; }; gpio3: gpio at 481ac000 { @@ -87,6 +93,9 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x481ac000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <32>; }; gpio4: gpio at 481ae000 { @@ -96,12 +105,18 @@ #gpio-cells = <2>; interrupt-controller; #interrupt-cells = <1>; + reg = <0x481ae000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <62>; }; uart1: serial at 44e09000 { compatible = "ti,omap3-uart"; ti,hwmods = "uart1"; clock-frequency = <48000000>; + reg = <0x44e09000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <72>; status = "disabled"; }; @@ -109,6 +124,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart2"; clock-frequency = <48000000>; + reg = <0x48022000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <73>; status = "disabled"; }; @@ -116,6 +134,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart3"; clock-frequency = <48000000>; + reg = <0x48024000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <74>; status = "disabled"; }; @@ -123,6 +144,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart4"; clock-frequency = <48000000>; + reg = <0x481a6000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <44>; status = "disabled"; }; @@ -130,6 +154,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart5"; clock-frequency = <48000000>; + reg = <0x481a8000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <45>; status = "disabled"; }; @@ -137,6 +164,9 @@ compatible = "ti,omap3-uart"; ti,hwmods = "uart6"; clock-frequency = <48000000>; + reg = <0x481aa000 0x2000>; + interrupt-parent = <&intc>; + interrupts = <46>; status = "disabled"; }; @@ -145,6 +175,9 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c1"; + reg = <0x44e0b000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <70>; status = "disabled"; }; @@ -153,6 +186,9 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c2"; + reg = <0x4802a000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <71>; status = "disabled"; }; @@ -161,12 +197,18 @@ #address-cells = <1>; #size-cells = <0>; ti,hwmods = "i2c3"; + reg = <0x4819c000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <30>; status = "disabled"; }; wdt2: wdt at 44e35000 { compatible = "ti,omap3-wdt"; ti,hwmods = "wd_timer2"; + reg = <0x44e35000 0x1000>; + interrupt-parent = <&intc>; + interrupts = <91>; }; }; }; -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes 2012-08-27 12:31 ` Vaibhav Hiremath (?) @ 2012-09-06 13:57 ` Benoit Cousson -1 siblings, 0 replies; 12+ messages in thread From: Benoit Cousson @ 2012-09-06 13:57 UTC (permalink / raw) To: Vaibhav Hiremath Cc: linux-omap, devicetree-discuss, linux-kernel, linux-arm-kernel Hi Vaihbav, On 08/27/2012 02:31 PM, Vaibhav Hiremath wrote: > This series is trivial patch-series and should be considered as > preparation for the future where we supposed to get rid of > hwmod dependency. > > 1/2: Converts all hex numbers to lowercase, fixing inconsistency > 2/2: Add reg and interrupt property to all device/module nodes inside > DTS files. > > Although currently hwmod overwrites resources, I have validated this > patch series by changing the omap_device layer to respect DT resources > and boot Tested on BeagleBone platform. > I will be submitting the changes for omap_device layer as well, still > needs to fix on certain things. > > Vaibhav Hiremath (2): > arm/dts: AM33XX: Convert all hex numbers to lower-case > arm/dts: AM33XX: Specify reg and interrupt property for all nodes > > arch/arm/boot/dts/am335x-bone.dts | 2 +- > arch/arm/boot/dts/am335x-evm.dts | 2 +- > arch/arm/boot/dts/am33xx.dtsi | 62 +++++++++++++++++++++++++++++++------ > 3 files changed, 54 insertions(+), 12 deletions(-) I'll queue that along with the other DT stuff for 3.7. Thanks, Benoit ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes @ 2012-09-06 13:57 ` Benoit Cousson 0 siblings, 0 replies; 12+ messages in thread From: Benoit Cousson @ 2012-09-06 13:57 UTC (permalink / raw) To: Vaibhav Hiremath Cc: linux-omap, devicetree-discuss, linux-kernel, linux-arm-kernel Hi Vaihbav, On 08/27/2012 02:31 PM, Vaibhav Hiremath wrote: > This series is trivial patch-series and should be considered as > preparation for the future where we supposed to get rid of > hwmod dependency. > > 1/2: Converts all hex numbers to lowercase, fixing inconsistency > 2/2: Add reg and interrupt property to all device/module nodes inside > DTS files. > > Although currently hwmod overwrites resources, I have validated this > patch series by changing the omap_device layer to respect DT resources > and boot Tested on BeagleBone platform. > I will be submitting the changes for omap_device layer as well, still > needs to fix on certain things. > > Vaibhav Hiremath (2): > arm/dts: AM33XX: Convert all hex numbers to lower-case > arm/dts: AM33XX: Specify reg and interrupt property for all nodes > > arch/arm/boot/dts/am335x-bone.dts | 2 +- > arch/arm/boot/dts/am335x-evm.dts | 2 +- > arch/arm/boot/dts/am33xx.dtsi | 62 +++++++++++++++++++++++++++++++------ > 3 files changed, 54 insertions(+), 12 deletions(-) I'll queue that along with the other DT stuff for 3.7. Thanks, Benoit ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes @ 2012-09-06 13:57 ` Benoit Cousson 0 siblings, 0 replies; 12+ messages in thread From: Benoit Cousson @ 2012-09-06 13:57 UTC (permalink / raw) To: linux-arm-kernel Hi Vaihbav, On 08/27/2012 02:31 PM, Vaibhav Hiremath wrote: > This series is trivial patch-series and should be considered as > preparation for the future where we supposed to get rid of > hwmod dependency. > > 1/2: Converts all hex numbers to lowercase, fixing inconsistency > 2/2: Add reg and interrupt property to all device/module nodes inside > DTS files. > > Although currently hwmod overwrites resources, I have validated this > patch series by changing the omap_device layer to respect DT resources > and boot Tested on BeagleBone platform. > I will be submitting the changes for omap_device layer as well, still > needs to fix on certain things. > > Vaibhav Hiremath (2): > arm/dts: AM33XX: Convert all hex numbers to lower-case > arm/dts: AM33XX: Specify reg and interrupt property for all nodes > > arch/arm/boot/dts/am335x-bone.dts | 2 +- > arch/arm/boot/dts/am335x-evm.dts | 2 +- > arch/arm/boot/dts/am33xx.dtsi | 62 +++++++++++++++++++++++++++++++------ > 3 files changed, 54 insertions(+), 12 deletions(-) I'll queue that along with the other DT stuff for 3.7. Thanks, Benoit ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2012-09-06 13:58 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-27 12:31 [PATCH 0/2] arm/dts: AM33XX: Add reg and interrupt property for all nodes Vaibhav Hiremath
2012-08-27 12:31 ` Vaibhav Hiremath
2012-08-27 12:31 ` Vaibhav Hiremath
[not found] ` <1346070714-6122-1-git-send-email-hvaibhav-l0cyMroinI0@public.gmane.org>
2012-08-27 12:31 ` [PATCH 1/2] arm/dts: AM33XX: Convert all hex numbers to lower-case Vaibhav Hiremath
2012-08-27 12:31 ` Vaibhav Hiremath
2012-08-27 12:31 ` Vaibhav Hiremath
2012-08-27 12:31 ` [PATCH 2/2] arm/dts: AM33XX: Specify reg and interrupt property for all nodes Vaibhav Hiremath
2012-08-27 12:31 ` Vaibhav Hiremath
2012-08-27 12:31 ` Vaibhav Hiremath
2012-09-06 13:57 ` [PATCH 0/2] arm/dts: AM33XX: Add " Benoit Cousson
2012-09-06 13:57 ` Benoit Cousson
2012-09-06 13:57 ` Benoit Cousson
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.