ARM: dts: kirkwood: Remove button address and fixup names
authorAndrew Lunn <andrew@lunn.ch>
Sun, 3 Apr 2016 02:03:43 +0000 (04:03 +0200)
committerGregory CLEMENT <gregory.clement@free-electrons.com>
Thu, 14 Apr 2016 17:04:45 +0000 (19:04 +0200)
The DT compiler is now warning about unit names with addresses but not
reg property. Fix all the gpio-key buttons which causes warnings.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
18 files changed:
arch/arm/boot/dts/kirkwood-blackarmor-nas220.dts
arch/arm/boot/dts/kirkwood-cloudbox.dts
arch/arm/boot/dts/kirkwood-dnskw.dtsi
arch/arm/boot/dts/kirkwood-ib62x0.dts
arch/arm/boot/dts/kirkwood-iconnect.dts
arch/arm/boot/dts/kirkwood-laplug.dts
arch/arm/boot/dts/kirkwood-lsxl.dtsi
arch/arm/boot/dts/kirkwood-mv88f6281gtw-ge.dts
arch/arm/boot/dts/kirkwood-netxbig.dtsi
arch/arm/boot/dts/kirkwood-ns2-common.dtsi
arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
arch/arm/boot/dts/kirkwood-openblocks_a6.dts
arch/arm/boot/dts/kirkwood-openblocks_a7.dts
arch/arm/boot/dts/kirkwood-pogoplug-series-4.dts
arch/arm/boot/dts/kirkwood-t5325.dts
arch/arm/boot/dts/kirkwood-ts219-6281.dts
arch/arm/boot/dts/kirkwood-ts219-6282.dts
arch/arm/boot/dts/kirkwood-ts419.dtsi

