[PATCH] mfd: ab8500: Fix up the GPADC binding example

[PATCH] mfd: ab8500: Fix up the GPADC binding example

[Linus Walleij]

This fixes up the missed device node examples from using
underscores to using dashes as pointed out by Rob.

Cc: Lee Jones <lee.jones@linaro.org>
Cc: Rob Herring <robh@kernel.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
Hi Jonathan, you can either apply this standalone or
squash it into the previous DT binding patch while fixing
up the Capitalization Of First Word After Dash in the
Subject.
---
 Documentation/devicetree/bindings/mfd/ab8500.txt | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/mfd/ab8500.txt b/Documentation/devicetree/bindings/mfd/ab8500.txt
index b6bc30d7777e..a4f70f0d653f 100644
--- a/Documentation/devicetree/bindings/mfd/ab8500.txt
+++ b/Documentation/devicetree/bindings/mfd/ab8500.txt
@@ -181,26 +181,26 @@ ab8500 {
 		};
         };
 
-	ab8500_temp {
+	ab8500-temp {
 		compatible = "stericsson,abx500-temp";
 		io-channels = <&gpadc 0x06>,
 			      <&gpadc 0x07>;
 		io-channel-name = "aux1", "aux2";
 	};
 
-	ab8500_battery: ab8500_battery {
+	ab8500_battery: ab8500-battery {
 		stericsson,battery-type = "LIPO";
 		thermistor-on-batctrl;
 	};
 
-	ab8500_fg {
+	ab8500-fg {
 		compatible = "stericsson,ab8500-fg";
 		battery	   = <&ab8500_battery>;
 		io-channels = <&gpadc 0x08>;
 		io-channel-name = "main_bat_v";
 	};
 
-	ab8500_btemp {
+	ab8500-btemp {
 		compatible = "stericsson,ab8500-btemp";
 		battery	   = <&ab8500_battery>;
 		io-channels = <&gpadc 0x02>,
@@ -209,7 +209,7 @@ ab8500 {
 				"bat_ctrl";
 	};
 
-	ab8500_charger {
+	ab8500-charger {
 		compatible	= "stericsson,ab8500-charger";
 		battery		= <&ab8500_battery>;
 		vddadc-supply	= <&ab8500_ldo_tvout_reg>;
-- 
2.21.0

Re: [PATCH] mfd: ab8500: Fix up the GPADC binding example

[Lee Jones]

On Fri, 18 Oct 2019, Linus Walleij wrote:

> This fixes up the missed device node examples from using
> underscores to using dashes as pointed out by Rob.

Patch is okay.  Subject is not correct.

dt-bindings: mfd: XXX

> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Rob Herring <robh@kernel.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> Hi Jonathan, you can either apply this standalone or
> squash it into the previous DT binding patch while fixing
> up the Capitalization Of First Word After Dash in the
> Subject.
> ---
>  Documentation/devicetree/bindings/mfd/ab8500.txt | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/ab8500.txt b/Documentation/devicetree/bindings/mfd/ab8500.txt
> index b6bc30d7777e..a4f70f0d653f 100644
> --- a/Documentation/devicetree/bindings/mfd/ab8500.txt
> +++ b/Documentation/devicetree/bindings/mfd/ab8500.txt
> @@ -181,26 +181,26 @@ ab8500 {
>  		};
>          };
>  
> -	ab8500_temp {
> +	ab8500-temp {
>  		compatible = "stericsson,abx500-temp";
>  		io-channels = <&gpadc 0x06>,
>  			      <&gpadc 0x07>;
>  		io-channel-name = "aux1", "aux2";
>  	};
>  
> -	ab8500_battery: ab8500_battery {
> +	ab8500_battery: ab8500-battery {
>  		stericsson,battery-type = "LIPO";
>  		thermistor-on-batctrl;
>  	};
>  
> -	ab8500_fg {
> +	ab8500-fg {
>  		compatible = "stericsson,ab8500-fg";
>  		battery	   = <&ab8500_battery>;
>  		io-channels = <&gpadc 0x08>;
>  		io-channel-name = "main_bat_v";
>  	};
>  
> -	ab8500_btemp {
> +	ab8500-btemp {
>  		compatible = "stericsson,ab8500-btemp";
>  		battery	   = <&ab8500_battery>;
>  		io-channels = <&gpadc 0x02>,
> @@ -209,7 +209,7 @@ ab8500 {
>  				"bat_ctrl";
>  	};
>  
> -	ab8500_charger {
> +	ab8500-charger {
>  		compatible	= "stericsson,ab8500-charger";
>  		battery		= <&ab8500_battery>;
>  		vddadc-supply	= <&ab8500_ldo_tvout_reg>;

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog