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

[Lee Jones <lee.jones@linaro.org>]

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