Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/hrefv60plus.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/hrefv60plus.dts b/arch/arm/boot/dts/hrefv60plus.dts
index 3d580d6..46fd8af 100644
--- a/arch/arm/boot/dts/hrefv60plus.dts
+++ b/arch/arm/boot/dts/hrefv60plus.dts
@@ -26,7 +26,7 @@
soc {
i2c@80110000 {
- bu21013_tp@0x5c {
+ bu21013_tp@5c {
reset-gpio = <&gpio4 15 0x4>;
};
};
--
1.8.1.2
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/ste-nomadik-stn8815.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/ste-nomadik-stn8815.dtsi b/arch/arm/boot/dts/ste-nomadik-stn8815.dtsi
index a3acfa7..c2ee51e 100644
--- a/arch/arm/boot/dts/ste-nomadik-stn8815.dtsi
+++ b/arch/arm/boot/dts/ste-nomadik-stn8815.dtsi
@@ -742,14 +742,14 @@
#size-cells = <1>;
ranges;
- vica: intc@0x10140000 {
+ vica: intc@10140000 {
compatible = "arm,versatile-vic";
interrupt-controller;
#interrupt-cells = <1>;
reg = <0x10140000 0x20>;
};
- vicb: intc@0x10140020 {
+ vicb: intc@10140020 {
compatible = "arm,versatile-vic";
interrupt-controller;
#interrupt-cells = <1>;
--
1.8.1.2
Cc: Christian Daudt <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/bcm11351-brt.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts
index 67ec524..fdace5d 100644
--- a/arch/arm/boot/dts/bcm11351-brt.dts
+++ b/arch/arm/boot/dts/bcm11351-brt.dts
@@ -27,18 +27,18 @@
status = "okay";
};
- sdio0: sdio@0x3f180000 {
+ sdio0: sdio@3f180000 {
max-frequency = <48000000>;
status = "okay";
};
- sdio1: sdio@0x3f190000 {
+ sdio1: sdio@3f190000 {
non-removable;
max-frequency = <48000000>;
status = "okay";
};
- sdio3: sdio@0x3f1b0000 {
+ sdio3: sdio@3f1b0000 {
max-frequency = <48000000>;
status = "okay";
};
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap2420-h4.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/omap2420-h4.dts b/arch/arm/boot/dts/omap2420-h4.dts
index 224c08f..34cdecb 100644
--- a/arch/arm/boot/dts/omap2420-h4.dts
+++ b/arch/arm/boot/dts/omap2420-h4.dts
@@ -50,15 +50,15 @@
label = "bootloader";
reg = <0 0x20000>;
};
- partition@0x20000 {
+ partition@20000 {
label = "params";
reg = <0x20000 0x20000>;
};
- partition@0x40000 {
+ partition@40000 {
label = "kernel";
reg = <0x40000 0x200000>;
};
- partition@0x240000 {
+ partition@240000 {
label = "file-system";
reg = <0x240000 0x3dc0000>;
};
--
1.8.1.2
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/exynos4x12.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos4x12.dtsi b/arch/arm/boot/dts/exynos4x12.dtsi
index 01da194..3aa2f06 100644
--- a/arch/arm/boot/dts/exynos4x12.dtsi
+++ b/arch/arm/boot/dts/exynos4x12.dtsi
@@ -28,7 +28,7 @@
pinctrl3 = &pinctrl_3;
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4412-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap3.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
index 7d95cda..16420ae 100644
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -111,7 +111,7 @@
pinctrl-single,function-mask = <0x7f1f>;
};
- omap3_pmx_wkup: pinmux@0x48002a00 {
+ omap3_pmx_wkup: pinmux@48002a00 {
compatible = "ti,omap3-padconf", "pinctrl-single";
reg = <0x48002a00 0x5c>;
#address-cells = <1>;
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap3430-sdp.dts | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index e2249bc..281914e 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -84,15 +84,15 @@
label = "bootloader-nor";
reg = <0 0x40000>;
};
- partition@0x40000 {
+ partition@40000 {
label = "params-nor";
reg = <0x40000 0x40000>;
};
- partition@0x80000 {
+ partition@80000 {
label = "kernel-nor";
reg = <0x80000 0x200000>;
};
- partition@0x280000 {
+ partition@280000 {
label = "filesystem-nor";
reg = <0x240000 0x7d80000>;
};
@@ -125,19 +125,19 @@
label = "xloader-nand";
reg = <0 0x80000>;
};
- partition@0x80000 {
+ partition@80000 {
label = "bootloader-nand";
reg = <0x80000 0x140000>;
};
- partition@0x1c0000 {
+ partition@1c0000 {
label = "params-nand";
reg = <0x1c0000 0xc0000>;
};
- partition@0x280000 {
+ partition@280000 {
label = "kernel-nand";
reg = <0x280000 0x500000>;
};
- partition@0x780000 {
+ partition@780000 {
label = "filesystem-nand";
reg = <0x780000 0x7880000>;
};
@@ -170,19 +170,19 @@
label = "xloader-onenand";
reg = <0 0x80000>;
};
- partition@0x80000 {
+ partition@80000 {
label = "bootloader-onenand";
reg = <0x80000 0x40000>;
};
- partition@0xc0000 {
+ partition@c0000 {
label = "params-onenand";
reg = <0xc0000 0x20000>;
};
- partition@0xe0000 {
+ partition@e0000 {
label = "kernel-onenand";
reg = <0xe0000 0x200000>;
};
- partition@0x2e0000 {
+ partition@2e0000 {
label = "filesystem-onenand";
reg = <0x2e0000 0xfd20000>;
};
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap4.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
index 22d9f2b..45708e1 100644
--- a/arch/arm/boot/dts/omap4.dtsi
+++ b/arch/arm/boot/dts/omap4.dtsi
@@ -56,7 +56,7 @@
cache-level = <2>;
};
- local-timer@0x48240600 {
+ local-timer@48240600 {
compatible = "arm,cortex-a9-twd-timer";
reg = <0x48240600 0x20>;
interrupts = <GIC_PPI 13 (GIC_CPU_MASK_RAW(3) | IRQ_TYPE_LEVEL_HIGH)>;
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap5.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
index e643620..be58604 100644
--- a/arch/arm/boot/dts/omap5.dtsi
+++ b/arch/arm/boot/dts/omap5.dtsi
@@ -604,7 +604,7 @@
ti,hwmods = "wd_timer2";
};
- emif1: emif@0x4c000000 {
+ emif1: emif@4c000000 {
compatible = "ti,emif-4d5";
ti,hwmods = "emif1";
phy-type = <2>; /* DDR PHY type: Intelli PHY */
@@ -615,7 +615,7 @@
hw-caps-temp-alert;
};
- emif2: emif@0x4d000000 {
+ emif2: emif@4d000000 {
compatible = "ti,emif-4d5";
ti,hwmods = "emif2";
phy-type = <2>; /* DDR PHY type: Intelli PHY */
--
1.8.1.2
Cc: Simon Horman <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/r8a7779.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
index 7f146c6..86bd487 100644
--- a/arch/arm/boot/dts/r8a7779.dtsi
+++ b/arch/arm/boot/dts/r8a7779.dtsi
@@ -65,7 +65,7 @@
sense-bitfield-width = <2>;
};
- i2c0: i2c@0xffc70000 {
+ i2c0: i2c@ffc70000 {
#address-cells = <1>;
#size-cells = <0>;
compatible = "renesas,rmobile-iic";
@@ -74,7 +74,7 @@
interrupts = <0 79 0x4>;
};
- i2c1: i2c@0xffc71000 {
+ i2c1: i2c@ffc71000 {
#address-cells = <1>;
#size-cells = <0>;
compatible = "renesas,rmobile-iic";
@@ -83,7 +83,7 @@
interrupts = <0 82 0x4>;
};
- i2c2: i2c@0xffc72000 {
+ i2c2: i2c@ffc72000 {
#address-cells = <1>;
#size-cells = <0>;
compatible = "renesas,rmobile-iic";
@@ -92,7 +92,7 @@
interrupts = <0 80 0x4>;
};
- i2c3: i2c@0xffc73000 {
+ i2c3: i2c@ffc73000 {
#address-cells = <1>;
#size-cells = <0>;
compatible = "renesas,rmobile-iic";
--
1.8.1.2
Cc: Viresh Kumar <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/spear300.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/spear300.dtsi b/arch/arm/boot/dts/spear300.dtsi
index f79b3df..8970a4a 100644
--- a/arch/arm/boot/dts/spear300.dtsi
+++ b/arch/arm/boot/dts/spear300.dtsi
@@ -52,7 +52,7 @@
status = "disabled";
};
- shirq: interrupt-controller@0x50000000 {
+ shirq: interrupt-controller@50000000 {
compatible = "st,spear300-shirq";
reg = <0x50000000 0x1000>;
interrupts = <28>;
--
1.8.1.2
Cc: Viresh Kumar <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/spear310.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/spear310.dtsi b/arch/arm/boot/dts/spear310.dtsi
index 9537208..dc263e4 100644
--- a/arch/arm/boot/dts/spear310.dtsi
+++ b/arch/arm/boot/dts/spear310.dtsi
@@ -40,7 +40,7 @@
status = "disabled";
};
- shirq: interrupt-controller@0xb4000000 {
+ shirq: interrupt-controller@b4000000 {
compatible = "st,spear310-shirq";
reg = <0xb4000000 0x1000>;
interrupts = <28 29 30 1>;
--
1.8.1.2
Cc: Viresh Kumar <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/spear320.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/spear320.dtsi b/arch/arm/boot/dts/spear320.dtsi
index ffea342..a82d76e5 100644
--- a/arch/arm/boot/dts/spear320.dtsi
+++ b/arch/arm/boot/dts/spear320.dtsi
@@ -55,7 +55,7 @@
status = "disabled";
};
- shirq: interrupt-controller@0xb3000000 {
+ shirq: interrupt-controller@b3000000 {
compatible = "st,spear320-shirq";
reg = <0xb3000000 0x1000>;
interrupts = <30 28 29 1>;
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap3-igep0020.dts | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/omap3-igep0020.dts b/arch/arm/boot/dts/omap3-igep0020.dts
index e8c4828..f7a8122 100644
--- a/arch/arm/boot/dts/omap3-igep0020.dts
+++ b/arch/arm/boot/dts/omap3-igep0020.dts
@@ -99,19 +99,19 @@
label = "SPL";
reg = <0 0x100000>;
};
- partition@0x80000 {
+ partition@80000 {
label = "U-Boot";
reg = <0x100000 0x180000>;
};
- partition@0x1c0000 {
+ partition@1c0000 {
label = "Environment";
reg = <0x280000 0x100000>;
};
- partition@0x280000 {
+ partition@280000 {
label = "Kernel";
reg = <0x380000 0x300000>;
};
- partition@0x780000 {
+ partition@780000 {
label = "Filesystem";
reg = <0x680000 0x1f980000>;
};
--
1.8.1.2
Cc: Benoît Cousson <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/omap3-igep0030.dts | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/omap3-igep0030.dts b/arch/arm/boot/dts/omap3-igep0030.dts
index 644d0538..f628200 100644
--- a/arch/arm/boot/dts/omap3-igep0030.dts
+++ b/arch/arm/boot/dts/omap3-igep0030.dts
@@ -74,19 +74,19 @@
label = "SPL";
reg = <0 0x100000>;
};
- partition@0x80000 {
+ partition@80000 {
label = "U-Boot";
reg = <0x100000 0x180000>;
};
- partition@0x1c0000 {
+ partition@1c0000 {
label = "Environment";
reg = <0x280000 0x100000>;
};
- partition@0x280000 {
+ partition@280000 {
label = "Kernel";
reg = <0x380000 0x300000>;
};
- partition@0x780000 {
+ partition@780000 {
label = "Filesystem";
reg = <0x680000 0x1f980000>;
};
--
1.8.1.2
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/exynos5440.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
index ff7f5d8..0cedba4 100644
--- a/arch/arm/boot/dts/exynos5440.dtsi
+++ b/arch/arm/boot/dts/exynos5440.dtsi
@@ -20,7 +20,7 @@
spi0 = &spi_0;
};
- clock: clock-controller@0x160000 {
+ clock: clock-controller@160000 {
compatible = "samsung,exynos5440-clock";
reg = <0x160000 0x1000>;
#clock-cells = <1>;
--
1.8.1.2
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/exynos4210.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b7f358a..53e2527 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -72,7 +72,7 @@
};
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4210-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
--
1.8.1.2
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/exynos5420.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 8c54c4b..9e90d1e 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -59,7 +59,7 @@
};
};
- clock: clock-controller@0x10010000 {
+ clock: clock-controller@10010000 {
compatible = "samsung,exynos5420-clock";
reg = <0x10010000 0x30000>;
#clock-cells = <1>;
--
1.8.1.2
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/exynos5250.dtsi | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index ef57277..24c0888 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -68,17 +68,17 @@
};
};
- pd_gsc: gsc-power-domain@0x10044000 {
+ pd_gsc: gsc-power-domain@10044000 {
compatible = "samsung,exynos4210-pd";
reg = <0x10044000 0x20>;
};
- pd_mfc: mfc-power-domain@0x10044040 {
+ pd_mfc: mfc-power-domain@10044040 {
compatible = "samsung,exynos4210-pd";
reg = <0x10044040 0x20>;
};
- clock: clock-controller@0x10010000 {
+ clock: clock-controller@10010000 {
compatible = "samsung,exynos5250-clock";
reg = <0x10010000 0x30000>;
#clock-cells = <1>;
@@ -562,7 +562,7 @@
};
};
- gsc_0: gsc@0x13e00000 {
+ gsc_0: gsc@13e00000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e00000 0x1000>;
interrupts = <0 85 0>;
@@ -571,7 +571,7 @@
clock-names = "gscl";
};
- gsc_1: gsc@0x13e10000 {
+ gsc_1: gsc@13e10000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e10000 0x1000>;
interrupts = <0 86 0>;
@@ -580,7 +580,7 @@
clock-names = "gscl";
};
- gsc_2: gsc@0x13e20000 {
+ gsc_2: gsc@13e20000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e20000 0x1000>;
interrupts = <0 87 0>;
@@ -589,7 +589,7 @@
clock-names = "gscl";
};
- gsc_3: gsc@0x13e30000 {
+ gsc_3: gsc@13e30000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e30000 0x1000>;
interrupts = <0 88 0>;
--
1.8.1.2
Cc: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
Cc: Nicolas Ferre <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/animeo_ip.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/animeo_ip.dts b/arch/arm/boot/dts/animeo_ip.dts
index 3a1de9e..4cb0342 100644
--- a/arch/arm/boot/dts/animeo_ip.dts
+++ b/arch/arm/boot/dts/animeo_ip.dts
@@ -105,7 +105,7 @@
reg = <0x48000 0x8000>;
};
- user_block@0x50000 {
+ user_block@50000 {
label = "user_block";
reg = <0x50000 0xb0000>;
};
--
1.8.1.2
Cc: Christian Daudt <[email protected]>
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/bcm11351.dtsi | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi
index c0cdf66..db5873a 100644
--- a/arch/arm/boot/dts/bcm11351.dtsi
+++ b/arch/arm/boot/dts/bcm11351.dtsi
@@ -34,7 +34,7 @@
<0x3ff00100 0x100>;
};
- smc@0x3404c000 {
+ smc@3404c000 {
compatible = "bcm,bcm11351-smc", "bcm,kona-smc";
reg = <0x3404c000 0x400>; /* 1 KiB in SRAM */
};
@@ -63,28 +63,28 @@
clock-frequency = <32768>;
};
- sdio0: sdio@0x3f180000 {
+ sdio0: sdio@3f180000 {
compatible = "bcm,kona-sdhci";
reg = <0x3f180000 0x10000>;
interrupts = <0x0 77 0x4>;
status = "disabled";
};
- sdio1: sdio@0x3f190000 {
+ sdio1: sdio@3f190000 {
compatible = "bcm,kona-sdhci";
reg = <0x3f190000 0x10000>;
interrupts = <0x0 76 0x4>;
status = "disabled";
};
- sdio2: sdio@0x3f1a0000 {
+ sdio2: sdio@3f1a0000 {
compatible = "bcm,kona-sdhci";
reg = <0x3f1a0000 0x10000>;
interrupts = <0x0 74 0x4>;
status = "disabled";
};
- sdio3: sdio@0x3f1b0000 {
+ sdio3: sdio@3f1b0000 {
compatible = "bcm,kona-sdhci";
reg = <0x3f1b0000 0x10000>;
interrupts = <0x0 73 0x4>;
--
1.8.1.2
Signed-off-by: Lee Jones <[email protected]>
---
arch/arm/boot/dts/ste-u300.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/ste-u300.dts b/arch/arm/boot/dts/ste-u300.dts
index 8a1032c..a9da480 100644
--- a/arch/arm/boot/dts/ste-u300.dts
+++ b/arch/arm/boot/dts/ste-u300.dts
@@ -307,7 +307,7 @@
clocks = <&i2c0_clk>;
#address-cells = <1>;
#size-cells = <0>;
- ab3100: ab3100@0x48 {
+ ab3100: ab3100@48 {
compatible = "stericsson,ab3100";
reg = <0x48>;
interrupt-parent = <&vica>;
@@ -385,10 +385,10 @@
clocks = <&i2c1_clk>;
#address-cells = <1>;
#size-cells = <0>;
- fwcam0: fwcam@0x10 {
+ fwcam0: fwcam@10 {
reg = <0x10>;
};
- fwcam1: fwcam@0x5d {
+ fwcam1: fwcam@5d {
reg = <0x5d>;
};
};
--
1.8.1.2
On 22 July 2013 16:22, Lee Jones <[email protected]> wrote:
> Cc: Viresh Kumar <[email protected]>
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/spear300.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
For all three SPEAr Patches.
Acked-by: Viresh Kumar <[email protected]>
On 22/07/13 11:52, Lee Jones wrote:
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/hrefv60plus.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/hrefv60plus.dts b/arch/arm/boot/dts/hrefv60plus.dts
> index 3d580d6..46fd8af 100644
> --- a/arch/arm/boot/dts/hrefv60plus.dts
> +++ b/arch/arm/boot/dts/hrefv60plus.dts
> @@ -26,7 +26,7 @@
>
> soc {
> i2c@80110000 {
> - bu21013_tp@0x5c {
> + bu21013_tp@5c {
Just a query ... why this one .... ^^^ ...
> reset-gpio = <&gpio4 15 0x4>;
... and not this one .................................. ^^^ ?
> };
> };
>
The same for all the other patches in this series.
If this is by design, it might be worth mentioning why some are
removed and not others.
Mark J.
On 13-07-22 03:52 AM, Lee Jones wrote:
> Cc: Christian Daudt <[email protected]>
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/bcm11351-brt.dts | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts
> index 67ec524..fdace5d 100644
> --- a/arch/arm/boot/dts/bcm11351-brt.dts
> +++ b/arch/arm/boot/dts/bcm11351-brt.dts
> @@ -27,18 +27,18 @@
> status = "okay";
> };
>
> - sdio0: sdio@0x3f180000 {
> + sdio0: sdio@3f180000 {
> max-frequency = <48000000>;
> status = "okay";
> };
>
> - sdio1: sdio@0x3f190000 {
> + sdio1: sdio@3f190000 {
> non-removable;
> max-frequency = <48000000>;
> status = "okay";
> };
>
> - sdio3: sdio@0x3f1b0000 {
> + sdio3: sdio@3f1b0000 {
> max-frequency = <48000000>;
> status = "okay";
> };
Hi Lee,
Is this convention documented ? Someone called my attention to the
fact that this dts files was inconsistent in its use of 0x in names, but
I was not aware of a coding guideline for this.
Thanks,
csd
On Mon, 22 Jul 2013, Christian Daudt wrote:
> On 13-07-22 03:52 AM, Lee Jones wrote:
> >Cc: Christian Daudt <[email protected]>
> >Signed-off-by: Lee Jones <[email protected]>
> >---
> > arch/arm/boot/dts/bcm11351-brt.dts | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> >diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts
> >index 67ec524..fdace5d 100644
> >--- a/arch/arm/boot/dts/bcm11351-brt.dts
> >+++ b/arch/arm/boot/dts/bcm11351-brt.dts
> >@@ -27,18 +27,18 @@
> > status = "okay";
> > };
> >- sdio0: sdio@0x3f180000 {
> >+ sdio0: sdio@3f180000 {
> > max-frequency = <48000000>;
> > status = "okay";
> > };
> >- sdio1: sdio@0x3f190000 {
> >+ sdio1: sdio@3f190000 {
> > non-removable;
> > max-frequency = <48000000>;
> > status = "okay";
> > };
> >- sdio3: sdio@0x3f1b0000 {
> >+ sdio3: sdio@3f1b0000 {
> > max-frequency = <48000000>;
> > status = "okay";
> > };
> Hi Lee,
> Is this convention documented ? Someone called my attention to the
> fact that this dts files was inconsistent in its use of 0x in names,
> but I was not aware of a coding guideline for this.
The best two places I can think of are:
Documentation/devicetree/usage-model.txt
and
http://www.devicetree.org/Device_Tree_Usage
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On Mon, 22 Jul 2013, Mark Jackson wrote:
> On 22/07/13 11:52, Lee Jones wrote:
> > Signed-off-by: Lee Jones <[email protected]>
> > ---
> > arch/arm/boot/dts/hrefv60plus.dts | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/hrefv60plus.dts b/arch/arm/boot/dts/hrefv60plus.dts
> > index 3d580d6..46fd8af 100644
> > --- a/arch/arm/boot/dts/hrefv60plus.dts
> > +++ b/arch/arm/boot/dts/hrefv60plus.dts
> > @@ -26,7 +26,7 @@
> >
> > soc {
> > i2c@80110000 {
> > - bu21013_tp@0x5c {
> > + bu21013_tp@5c {
>
> Just a query ... why this one .... ^^^ ...
>
> > reset-gpio = <&gpio4 15 0x4>;
>
> ... and not this one .................................. ^^^ ?
>
> > };
> > };
> >
>
> The same for all the other patches in this series.
>
> If this is by design, it might be worth mentioning why some are
> removed and not others.
I can't explain why there seems to be inconsistencies between node
names and the remaining properties. The best I can do is refer you to
the documentation where you can see that this is the standard:
Documentation/devicetree/usage-model.txt
and
http://www.devicetree.org/Device_Tree_Usage
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On 22/07/13 14:41, Lee Jones wrote:
> On Mon, 22 Jul 2013, Mark Jackson wrote:
>
>> On 22/07/13 11:52, Lee Jones wrote:
>>> Signed-off-by: Lee Jones <[email protected]>
>>> ---
>>> arch/arm/boot/dts/hrefv60plus.dts | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/boot/dts/hrefv60plus.dts b/arch/arm/boot/dts/hrefv60plus.dts
>>> index 3d580d6..46fd8af 100644
>>> --- a/arch/arm/boot/dts/hrefv60plus.dts
>>> +++ b/arch/arm/boot/dts/hrefv60plus.dts
>>> @@ -26,7 +26,7 @@
>>>
>>> soc {
>>> i2c@80110000 {
>>> - bu21013_tp@0x5c {
>>> + bu21013_tp@5c {
>>
>> Just a query ... why this one .... ^^^ ...
>>
>>> reset-gpio = <&gpio4 15 0x4>;
>>
>> ... and not this one .................................. ^^^ ?
>>
>>> };
>>> };
>>>
>>
>> The same for all the other patches in this series.
>>
>> If this is by design, it might be worth mentioning why some are
>> removed and not others.
>
> I can't explain why there seems to be inconsistencies between node
> names and the remaining properties. The best I can do is refer you to
> the documentation where you can see that this is the standard:
>
> Documentation/devicetree/usage-model.txt
> and
> http://www.devicetree.org/Device_Tree_Usage
Ho, ho ... don't you just love "standards" !!
On 07/22/2013 03:38 PM, Lee Jones wrote:
> On Mon, 22 Jul 2013, Christian Daudt wrote:
>
>> On 13-07-22 03:52 AM, Lee Jones wrote:
>>> Cc: Christian Daudt <[email protected]>
>>> Signed-off-by: Lee Jones <[email protected]>
>>> ---
>>> arch/arm/boot/dts/bcm11351-brt.dts | 6 +++---
>>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts
>>> index 67ec524..fdace5d 100644
>>> --- a/arch/arm/boot/dts/bcm11351-brt.dts
>>> +++ b/arch/arm/boot/dts/bcm11351-brt.dts
>>> @@ -27,18 +27,18 @@
>>> status = "okay";
>>> };
>>> - sdio0: sdio@0x3f180000 {
>>> + sdio0: sdio@3f180000 {
>>> max-frequency = <48000000>;
>>> status = "okay";
>>> };
>>> - sdio1: sdio@0x3f190000 {
>>> + sdio1: sdio@3f190000 {
>>> non-removable;
>>> max-frequency = <48000000>;
>>> status = "okay";
>>> };
>>> - sdio3: sdio@0x3f1b0000 {
>>> + sdio3: sdio@3f1b0000 {
>>> max-frequency = <48000000>;
>>> status = "okay";
>>> };
>> Hi Lee,
>> Is this convention documented ? Someone called my attention to the
>> fact that this dts files was inconsistent in its use of 0x in names,
>> but I was not aware of a coding guideline for this.
>
> The best two places I can think of are:
>
> Documentation/devicetree/usage-model.txt
> and
> http://www.devicetree.org/Device_Tree_Usage
And even those do not stick to a single convention, but that is what can
be expected of a twiki page :-p
Regards,
Arend
On 11:52 Mon 22 Jul , Lee Jones wrote:
> Cc: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
> Cc: Nicolas Ferre <[email protected]>
Acked-by: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/animeo_ip.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/animeo_ip.dts b/arch/arm/boot/dts/animeo_ip.dts
> index 3a1de9e..4cb0342 100644
> --- a/arch/arm/boot/dts/animeo_ip.dts
> +++ b/arch/arm/boot/dts/animeo_ip.dts
> @@ -105,7 +105,7 @@
> reg = <0x48000 0x8000>;
> };
>
> - user_block@0x50000 {
> + user_block@50000 {
> label = "user_block";
> reg = <0x50000 0xb0000>;
> };
> --
> 1.8.1.2
>
On 13-07-22 06:53 AM, Arend van Spriel wrote:
> On 07/22/2013 03:38 PM, Lee Jones wrote:
>> On Mon, 22 Jul 2013, Christian Daudt wrote:
>>
>>> On 13-07-22 03:52 AM, Lee Jones wrote:
>>>> Cc: Christian Daudt <[email protected]>
>>>> Signed-off-by: Lee Jones <[email protected]>
>>>> ---
>>>> arch/arm/boot/dts/bcm11351-brt.dts | 6 +++---
>>>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/bcm11351-brt.dts
>>>> b/arch/arm/boot/dts/bcm11351-brt.dts
>>>> index 67ec524..fdace5d 100644
>>>> --- a/arch/arm/boot/dts/bcm11351-brt.dts
>>>> +++ b/arch/arm/boot/dts/bcm11351-brt.dts
>>>> @@ -27,18 +27,18 @@
>>>> status = "okay";
>>>> };
>>>> - sdio0: sdio@0x3f180000 {
>>>> + sdio0: sdio@3f180000 {
>>>> max-frequency = <48000000>;
>>>> status = "okay";
>>>> };
>>>> - sdio1: sdio@0x3f190000 {
>>>> + sdio1: sdio@3f190000 {
>>>> non-removable;
>>>> max-frequency = <48000000>;
>>>> status = "okay";
>>>> };
>>>> - sdio3: sdio@0x3f1b0000 {
>>>> + sdio3: sdio@3f1b0000 {
>>>> max-frequency = <48000000>;
>>>> status = "okay";
>>>> };
>>> Hi Lee,
>>> Is this convention documented ? Someone called my attention to the
>>> fact that this dts files was inconsistent in its use of 0x in names,
>>> but I was not aware of a coding guideline for this.
>>
>> The best two places I can think of are:
>>
>> Documentation/devicetree/usage-model.txt
>> and
>> http://www.devicetree.org/Device_Tree_Usage
>
> And even those do not stick to a single convention, but that is what
> can be expected of a twiki page :-p
[take 2 with in plain text]
Agreed. Quick search shows the following example on that page:
pci@0x10180000 {
compatible = "arm,versatile-pci-hostbridge", "pci";
reg = <0x10180000 0x1000>;
interrupts = <8 0>;
bus-ranges = <0 0>;
};
though it is the minority in the examples, ultimately having more
examples following one convention than the other does not a standard
make :) While I like this just for the consistency, it'd be nice to have
something to point people to as this being the best practice for kernel
dev. It would be good to have a DeviceTreeGuidelines along the lines of
the CodingGuidelines file in the kernel tree itself, to get some
consistency on this.
Thanks,
csd
Hi Lee,
On Monday 22 of July 2013 11:52:26 Lee Jones wrote:
> Cc: Kukjin Kim <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/exynos4210.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
> };
>
> - clock: clock-controller@0x10030000 {
> + clock: clock-controller@10030000 {
> compatible = "samsung,exynos4210-clock";
> reg = <0x10030000 0x20000>;
> #clock-cells = <1>;
This looks fine, but please fix commit message - it should be Exynos4210.
Also some explanation why this change is needed would be good, even if
it's obvious.
Best regards,
Tomasz
On 13-07-22 03:52 AM, Lee Jones wrote:
> Cc: Christian Daudt <[email protected]>
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/bcm11351-brt.dts | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts
> index 67ec524..fdace5d 100644
> --- a/arch/arm/boot/dts/bcm11351-brt.dts
> +++ b/arch/arm/boot/dts/bcm11351-brt.dts
> @@ -27,18 +27,18 @@
> status = "okay";
> };
>
> - sdio0: sdio@0x3f180000 {
> + sdio0: sdio@3f180000 {
> max-frequency = <48000000>;
> status = "okay";
> };
>
> - sdio1: sdio@0x3f190000 {
> + sdio1: sdio@3f190000 {
> non-removable;
> max-frequency = <48000000>;
> status = "okay";
> };
>
> - sdio3: sdio@0x3f1b0000 {
> + sdio3: sdio@3f1b0000 {
> max-frequency = <48000000>;
> status = "okay";
> };
Acked-by: Christian Daudt <[email protected]>
Thanks,
csd
On 13-07-22 03:52 AM, Lee Jones wrote:
> Cc: Christian Daudt <[email protected]>
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/bcm11351.dtsi | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi
> index c0cdf66..db5873a 100644
> --- a/arch/arm/boot/dts/bcm11351.dtsi
> +++ b/arch/arm/boot/dts/bcm11351.dtsi
> @@ -34,7 +34,7 @@
> <0x3ff00100 0x100>;
> };
>
> - smc@0x3404c000 {
> + smc@3404c000 {
> compatible = "bcm,bcm11351-smc", "bcm,kona-smc";
> reg = <0x3404c000 0x400>; /* 1 KiB in SRAM */
> };
> @@ -63,28 +63,28 @@
> clock-frequency = <32768>;
> };
>
> - sdio0: sdio@0x3f180000 {
> + sdio0: sdio@3f180000 {
> compatible = "bcm,kona-sdhci";
> reg = <0x3f180000 0x10000>;
> interrupts = <0x0 77 0x4>;
> status = "disabled";
> };
>
> - sdio1: sdio@0x3f190000 {
> + sdio1: sdio@3f190000 {
> compatible = "bcm,kona-sdhci";
> reg = <0x3f190000 0x10000>;
> interrupts = <0x0 76 0x4>;
> status = "disabled";
> };
>
> - sdio2: sdio@0x3f1a0000 {
> + sdio2: sdio@3f1a0000 {
> compatible = "bcm,kona-sdhci";
> reg = <0x3f1a0000 0x10000>;
> interrupts = <0x0 74 0x4>;
> status = "disabled";
> };
>
> - sdio3: sdio@0x3f1b0000 {
> + sdio3: sdio@3f1b0000 {
> compatible = "bcm,kona-sdhci";
> reg = <0x3f1b0000 0x10000>;
> interrupts = <0x0 73 0x4>;
Acked-by: Christian Daudt <[email protected]>
Thanks,
csd
On Mon, Jul 22, 2013 at 11:52:38AM +0100, Lee Jones wrote:
> Cc: Simon Horman <[email protected]>
> Signed-off-by: Lee Jones <[email protected]>
Acked-by: Simon Horman <[email protected]>
Let me know if you want me to take this through the renesas tree.
> ---
> arch/arm/boot/dts/r8a7779.dtsi | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
> index 7f146c6..86bd487 100644
> --- a/arch/arm/boot/dts/r8a7779.dtsi
> +++ b/arch/arm/boot/dts/r8a7779.dtsi
> @@ -65,7 +65,7 @@
> sense-bitfield-width = <2>;
> };
>
> - i2c0: i2c@0xffc70000 {
> + i2c0: i2c@ffc70000 {
> #address-cells = <1>;
> #size-cells = <0>;
> compatible = "renesas,rmobile-iic";
> @@ -74,7 +74,7 @@
> interrupts = <0 79 0x4>;
> };
>
> - i2c1: i2c@0xffc71000 {
> + i2c1: i2c@ffc71000 {
> #address-cells = <1>;
> #size-cells = <0>;
> compatible = "renesas,rmobile-iic";
> @@ -83,7 +83,7 @@
> interrupts = <0 82 0x4>;
> };
>
> - i2c2: i2c@0xffc72000 {
> + i2c2: i2c@ffc72000 {
> #address-cells = <1>;
> #size-cells = <0>;
> compatible = "renesas,rmobile-iic";
> @@ -92,7 +92,7 @@
> interrupts = <0 80 0x4>;
> };
>
> - i2c3: i2c@0xffc73000 {
> + i2c3: i2c@ffc73000 {
> #address-cells = <1>;
> #size-cells = <0>;
> compatible = "renesas,rmobile-iic";
> --
> 1.8.1.2
>
On Tue, 23 Jul 2013, Simon Horman wrote:
> On Mon, Jul 22, 2013 at 11:52:38AM +0100, Lee Jones wrote:
> > Cc: Simon Horman <[email protected]>
> > Signed-off-by: Lee Jones <[email protected]>
>
> Acked-by: Simon Horman <[email protected]>
>
> Let me know if you want me to take this through the renesas tree.
Thanks Simon. To save on conflicts, I think that's best.
I will take all orphaned patches or ones without maintainers associated
with them through ARM SoC, but if you can take it, please do.
> > ---
> > arch/arm/boot/dts/r8a7779.dtsi | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
> > index 7f146c6..86bd487 100644
> > --- a/arch/arm/boot/dts/r8a7779.dtsi
> > +++ b/arch/arm/boot/dts/r8a7779.dtsi
> > @@ -65,7 +65,7 @@
> > sense-bitfield-width = <2>;
> > };
> >
> > - i2c0: i2c@0xffc70000 {
> > + i2c0: i2c@ffc70000 {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > compatible = "renesas,rmobile-iic";
> > @@ -74,7 +74,7 @@
> > interrupts = <0 79 0x4>;
> > };
> >
> > - i2c1: i2c@0xffc71000 {
> > + i2c1: i2c@ffc71000 {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > compatible = "renesas,rmobile-iic";
> > @@ -83,7 +83,7 @@
> > interrupts = <0 82 0x4>;
> > };
> >
> > - i2c2: i2c@0xffc72000 {
> > + i2c2: i2c@ffc72000 {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > compatible = "renesas,rmobile-iic";
> > @@ -92,7 +92,7 @@
> > interrupts = <0 80 0x4>;
> > };
> >
> > - i2c3: i2c@0xffc73000 {
> > + i2c3: i2c@ffc73000 {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > compatible = "renesas,rmobile-iic";
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
> Acked-by: Christian Daudt <[email protected]>
Thanks Christian, for both.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On Mon, 22 Jul 2013, Tomasz Figa wrote:
> Hi Lee,
>
> On Monday 22 of July 2013 11:52:26 Lee Jones wrote:
> > Cc: Kukjin Kim <[email protected]>
> > Cc: [email protected]
> > Signed-off-by: Lee Jones <[email protected]>
> > ---
> > arch/arm/boot/dts/exynos4210.dtsi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > @@ -72,7 +72,7 @@
> > };
> > };
> >
> > - clock: clock-controller@0x10030000 {
> > + clock: clock-controller@10030000 {
> > compatible = "samsung,exynos4210-clock";
> > reg = <0x10030000 0x20000>;
> > #clock-cells = <1>;
>
> This looks fine, but please fix commit message - it should be Exynos4210.
> Also some explanation why this change is needed would be good, even if
> it's obvious.
Hi Tomasz,
I'm happy to fixup the $SUBJECT line, but do we really have to enter
an explanation if it's obvious? Seems a little belt and braces.
Kind regards,
Lee
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On Mon, 22 Jul 2013, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 11:52 Mon 22 Jul , Lee Jones wrote:
> > Cc: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
> > Cc: Nicolas Ferre <[email protected]>
>
> Acked-by: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
Thanks Jean-Christophe.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On Tuesday 23 of July 2013 09:55:52 Lee Jones wrote:
> On Mon, 22 Jul 2013, Tomasz Figa wrote:
> > Hi Lee,
> >
> > On Monday 22 of July 2013 11:52:26 Lee Jones wrote:
> > > Cc: Kukjin Kim <[email protected]>
> > > Cc: [email protected]
> > > Signed-off-by: Lee Jones <[email protected]>
> > > ---
> > >
> > > arch/arm/boot/dts/exynos4210.dtsi | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > > @@ -72,7 +72,7 @@
> > >
> > > };
> > >
> > > };
> > >
> > > - clock: clock-controller@0x10030000 {
> > > + clock: clock-controller@10030000 {
> > >
> > > compatible = "samsung,exynos4210-clock";
> > > reg = <0x10030000 0x20000>;
> > > #clock-cells = <1>;
> >
> > This looks fine, but please fix commit message - it should be
> > Exynos4210. Also some explanation why this change is needed would be
> > good, even if it's obvious.
>
> Hi Tomasz,
>
> I'm happy to fixup the $SUBJECT line, but do we really have to enter
> an explanation if it's obvious? Seems a little belt and braces.
It's obvious for us, people working with device tree, but for people that
usually don't it might not be.
Something among following lines would be fine:
This patch removes "0x" prefix from addresses in DT nodes for the sake of
consistency with other nodes and assumed convention.
Best regards,
Tomasz
On Tue, Jul 23, 2013 at 08:52:32AM +0100, Lee Jones wrote:
> On Tue, 23 Jul 2013, Simon Horman wrote:
>
> > On Mon, Jul 22, 2013 at 11:52:38AM +0100, Lee Jones wrote:
> > > Cc: Simon Horman <[email protected]>
> > > Signed-off-by: Lee Jones <[email protected]>
> >
> > Acked-by: Simon Horman <[email protected]>
> >
> > Let me know if you want me to take this through the renesas tree.
>
> Thanks Simon. To save on conflicts, I think that's best.
>
> I will take all orphaned patches or ones without maintainers associated
> with them through ARM SoC, but if you can take it, please do.
Understood.
I have queued this up for v3.12 in the fixes2-for-v3.12 branch
of my renesas tree on kernel.org.
... for the sake of consistency and assumed convention.
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b7f358a..53e2527 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -72,7 +72,7 @@
};
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4210-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
On Wednesday 24 of July 2013 13:39:31 Lee Jones wrote:
> ... for the sake of consistency and assumed convention.
>
> Cc: Kukjin Kim <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
> };
>
> - clock: clock-controller@0x10030000 {
> + clock: clock-controller@10030000 {
> compatible = "samsung,exynos4210-clock";
> reg = <0x10030000 0x20000>;
> #clock-cells = <1>;
Looks good to me, except the subject which is still wrong -
s/Exynos4110/Exynos4210/.
I guess it might be fixed when applying, though.
Best regards,
Tomasz
On Wed, 24 Jul 2013, Tomasz Figa wrote:
> On Wednesday 24 of July 2013 13:39:31 Lee Jones wrote:
> > ... for the sake of consistency and assumed convention.
> >
> > Cc: Kukjin Kim <[email protected]>
> > Cc: [email protected]
> > Signed-off-by: Lee Jones <[email protected]>
> >
> > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > @@ -72,7 +72,7 @@
> > };
> > };
> >
> > - clock: clock-controller@0x10030000 {
> > + clock: clock-controller@10030000 {
> > compatible = "samsung,exynos4210-clock";
> > reg = <0x10030000 0x20000>;
> > #clock-cells = <1>;
>
> Looks good to me, except the subject which is still wrong -
> s/Exynos4110/Exynos4210/.
Ah, I get you know. My bad, will fix.
> I guess it might be fixed when applying, though.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
... for the sake of consistency and assumed convention.
Cc: Kukjin Kim <[email protected]>
Cc: [email protected]
Signed-off-by: Lee Jones <[email protected]>
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b7f358a..53e2527 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -72,7 +72,7 @@
};
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4210-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
On Wednesday 24 of July 2013 16:09:37 Lee Jones wrote:
> ... for the sake of consistency and assumed convention.
>
> Cc: Kukjin Kim <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
> };
>
> - clock: clock-controller@0x10030000 {
> + clock: clock-controller@10030000 {
> compatible = "samsung,exynos4210-clock";
> reg = <0x10030000 0x20000>;
> #clock-cells = <1>;
Acked-by: Tomasz Figa <[email protected]>
Best regards,
Tomasz
On Wed, 24 Jul 2013, Tomasz Figa wrote:
> On Wednesday 24 of July 2013 16:09:37 Lee Jones wrote:
> > ... for the sake of consistency and assumed convention.
> >
> > Cc: Kukjin Kim <[email protected]>
> > Cc: [email protected]
> > Signed-off-by: Lee Jones <[email protected]>
> >
> > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > @@ -72,7 +72,7 @@
> > };
> > };
> >
> > - clock: clock-controller@0x10030000 {
> > + clock: clock-controller@10030000 {
> > compatible = "samsung,exynos4210-clock";
> > reg = <0x10030000 0x20000>;
> > #clock-cells = <1>;
>
> Acked-by: Tomasz Figa <[email protected]>
Thanks Tomasz.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On Mon, Jul 22, 2013 at 9:57 AM, Christian Daudt <[email protected]> wrote:
> it'd be nice to have something to point
> people to as this being the best practice for kernel dev.
Power.org's ePAPR exclusively omits the 0x in the unit-address of node names.
https://www.power.org/wp-content/uploads/2012/06/Power_ePAPR_APPROVED_v1.1.pdf
On 07/25/2013 05:43 PM, Tim Kryger wrote:
> On Mon, Jul 22, 2013 at 9:57 AM, Christian Daudt <[email protected]> wrote:
>
>> it'd be nice to have something to point
>> people to as this being the best practice for kernel dev.
>
> Power.org's ePAPR exclusively omits the 0x in the unit-address of node names.
>
> https://www.power.org/wp-content/uploads/2012/06/Power_ePAPR_APPROVED_v1.1.pdf
>
Thanks for the link. While browsing on power.org I got the impression
that I needed to have a membership to get it.
Regards,
Arend
On 07/25/13 16:52, Lee Jones wrote:
> On Wed, 24 Jul 2013, Tomasz Figa wrote:
>
>> On Wednesday 24 of July 2013 16:09:37 Lee Jones wrote:
>>> ... for the sake of consistency and assumed convention.
>>>
>>> Cc: Kukjin Kim<[email protected]>
>>> Cc: [email protected]
>>> Signed-off-by: Lee Jones<[email protected]>
>>>
>>> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
>>> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
>>> --- a/arch/arm/boot/dts/exynos4210.dtsi
>>> +++ b/arch/arm/boot/dts/exynos4210.dtsi
>>> @@ -72,7 +72,7 @@
>>> };
>>> };
>>>
>>> - clock: clock-controller@0x10030000 {
>>> + clock: clock-controller@10030000 {
>>> compatible = "samsung,exynos4210-clock";
>>> reg =<0x10030000 0x20000>;
>>> #clock-cells =<1>;
>>
>> Acked-by: Tomasz Figa<[email protected]>
>
> Thanks Tomasz.
>
Thanks, Lee Jones and Tomasz.
Applied #7 ~ #11 into the cleanup of samsung tree.
- Kukjin
On 07/25/13 00:09, Lee Jones wrote:
> ... for the sake of consistency and assumed convention.
>
> Cc: Kukjin Kim<[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones<[email protected]>
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
^^^^^^^^^^^^^^^^
> };
^^^^^^^^
>
> - clock: clock-controller@0x10030000 {
^^^^^^^^
> + clock: clock-controller@10030000 {
^^^^^^^^
> compatible = "samsung,exynos4210-clock";
^^^^^^^^^^^^^^^^
> reg =<0x10030000 0x20000>;
^^^^^^^^^^^^^^^^
> #clock-cells =<1>;
^^^^^^^^^^^^^^^^
BTW, should be tab at the mark '^' not white space :(
- Kukjin
On Tue, 06 Aug 2013, Kukjin Kim wrote:
> On 07/25/13 00:09, Lee Jones wrote:
> >... for the sake of consistency and assumed convention.
> >
> >Cc: Kukjin Kim<[email protected]>
> >Cc: [email protected]
> >Signed-off-by: Lee Jones<[email protected]>
> >
> >diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> >index b7f358a..53e2527 100644
> >--- a/arch/arm/boot/dts/exynos4210.dtsi
> >+++ b/arch/arm/boot/dts/exynos4210.dtsi
> >@@ -72,7 +72,7 @@
> > };
> ^^^^^^^^^^^^^^^^
> > };
> ^^^^^^^^
> >
> >- clock: clock-controller@0x10030000 {
> ^^^^^^^^
> >+ clock: clock-controller@10030000 {
> ^^^^^^^^
> > compatible = "samsung,exynos4210-clock";
> ^^^^^^^^^^^^^^^^
> > reg =<0x10030000 0x20000>;
> ^^^^^^^^^^^^^^^^
> > #clock-cells =<1>;
> ^^^^^^^^^^^^^^^^
>
> BTW, should be tab at the mark '^' not white space :(
Agreed, but this should be a separate patch.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On Mon, Jul 22, 2013 at 12:52 PM, Lee Jones <[email protected]> wrote:
> Signed-off-by: Lee Jones <[email protected]>
Applied.
Yours,
Linus Walleij
On Mon, Jul 22, 2013 at 12:52 PM, Lee Jones <[email protected]> wrote:
> Signed-off-by: Lee Jones <[email protected]>
Patch applied.
Yours,
Linus Walleij
On Mon, Jul 22, 2013 at 12:52 PM, Lee Jones <[email protected]> wrote:
> Signed-off-by: Lee Jones <[email protected]>
None of these patches apply since I applied your other patch series
that rename all the files ... can you respin the ux500 "0x"-strip patches
on top of the rename set? My ux500-devicetree branch can be used
as a baseline.
Yours,
Linus Walleij
On Wed, 07 Aug 2013, Linus Walleij wrote:
> On Mon, Jul 22, 2013 at 12:52 PM, Lee Jones <[email protected]> wrote:
>
> > Signed-off-by: Lee Jones <[email protected]>
>
> None of these patches apply since I applied your other patch series
> that rename all the files ... can you respin the ux500 "0x"-strip patches
> on top of the rename set? My ux500-devicetree branch can be used
> as a baseline.
I can do that. Although, would you prefer that I fixed-up my renaming
patches, then applied the 0x patches on top instead?
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Hi Lee,
I had to change some of the lines in the dts files you modified, so
I've added your mods. This patch:
https://lkml.org/lkml/2013/8/8/477
includes your changes to bcm11351.dtsi and bcm11351-brt.dts. Not sure
where your patch is at this point, but I just wanted to point that
out.
Thanks,
csd
On Thu, 08 Aug 2013, Christian Daudt wrote:
> Hi Lee,
> I had to change some of the lines in the dts files you modified, so
> I've added your mods. This patch:
> https://lkml.org/lkml/2013/8/8/477
>
> includes your changes to bcm11351.dtsi and bcm11351-brt.dts. Not sure
> where your patch is at this point, but I just wanted to point that
> out.
I haven't done anything with that patch yet, but I'd prefer they
stayed separate to be honest. If you make your changes and ensure
they're in an upstream tree (and let me know which one), I'll rebase
my patches on top of that tree.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap5.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
> index e643620..be58604 100644
> --- a/arch/arm/boot/dts/omap5.dtsi
> +++ b/arch/arm/boot/dts/omap5.dtsi
> @@ -604,7 +604,7 @@
> ti,hwmods = "wd_timer2";
> };
>
> - emif1: emif@0x4c000000 {
> + emif1: emif@4c000000 {
> compatible = "ti,emif-4d5";
> ti,hwmods = "emif1";
> phy-type = <2>; /* DDR PHY type: Intelli PHY */
> @@ -615,7 +615,7 @@
> hw-caps-temp-alert;
> };
>
> - emif2: emif@0x4d000000 {
> + emif2: emif@4d000000 {
> compatible = "ti,emif-4d5";
> ti,hwmods = "emif2";
> phy-type = <2>; /* DDR PHY type: Intelli PHY */
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
On Mon, 22 Jul 2013, Lee Jones wrote:
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap4.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
> index 22d9f2b..45708e1 100644
> --- a/arch/arm/boot/dts/omap4.dtsi
> +++ b/arch/arm/boot/dts/omap4.dtsi
> @@ -56,7 +56,7 @@
> cache-level = <2>;
> };
>
> - local-timer@0x48240600 {
> + local-timer@48240600 {
> compatible = "arm,cortex-a9-twd-timer";
> reg = <0x48240600 0x20>;
> interrupts = <GIC_PPI 13 (GIC_CPU_MASK_RAW(3) | IRQ_TYPE_LEVEL_HIGH)>;
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
On Mon, 22 Jul 2013, Lee Jones wrote:
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap3430-sdp.dts | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
> index e2249bc..281914e 100644
> --- a/arch/arm/boot/dts/omap3430-sdp.dts
> +++ b/arch/arm/boot/dts/omap3430-sdp.dts
> @@ -84,15 +84,15 @@
> label = "bootloader-nor";
> reg = <0 0x40000>;
> };
> - partition@0x40000 {
> + partition@40000 {
> label = "params-nor";
> reg = <0x40000 0x40000>;
> };
> - partition@0x80000 {
> + partition@80000 {
> label = "kernel-nor";
> reg = <0x80000 0x200000>;
> };
> - partition@0x280000 {
> + partition@280000 {
> label = "filesystem-nor";
> reg = <0x240000 0x7d80000>;
> };
> @@ -125,19 +125,19 @@
> label = "xloader-nand";
> reg = <0 0x80000>;
> };
> - partition@0x80000 {
> + partition@80000 {
> label = "bootloader-nand";
> reg = <0x80000 0x140000>;
> };
> - partition@0x1c0000 {
> + partition@1c0000 {
> label = "params-nand";
> reg = <0x1c0000 0xc0000>;
> };
> - partition@0x280000 {
> + partition@280000 {
> label = "kernel-nand";
> reg = <0x280000 0x500000>;
> };
> - partition@0x780000 {
> + partition@780000 {
> label = "filesystem-nand";
> reg = <0x780000 0x7880000>;
> };
> @@ -170,19 +170,19 @@
> label = "xloader-onenand";
> reg = <0 0x80000>;
> };
> - partition@0x80000 {
> + partition@80000 {
> label = "bootloader-onenand";
> reg = <0x80000 0x40000>;
> };
> - partition@0xc0000 {
> + partition@c0000 {
> label = "params-onenand";
> reg = <0xc0000 0x20000>;
> };
> - partition@0xe0000 {
> + partition@e0000 {
> label = "kernel-onenand";
> reg = <0xe0000 0x200000>;
> };
> - partition@0x2e0000 {
> + partition@2e0000 {
> label = "filesystem-onenand";
> reg = <0x2e0000 0xfd20000>;
> };
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
On Mon, 22 Jul 2013, Lee Jones wrote:
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap3.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
> index 7d95cda..16420ae 100644
> --- a/arch/arm/boot/dts/omap3.dtsi
> +++ b/arch/arm/boot/dts/omap3.dtsi
> @@ -111,7 +111,7 @@
> pinctrl-single,function-mask = <0x7f1f>;
> };
>
> - omap3_pmx_wkup: pinmux@0x48002a00 {
> + omap3_pmx_wkup: pinmux@48002a00 {
> compatible = "ti,omap3-padconf", "pinctrl-single";
> reg = <0x48002a00 0x5c>;
> #address-cells = <1>;
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
On Mon, 22 Jul 2013, Lee Jones wrote:
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap3-igep0030.dts | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap3-igep0030.dts b/arch/arm/boot/dts/omap3-igep0030.dts
> index 644d0538..f628200 100644
> --- a/arch/arm/boot/dts/omap3-igep0030.dts
> +++ b/arch/arm/boot/dts/omap3-igep0030.dts
> @@ -74,19 +74,19 @@
> label = "SPL";
> reg = <0 0x100000>;
> };
> - partition@0x80000 {
> + partition@80000 {
> label = "U-Boot";
> reg = <0x100000 0x180000>;
> };
> - partition@0x1c0000 {
> + partition@1c0000 {
> label = "Environment";
> reg = <0x280000 0x100000>;
> };
> - partition@0x280000 {
> + partition@280000 {
> label = "Kernel";
> reg = <0x380000 0x300000>;
> };
> - partition@0x780000 {
> + partition@780000 {
> label = "Filesystem";
> reg = <0x680000 0x1f980000>;
> };
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
On Mon, 22 Jul 2013, Lee Jones wrote:
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap3-igep0020.dts | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap3-igep0020.dts b/arch/arm/boot/dts/omap3-igep0020.dts
> index e8c4828..f7a8122 100644
> --- a/arch/arm/boot/dts/omap3-igep0020.dts
> +++ b/arch/arm/boot/dts/omap3-igep0020.dts
> @@ -99,19 +99,19 @@
> label = "SPL";
> reg = <0 0x100000>;
> };
> - partition@0x80000 {
> + partition@80000 {
> label = "U-Boot";
> reg = <0x100000 0x180000>;
> };
> - partition@0x1c0000 {
> + partition@1c0000 {
> label = "Environment";
> reg = <0x280000 0x100000>;
> };
> - partition@0x280000 {
> + partition@280000 {
> label = "Kernel";
> reg = <0x380000 0x300000>;
> };
> - partition@0x780000 {
> + partition@780000 {
> label = "Filesystem";
> reg = <0x680000 0x1f980000>;
> };
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Tony, Benoit,
Poke
On Mon, 22 Jul 2013, Lee Jones wrote:
> Cc: Benoît Cousson <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Cc: [email protected]
> Signed-off-by: Lee Jones <[email protected]>
> ---
> arch/arm/boot/dts/omap2420-h4.dts | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap2420-h4.dts b/arch/arm/boot/dts/omap2420-h4.dts
> index 224c08f..34cdecb 100644
> --- a/arch/arm/boot/dts/omap2420-h4.dts
> +++ b/arch/arm/boot/dts/omap2420-h4.dts
> @@ -50,15 +50,15 @@
> label = "bootloader";
> reg = <0 0x20000>;
> };
> - partition@0x20000 {
> + partition@20000 {
> label = "params";
> reg = <0x20000 0x20000>;
> };
> - partition@0x40000 {
> + partition@40000 {
> label = "kernel";
> reg = <0x40000 0x200000>;
> };
> - partition@0x240000 {
> + partition@240000 {
> label = "file-system";
> reg = <0x240000 0x3dc0000>;
> };
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
* Lee Jones <[email protected]> [130813 08:51]:
> Tony, Benoit,
>
> Poke
Looks like Benoit's new email address is [email protected].
Probably best for Benoit to pick these to avoid merge conflicts.
Regards,
Tony
> On Mon, 22 Jul 2013, Lee Jones wrote:
>
> > Cc: Benoît Cousson <[email protected]>
> > Cc: Tony Lindgren <[email protected]>
> > Cc: [email protected]
> > Signed-off-by: Lee Jones <[email protected]>
> > ---
> > arch/arm/boot/dts/omap2420-h4.dts | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/omap2420-h4.dts b/arch/arm/boot/dts/omap2420-h4.dts
> > index 224c08f..34cdecb 100644
> > --- a/arch/arm/boot/dts/omap2420-h4.dts
> > +++ b/arch/arm/boot/dts/omap2420-h4.dts
> > @@ -50,15 +50,15 @@
> > label = "bootloader";
> > reg = <0 0x20000>;
> > };
> > - partition@0x20000 {
> > + partition@20000 {
> > label = "params";
> > reg = <0x20000 0x20000>;
> > };
> > - partition@0x40000 {
> > + partition@40000 {
> > label = "kernel";
> > reg = <0x40000 0x200000>;
> > };
> > - partition@0x240000 {
> > + partition@240000 {
> > label = "file-system";
> > reg = <0x240000 0x3dc0000>;
> > };
>
> --
> Lee Jones
> Linaro ST-Ericsson Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
Hi Lee,
Thanks for that cleanup.
On 14/08/2013 09:30, Tony Lindgren wrote:
> * Lee Jones <[email protected]> [130813 08:51]:
>> Tony, Benoit,
>>
>> Poke
>
> Looks like Benoit's new email address is [email protected].
> Probably best for Benoit to pick these to avoid merge conflicts.
I've just applied the 7 following patches. Let me know if I missed
something.
6b9fa1b ARM: dts: Remove '0x's from OMAP5 DTS file
79390b8 ARM: dts: Remove '0x's from OMAP4 DTS file
dd60fef ARM: dts: Remove '0x's from OMAP3430 SDP DTS file
bfef1cb ARM: dts: Remove '0x's from OMAP3 DTS file
5f547ac ARM: dts: Remove '0x's from OMAP3 IGEP0030 DTS file
27939fc ARM: dts: Remove '0x's from OMAP3 IGEP0020 DTS file
62eb4d1 ARM: dts: Remove '0x's from OMAP2420 H4 DTS file
Regards,
Benoit
>
> Regards,
>
> Tony
>
>> On Mon, 22 Jul 2013, Lee Jones wrote:
>>
>>> Cc: Benoît Cousson <[email protected]>
>>> Cc: Tony Lindgren <[email protected]>
>>> Cc: [email protected]
>>> Signed-off-by: Lee Jones <[email protected]>
>>> ---
>>> arch/arm/boot/dts/omap2420-h4.dts | 6 +++---
>>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/omap2420-h4.dts b/arch/arm/boot/dts/omap2420-h4.dts
>>> index 224c08f..34cdecb 100644
>>> --- a/arch/arm/boot/dts/omap2420-h4.dts
>>> +++ b/arch/arm/boot/dts/omap2420-h4.dts
>>> @@ -50,15 +50,15 @@
>>> label = "bootloader";
>>> reg = <0 0x20000>;
>>> };
>>> - partition@0x20000 {
>>> + partition@20000 {
>>> label = "params";
>>> reg = <0x20000 0x20000>;
>>> };
>>> - partition@0x40000 {
>>> + partition@40000 {
>>> label = "kernel";
>>> reg = <0x40000 0x200000>;
>>> };
>>> - partition@0x240000 {
>>> + partition@240000 {
>>> label = "file-system";
>>> reg = <0x240000 0x3dc0000>;
>>> };
>>
>> --
>> Lee Jones
>> Linaro ST-Ericsson Landing Team Lead
>> Linaro.org │ Open source software for ARM SoCs
>> Follow Linaro: Facebook | Twitter | Blog
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> >Looks like Benoit's new email address is [email protected].
> >Probably best for Benoit to pick these to avoid merge conflicts.
Ah nice, thanks Tony.
> I've just applied the 7 following patches. Let me know if I missed
> something.
>
> 6b9fa1b ARM: dts: Remove '0x's from OMAP5 DTS file
> 79390b8 ARM: dts: Remove '0x's from OMAP4 DTS file
> dd60fef ARM: dts: Remove '0x's from OMAP3430 SDP DTS file
> bfef1cb ARM: dts: Remove '0x's from OMAP3 DTS file
> 5f547ac ARM: dts: Remove '0x's from OMAP3 IGEP0030 DTS file
> 27939fc ARM: dts: Remove '0x's from OMAP3 IGEP0020 DTS file
> 62eb4d1 ARM: dts: Remove '0x's from OMAP2420 H4 DTS file
Nope, looks good. Thanks Benoit.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
On 13-08-12 06:54 AM, Lee Jones wrote:
> On Thu, 08 Aug 2013, Christian Daudt wrote:
>
>> Hi Lee,
>> I had to change some of the lines in the dts files you modified, so
>> I've added your mods. This patch:
>> https://lkml.org/lkml/2013/8/8/477
>>
>> includes your changes to bcm11351.dtsi and bcm11351-brt.dts. Not sure
>> where your patch is at this point, but I just wanted to point that
>> out.
> I haven't done anything with that patch yet, but I'd prefer they
> stayed separate to be honest. If you make your changes and ensure
> they're in an upstream tree (and let me know which one), I'll rebase
> my patches on top of that tree.
These changes are in 3.12-rc1 now:
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=d7358f845645c588aff84ac6b476a17d3eec75fa
thanks,
csd
On Wed, 18 Sep 2013, Christian Daudt wrote:
> On 13-08-12 06:54 AM, Lee Jones wrote:
> >On Thu, 08 Aug 2013, Christian Daudt wrote:
> >
> >>Hi Lee,
> >> I had to change some of the lines in the dts files you modified, so
> >>I've added your mods. This patch:
> >>https://lkml.org/lkml/2013/8/8/477
> >>
> >>includes your changes to bcm11351.dtsi and bcm11351-brt.dts. Not sure
> >>where your patch is at this point, but I just wanted to point that
> >>out.
> >I haven't done anything with that patch yet, but I'd prefer they
> >stayed separate to be honest. If you make your changes and ensure
> >they're in an upstream tree (and let me know which one), I'll rebase
> >my patches on top of that tree.
>
> These changes are in 3.12-rc1 now:
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=d7358f845645c588aff84ac6b476a17d3eec75fa
Not even so much as a Reviewed-by:, nice!
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog