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

From: A.S. Dong
Date: Wed Apr 19 2017 - 11:33:41 EST


> -----Original Message-----
> From: Dong Aisheng [mailto:aisheng.dong@xxxxxxx]
> Sent: Friday, April 14, 2017 10:31 PM
> To: linux-kernel@xxxxxxxxxxxxxxx
> Cc: A.S. Dong; Rob Herring; Mark Rutland; devicetree@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxx>
> Cc: Mark Rutland <mark.rutland@xxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx
> Signed-off-by: Dong Aisheng <aisheng.dong@xxxxxxx>

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