index fa02a9a..f16a73e 100644 (file)
        gpio_keys {
                compatible = "gpio-keys";
 
-               button@1{
+               reset {
                        label = "Reset";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio0 29 GPIO_ACTIVE_HIGH>;
                };
 
-               button@2{
+               button {
                        label = "Power";
                        linux,code = <KEY_SLEEP>;
                        gpios = <&gpio0 26 GPIO_ACTIVE_LOW>;
index 7ec7656..555b7e4 100644 (file)
@@ -60,7 +60,7 @@
                #address-cells = <1>;
                #size-cells = <0>;
 
-               button@1 {
+               power {
                        label = "Power push button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio0 16 GPIO_ACTIVE_LOW>;
index 113dcf0..d8fca9d 100644 (file)
                             &pmx_button_reset>;
                pinctrl-names = "default";
 
-               button@1 {
+               power {
                        label = "Power button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 2 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               eject {
                        label = "USB unmount button";
                        linux,code = <KEY_EJECTCD>;
                        gpios = <&gpio1 15 GPIO_ACTIVE_LOW>;
                };
-               button@3 {
+               reset {
                        label = "Reset button";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio1 16 GPIO_ACTIVE_LOW>;
index bfa5edd..ef84d86 100644 (file)
                pinctrl-0 = <&pmx_button_reset &pmx_button_usb_copy>;
                pinctrl-names = "default";
 
-               button@1 {
+               copy {
                        label = "USB Copy";
                        linux,code = <KEY_COPY>;
                        gpios = <&gpio0 29 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               reset {
                        label = "Reset";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio0 28 GPIO_ACTIVE_LOW>;
index 38e31d1..6743067 100644 (file)
                pinctrl-0 = < &pmx_button_reset &pmx_button_otb >;
                pinctrl-names = "default";
 
-               button@1 {
+               otb {
                        label = "OTB Button";
                        linux,code = <KEY_COPY>;
                        gpios = <&gpio1 3 GPIO_ACTIVE_LOW>;
                        debounce-interval = <100>;
                };
-               button@2 {
+               reset {
                        label = "Reset";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio0 12 GPIO_ACTIVE_LOW>;
index 2442566..c39e17a 100644 (file)
@@ -62,7 +62,7 @@
        gpio_keys {
                compatible = "gpio-keys";
 
-               button@1{
+               power {
                        label = "Power push button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 0 GPIO_ACTIVE_HIGH>;
index 1d6528d..2869c5d 100644 (file)
                             &pmx_power_auto_switch>;
                pinctrl-names = "default";
 
-               button@1 {
+               option {
                        label = "Function Button";
                        linux,code = <KEY_OPTION>;
                        gpios = <&gpio1 9 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               reserved {
                        label = "Power-on Switch";
                        linux,code = <KEY_RESERVED>;
                        linux,input-type = <5>;
                        gpios = <&gpio1 10 GPIO_ACTIVE_LOW>;
                };
-               button@3 {
+               power {
                        label = "Power-auto Switch";
                        linux,code = <KEY_ESC>;
                        linux,input-type = <5>;
index b7e7d78..6a8f59a 100644 (file)
                pinctrl-0 = <&pmx_keys>;
                pinctrl-names = "default";
 
-               button@1 {
+               restart {
                        label = "SWR Button";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio1 15 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               wps {
                        label = "WPS Button";
                        linux,code = <KEY_WPS_BUTTON>;
                        gpios = <&gpio1 14 GPIO_ACTIVE_LOW>;
index 62515a8..52b58fe 100644 (file)
                #size-cells = <0>;
 
                /*
-                * button@1 and button@2 represent a three position rocker
+                * esc and power represent a three position rocker
                 * switch. Thus the conventional KEY_POWER does not fit
                 */
-               button@1 {
+               exc {
                        label = "Back power switch (on|auto)";
                        linux,code = <KEY_ESC>;
                        linux,input-type = <5>;
                        gpios = <&gpio0 13 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               power {
                        label = "Back power switch (auto|off)";
                        linux,code = <KEY_1>;
                        linux,input-type = <5>;
                        gpios = <&gpio0 15 GPIO_ACTIVE_LOW>;
                };
-               button@3 {
+               option {
                        label = "Function button";
                        linux,code = <KEY_OPTION>;
                        gpios = <&gpio1 2 GPIO_ACTIVE_LOW>;
index e832b63..b4d5926 100644 (file)
@@ -57,7 +57,7 @@
                #address-cells = <1>;
                #size-cells = <0>;
 
-               button@1 {
+               power {
                        label = "Power push button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 0 GPIO_ACTIVE_HIGH>;
index 2075a2e..5ccc461 100644 (file)
                pinctrl-0 = <&pmx_btn_reset &pmx_btn_copy &pmx_btn_power>;
                pinctrl-names = "default";
 
-               button@1 {
+               power {
                        label = "Power Button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>;
                };
-               button@2 {
+               copy {
                        label = "Copy Button";
                        linux,code = <KEY_COPY>;
                        gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
                };
-               button@3 {
+               reset {
                        label = "Reset Button";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio1 4 GPIO_ACTIVE_LOW>;
index fb9dc22..0db0e3e 100644 (file)
                #address-cells = <1>;
                #size-cells = <0>;
 
-               button@1 {
+               power {
                        label = "Init Button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 6 GPIO_ACTIVE_HIGH>;
index d5e3bc5..cf2f524 100644 (file)
                #address-cells = <1>;
                #size-cells = <0>;
 
-               button@1 {
+               button {
                        label = "Init Button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 6 GPIO_ACTIVE_HIGH>;
index 8082d64..b2f2623 100644 (file)
@@ -33,7 +33,7 @@
                pinctrl-0 = <&pmx_button_eject>;
                pinctrl-names = "default";
 
-               button@1 {
+               eject {
                        debounce_interval = <50>;
                        wakeup-source;
                        linux,code = <KEY_EJECTCD>;
index ed956b8..af5ce85 100644 (file)
                pinctrl-0 = <&pmx_button_power>;
                pinctrl-names = "default";
 
-               button@1 {
+               power {
                        label = "Power Button";
                        linux,code = <KEY_POWER>;
                        gpios = <&gpio1 13 GPIO_ACTIVE_HIGH>;
index 9767d73..ee62204 100644 (file)
                pinctrl-0 = <&pmx_reset_button &pmx_USB_copy_button>;
                pinctrl-names = "default";
 
-               button@1 {
+               copy {
                        label = "USB Copy";
                        linux,code = <KEY_COPY>;
                        gpios = <&gpio0 15 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               reset {
                        label = "Reset";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio0 16 GPIO_ACTIVE_LOW>;
index bfc1a32..1d291f6 100644 (file)
                pinctrl-0 = <&pmx_reset_button &pmx_USB_copy_button>;
                pinctrl-names = "default";
 
-               button@1 {
+               copy {
                        label = "USB Copy";
                        linux,code = <KEY_COPY>;
                        gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               reset {
                        label = "Reset";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
index 30ab93b..02bd537 100644 (file)
                pinctrl-0 = <&pmx_reset_button &pmx_USB_copy_button>;
                pinctrl-names = "default";
 
-               button@1 {
+               copy {
                        label = "USB Copy";
                        linux,code = <KEY_COPY>;
                        gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
                };
-               button@2 {
+               reset {
                        label = "Reset";
                        linux,code = <KEY_RESTART>;
                        gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;