Skip to contentSkip to conversation

ARM: dts: am33xx: Remove unnecessary properties

PATCH In review Felix Brack 2 participants 4 messages Dec 18, 2018 – Jan 24, 2019 Device tree bindings

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: dts: am33xx: Remove unnecessary properties

Remove the unnecessary properties #address-cells and #size-cells of node pinmux as there are no child-nodes with property reg.

---
 arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
 1 file changed, 2 deletions(-)
arch/arm/boot/dts/am33xx-l4.dtsi +0 -2
index 7b818d9d2eab..e957370f8aec 100644
@@ -288,8 +288,6 @@
 				am33xx_pinmux: pinmux@800 {
 					compatible = "pinctrl-single";
 					reg = <0x800 0x238>;
-					#address-cells = <1>;
-					#size-cells = <0>;
 					#pinctrl-cells = <1>;
 					pinctrl-single,register-width = <32>;
 					pinctrl-single,function-mask = <0x7f>;
Signature · 1 line
2.17.1
#1 lore raw

Hi Tony,

Quoted 14 lines · Felix Brack

Remove the unnecessary properties #address-cells and #size-cells of node pinmux as there are no child-nodes with property reg.

---
  arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
  1 file changed, 2 deletions(-)
arch/arm/boot/dts/am33xx-l4.dtsi +0 -0
index 7b818d9d2eab..e957370f8aec 100644

arch/arm/boot/dts/am33xx-l4.dtsi line 293

Quoted 8 lines
arch/arm/boot/dts/am33xx-l4.dtsi +0 -2
@@ -288,8 +288,6 @@
  				am33xx_pinmux: pinmux@800 {
  					compatible = "pinctrl-single";
  					reg = <0x800 0x238>;
-					#address-cells = <1>;
-					#size-cells = <0>;
  					#pinctrl-cells = <1>;
  					pinctrl-single,register-width = <32>;
  					pinctrl-single,function-mask = <0x7f>;

Just a gentle ping. BTW: the patch applies not just to linux-next correctly but now also to linux.

regards Felix

#2 lore raw
Quoted 19 lines · Felix Brack

Hi Tony,

Quoted 17 lines · Felix Brack

Remove the unnecessary properties #address-cells and #size-cells of node pinmux as there are no child-nodes with property reg.

---
  arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
  1 file changed, 2 deletions(-)
arch/arm/boot/dts/am33xx-l4.dtsi +0 -1
index 7b818d9d2eab..e957370f8aec 100644
@@ -288,8 +288,6 @@
  				am33xx_pinmux: pinmux@800 {
  					compatible = "pinctrl-single";
  					reg = <0x800 0x238>;
-					#address-cells = <1>;
Quoted 8 lines

arch/arm/boot/dts/am33xx-l4.dtsi line 293

Quoted 5 lines
arch/arm/boot/dts/am33xx-l4.dtsi +0 -1
-					#size-cells = <0>;
  					#pinctrl-cells = <1>;
  					pinctrl-single,register-width = <32>;
  					pinctrl-single,function-mask = <0x7f>;

Just a gentle ping. BTW: the patch applies not just to linux-next correctly but now also to linux.

Sorry I'm still struggling with pending regressions for the -rc cycle.. I should get to applying patches for next merge window this week though.

Regards,

Tony

#3 lore raw
Quoted 29 lines · Tony Lindgren
Quoted 19 lines · Felix Brack

Hi Tony,

Quoted 17 lines · Felix Brack

Remove the unnecessary properties #address-cells and #size-cells of node pinmux as there are no child-nodes with property reg.

---
  arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
  1 file changed, 2 deletions(-)
arch/arm/boot/dts/am33xx-l4.dtsi +0 -1
index 7b818d9d2eab..e957370f8aec 100644
@@ -288,8 +288,6 @@
  				am33xx_pinmux: pinmux@800 {
  					compatible = "pinctrl-single";
  					reg = <0x800 0x238>;
-					#address-cells = <1>;
Quoted 8 lines

arch/arm/boot/dts/am33xx-l4.dtsi line 293

Quoted 5 lines
arch/arm/boot/dts/am33xx-l4.dtsi +0 -1
-					#size-cells = <0>;
  					#pinctrl-cells = <1>;
  					pinctrl-single,register-width = <32>;
  					pinctrl-single,function-mask = <0x7f>;

Just a gentle ping. BTW: the patch applies not just to linux-next correctly but now also to linux.

Quoted 3 lines

Sorry I'm still struggling with pending regressions for the -rc cycle.. I should get to applying patches for next merge window this week though.

Applying into omap-for-v5.1/dt finally today thanks.

Tony