Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1626393 > unrolled thread

RE: [PATCH v2 1/2] dt-binding: regulator: anatop: make regulator name property required

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  RE: [PATCH v2 1/2] dt-binding: regulator: anatop: make regulator name  property required "A.S. Dong" <aisheng.dong@nxp.com> - 2017-04-19 17:40 +0200
    Re: [PATCH v2 1/2] dt-binding: regulator: anatop: make regulator  name property required Mark Brown <broonie@kernel.org> - 2017-04-20 22:30 +0200

#1626393 — RE: [PATCH v2 1/2] dt-binding: regulator: anatop: make regulator name property required

> -----Original Message-----
> From: Dong Aisheng [mailto:aisheng.dong@nxp.com]
> Sent: Friday, April 14, 2017 10:31 PM
> To: linux-kernel@vger.kernel.org
> Cc: A.S. Dong; Rob Herring; Mark Rutland; devicetree@vger.kernel.org
> Subject: [PATCH v2 1/2] dt-binding: regulator: anatop: make regulator name
> property required
> 
> We actually can't allow the missing of the regualor name, thus update the
> binding doc to make regulator-name property to be required.
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>

Sorry, Mark, missed you about this one.

Regards
Dong Aisheng

> ---
>  Documentation/devicetree/bindings/regulator/anatop-regulator.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/regulator/anatop-
> regulator.txt b/Documentation/devicetree/bindings/regulator/anatop-
> regulator.txt
> index 1d58c8c..a3106c7 100644
> --- a/Documentation/devicetree/bindings/regulator/anatop-regulator.txt
> +++ b/Documentation/devicetree/bindings/regulator/anatop-regulator.txt
> @@ -2,6 +2,7 @@ Anatop Voltage regulators
> 
>  Required properties:
>  - compatible: Must be "fsl,anatop-regulator"
> +- regulator-name: A string used as a descriptive name for regulator
> +outputs
>  - anatop-reg-offset: Anatop MFD register offset
>  - anatop-vol-bit-shift: Bit shift for the register
>  - anatop-vol-bit-width: Number of bits used in the register
> --
> 2.7.4

[toc] | [next] | [standalone]

#1627791 — Re: [PATCH v2 1/2] dt-binding: regulator: anatop: make regulator name property required

[Multipart message — attachments visible in raw view] — view raw

On Wed, Apr 19, 2017 at 03:33:03PM +0000, A.S. Dong wrote:

> > We actually can't allow the missing of the regualor name, thus update the
> > binding doc to make regulator-name property to be required.

> Sorry, Mark, missed you about this one.

Please resend, I don't have a copy.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web