[PATCH] regulator: tps65911: fix regulator-compatible documentation

[PATCH] regulator: tps65911: fix regulator-compatible documentation

[Christoph Fritz]

Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
---
 Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/tps65910.txt b/Documentation/devicetree/bindings/mfd/tps65910.txt
index 8af1202..4f62143 100644
--- a/Documentation/devicetree/bindings/mfd/tps65910.txt
+++ b/Documentation/devicetree/bindings/mfd/tps65910.txt
@@ -22,7 +22,7 @@ Required properties:
   The valid regulator-compatible values are:
   tps65910: vrtc, vio, vdd1, vdd2, vdd3, vdig1, vdig2, vpll, vdac, vaux1,
             vaux2, vaux33, vmmc, vbb
-  tps65911: vrtc, vio, vdd1, vdd3, vddctrl, ldo1, ldo2, ldo3, ldo4, ldo5,
+  tps65911: vrtc, vio, vdd1, vdd2, vddctrl, ldo1, ldo2, ldo3, ldo4, ldo5,
             ldo6, ldo7, ldo8
 
 - xxx-supply: Input voltage supply regulator.
-- 
2.1.4

Re: [PATCH] regulator: tps65911: fix regulator-compatible documentation

[Lee Jones]

On Mon, 04 Jun 2018, Christoph Fritz wrote:

> 
> Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
> ---
>  Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/tps65910.txt b/Documentation/devicetree/bindings/mfd/tps65910.txt
> index 8af1202..4f62143 100644
> --- a/Documentation/devicetree/bindings/mfd/tps65910.txt
> +++ b/Documentation/devicetree/bindings/mfd/tps65910.txt
> @@ -22,7 +22,7 @@ Required properties:
>    The valid regulator-compatible values are:
>    tps65910: vrtc, vio, vdd1, vdd2, vdd3, vdig1, vdig2, vpll, vdac, vaux1,
>              vaux2, vaux33, vmmc, vbb
> -  tps65911: vrtc, vio, vdd1, vdd3, vddctrl, ldo1, ldo2, ldo3, ldo4, ldo5,
> +  tps65911: vrtc, vio, vdd1, vdd2, vddctrl, ldo1, ldo2, ldo3, ldo4, ldo5,
>              ldo6, ldo7, ldo8
>  
>  - xxx-supply: Input voltage supply regulator.

<playing_dumb>

I have no idea what I'm meant to be looking for.

Please add a suitable commit log.

</playing_dumb>

Also, this is not a "regulator" patch, as your $SUBJECT suggests.
Please change the pre-fix to "dt-bindings: mfd: "

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

[PATCH v2] dt-bindings: mfd: fix documentation of tps65911

[Christoph Fritz]

This patch fixes documentation of tps65911 because its list of
compatible regulators contains wrongly vdd3 instead of vdd2.

Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
---
 Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/tps65910.txt b/Documentation/devicetree/bindings/mfd/tps65910.txt
index 8af1202..4f62143 100644
--- a/Documentation/devicetree/bindings/mfd/tps65910.txt
+++ b/Documentation/devicetree/bindings/mfd/tps65910.txt
@@ -22,7 +22,7 @@ Required properties:
   The valid regulator-compatible values are:
   tps65910: vrtc, vio, vdd1, vdd2, vdd3, vdig1, vdig2, vpll, vdac, vaux1,
             vaux2, vaux33, vmmc, vbb
-  tps65911: vrtc, vio, vdd1, vdd3, vddctrl, ldo1, ldo2, ldo3, ldo4, ldo5,
+  tps65911: vrtc, vio, vdd1, vdd2, vddctrl, ldo1, ldo2, ldo3, ldo4, ldo5,
             ldo6, ldo7, ldo8
 
 - xxx-supply: Input voltage supply regulator.
-- 
2.1.4

Re: [PATCH v2] dt-bindings: mfd: fix documentation of tps65911

[Rob Herring]

On Tue, Jun 05, 2018 at 11:26:49AM +0200, Christoph Fritz wrote:
> This patch fixes documentation of tps65911 because its list of
> compatible regulators contains wrongly vdd3 instead of vdd2.
> 
> Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
> ---
>  Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH v2] dt-bindings: mfd: fix documentation of tps65911

[Christoph Fritz]

Hey Lee

On Mon, 2018-06-11 at 12:27 -0600, Rob Herring wrote:
> On Tue, Jun 05, 2018 at 11:26:49AM +0200, Christoph Fritz wrote:
> > This patch fixes documentation of tps65911 because its list of
> > compatible regulators contains wrongly vdd3 instead of vdd2.
> > 
> > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
> > ---
> >  Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Acked-by: Rob Herring <robh@kernel.org>

any chance to get this patch queued soon?

Thanks
 -- Christoph

Re: [PATCH v2] dt-bindings: mfd: fix documentation of tps65911

[Lee Jones]

On Sat, 23 Jun 2018, Christoph Fritz wrote:
> On Mon, 2018-06-11 at 12:27 -0600, Rob Herring wrote:
> > On Tue, Jun 05, 2018 at 11:26:49AM +0200, Christoph Fritz wrote:
> > > This patch fixes documentation of tps65911 because its list of
> > > compatible regulators contains wrongly vdd3 instead of vdd2.
> > > 
> > > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
> > > ---
> > >  Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> 
> any chance to get this patch queued soon?

It's in the queue to be reviewed.  I have just returned from paternity
leave and have 290 emails to get through.  Please bear with me.

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

Re: [PATCH v2] dt-bindings: mfd: fix documentation of tps65911

[Lee Jones]

On Tue, 05 Jun 2018, Christoph Fritz wrote:

> This patch fixes documentation of tps65911 because its list of
> compatible regulators contains wrongly vdd3 instead of vdd2.
> 
> Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
> ---
>  Documentation/devicetree/bindings/mfd/tps65910.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

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