diff mbox

ARM: dts: da850: fix W=1 warnings with pinmux node

Message ID 20180417123600.23548-1-nsekhar@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sekhar Nori April 17, 2018, 12:36 p.m. UTC 
Remove unused #address-cells and #size-cells from pinmux
node. This fixes W=1 warnings of the type:

arch/arm/boot/dts/da850-lcdk.dtb: Warning (avoid_unnecessary_addr_size): /soc@1c00000/pinmux@14120: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property

Tested on DA850 LCDK by checking output of:

/sys/kernel/debug/pinctrl/1c14120.pinmux-pinctrl-single/pins

before and after the change.

Signed-off-by: Sekhar Nori <nsekhar@ti.com>
---
 arch/arm/boot/dts/da850.dtsi | 2 --
 1 file changed, 2 deletions(-)

Comments

David Lechner April 17, 2018, 3:56 p.m. UTC | #1 
On 04/17/2018 07:36 AM, Sekhar Nori wrote:
> Remove unused #address-cells and #size-cells from pinmux
> node. This fixes W=1 warnings of the type:
> 
> arch/arm/boot/dts/da850-lcdk.dtb: Warning (avoid_unnecessary_addr_size): /soc@1c00000/pinmux@14120: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> 
> Tested on DA850 LCDK by checking output of:
> 
> /sys/kernel/debug/pinctrl/1c14120.pinmux-pinctrl-single/pins
> 
> before and after the change.
> 
> Signed-off-by: Sekhar Nori <nsekhar@ti.com>
> ---

Reviewed-by: David Lechner <david@lechnology.com>
Sekhar Nori April 24, 2018, 9:01 a.m. UTC | #2 
On Tuesday 17 April 2018 09:26 PM, David Lechner wrote:
> On 04/17/2018 07:36 AM, Sekhar Nori wrote:
>> Remove unused #address-cells and #size-cells from pinmux
>> node. This fixes W=1 warnings of the type:
>>
>> arch/arm/boot/dts/da850-lcdk.dtb: Warning
>> (avoid_unnecessary_addr_size): /soc@1c00000/pinmux@14120: unnecessary
>> #address-cells/#size-cells without "ranges" or child "reg" property
>>
>> Tested on DA850 LCDK by checking output of:
>>
>> /sys/kernel/debug/pinctrl/1c14120.pinmux-pinctrl-single/pins
>>
>> before and after the change.
>>
>> Signed-off-by: Sekhar Nori <nsekhar@ti.com>
>> ---
> 
> Reviewed-by: David Lechner <david@lechnology.com>

Applied to fixes.

Thanks,
Sekhar
diff mbox

Patch

diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index cba3afd00763..f6f1597b03df 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -55,8 +55,6 @@ 
 		pmx_core: pinmux@14120 {
 			compatible = "pinctrl-single";
 			reg = <0x14120 0x50>;
-			#address-cells = <1>;
-			#size-cells = <0>;
 			#pinctrl-cells = <2>;
 			pinctrl-single,bit-per-mux;
 			pinctrl-single,register-width = <32>;