[1/3] arm64: dts: juno: Fix "debounce-interval" property misspelling

Message ID 1525884291-18851-1-git-send-email-sudeep.holla@arm.com
State New
Headers show
Series
  • [1/3] arm64: dts: juno: Fix "debounce-interval" property misspelling
Related show

Commit Message

Sudeep Holla May 9, 2018, 4:44 p.m.
"debounce_interval" was never supported in the bindings. It should be
"debounce-interval". Moreover, latest DTC complains the following:

Warning (property_name_chars_strict): debounce_interval: Character '_' not recommended in property name

This patch fixes the above warning by using the correct property as
per the bindings.

Cc: Liviu Dudau <liviu.dudau@arm.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

---
 arch/arm64/boot/dts/arm/juno-motherboard.dtsi | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Liviu Dudau May 10, 2018, 9:44 a.m. | #1
On Wed, May 09, 2018 at 05:44:49PM +0100, Sudeep Holla wrote:
> "debounce_interval" was never supported in the bindings. It should be

> "debounce-interval". Moreover, latest DTC complains the following:

> 

> Warning (property_name_chars_strict): debounce_interval: Character '_' not recommended in property name

> 

> This patch fixes the above warning by using the correct property as

> per the bindings.

> 

> Cc: Liviu Dudau <liviu.dudau@arm.com>


Acked-by: Liviu Dudau <liviu.dudau@arm.com>


> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

> ---

>  arch/arm64/boot/dts/arm/juno-motherboard.dtsi | 12 ++++++------

>  1 file changed, 6 insertions(+), 6 deletions(-)

> 

> diff --git a/arch/arm64/boot/dts/arm/juno-motherboard.dtsi b/arch/arm64/boot/dts/arm/juno-motherboard.dtsi

> index 69804c5f1197..4b38195a8bea 100644

> --- a/arch/arm64/boot/dts/arm/juno-motherboard.dtsi

> +++ b/arch/arm64/boot/dts/arm/juno-motherboard.dtsi

> @@ -58,42 +58,42 @@

>  				compatible = "gpio-keys";

>  

>  				power-button {

> -					debounce_interval = <50>;

> +					debounce-interval = <50>;

>  					wakeup-source;

>  					linux,code = <116>;

>  					label = "POWER";

>  					gpios = <&iofpga_gpio0 0 0x4>;

>  				};

>  				home-button {

> -					debounce_interval = <50>;

> +					debounce-interval = <50>;

>  					wakeup-source;

>  					linux,code = <102>;

>  					label = "HOME";

>  					gpios = <&iofpga_gpio0 1 0x4>;

>  				};

>  				rlock-button {

> -					debounce_interval = <50>;

> +					debounce-interval = <50>;

>  					wakeup-source;

>  					linux,code = <152>;

>  					label = "RLOCK";

>  					gpios = <&iofpga_gpio0 2 0x4>;

>  				};

>  				vol-up-button {

> -					debounce_interval = <50>;

> +					debounce-interval = <50>;

>  					wakeup-source;

>  					linux,code = <115>;

>  					label = "VOL+";

>  					gpios = <&iofpga_gpio0 3 0x4>;

>  				};

>  				vol-down-button {

> -					debounce_interval = <50>;

> +					debounce-interval = <50>;

>  					wakeup-source;

>  					linux,code = <114>;

>  					label = "VOL-";

>  					gpios = <&iofpga_gpio0 4 0x4>;

>  				};

>  				nmi-button {

> -					debounce_interval = <50>;

> +					debounce-interval = <50>;

>  					wakeup-source;

>  					linux,code = <99>;

>  					label = "NMI";

> -- 

> 2.7.4

> 


-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ¯\_(ツ)_/¯
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/arch/arm64/boot/dts/arm/juno-motherboard.dtsi b/arch/arm64/boot/dts/arm/juno-motherboard.dtsi
index 69804c5f1197..4b38195a8bea 100644
--- a/arch/arm64/boot/dts/arm/juno-motherboard.dtsi
+++ b/arch/arm64/boot/dts/arm/juno-motherboard.dtsi
@@ -58,42 +58,42 @@ 
 				compatible = "gpio-keys";
 
 				power-button {
-					debounce_interval = <50>;
+					debounce-interval = <50>;
 					wakeup-source;
 					linux,code = <116>;
 					label = "POWER";
 					gpios = <&iofpga_gpio0 0 0x4>;
 				};
 				home-button {
-					debounce_interval = <50>;
+					debounce-interval = <50>;
 					wakeup-source;
 					linux,code = <102>;
 					label = "HOME";
 					gpios = <&iofpga_gpio0 1 0x4>;
 				};
 				rlock-button {
-					debounce_interval = <50>;
+					debounce-interval = <50>;
 					wakeup-source;
 					linux,code = <152>;
 					label = "RLOCK";
 					gpios = <&iofpga_gpio0 2 0x4>;
 				};
 				vol-up-button {
-					debounce_interval = <50>;
+					debounce-interval = <50>;
 					wakeup-source;
 					linux,code = <115>;
 					label = "VOL+";
 					gpios = <&iofpga_gpio0 3 0x4>;
 				};
 				vol-down-button {
-					debounce_interval = <50>;
+					debounce-interval = <50>;
 					wakeup-source;
 					linux,code = <114>;
 					label = "VOL-";
 					gpios = <&iofpga_gpio0 4 0x4>;
 				};
 				nmi-button {
-					debounce_interval = <50>;
+					debounce-interval = <50>;
 					wakeup-source;
 					linux,code = <99>;
 					label = "NMI";