Re: [PATCH v2 1/7] regulator: dt-bindings: remove useless properties

From: Rob Herring
Date: Thu Dec 17 2020 - 18:50:34 EST


On Wed, Dec 16, 2020 at 12:56:33AM +0100, Adrien Grassein wrote:
> regulator-name is a generic property of the regulator.
> Don't repeat it here.

Yes, but the values aren't generic.

>
> Signed-off-by: Adrien Grassein <adrien.grassein@xxxxxxxxx>
> ---
> .../regulator/nxp,pf8x00-regulator.yaml | 17 -----------------
> 1 file changed, 17 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/regulator/nxp,pf8x00-regulator.yaml b/Documentation/devicetree/bindings/regulator/nxp,pf8x00-regulator.yaml
> index a6c259ce9785..913532d0532e 100644
> --- a/Documentation/devicetree/bindings/regulator/nxp,pf8x00-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/nxp,pf8x00-regulator.yaml
> @@ -36,12 +36,6 @@ properties:
> description:
> Properties for single LDO regulator.
>
> - properties:
> - regulator-name:
> - pattern: "^ldo[1-4]$"
> - description:
> - should be "ldo1", ..., "ldo4"
> -
> unevaluatedProperties: false
>
> "^buck[1-7]$":
> @@ -51,11 +45,6 @@ properties:
> Properties for single BUCK regulator.
>
> properties:
> - regulator-name:
> - pattern: "^buck[1-7]$"
> - description:
> - should be "buck1", ..., "buck7"
> -
> nxp,ilim-ma:
> $ref: "/schemas/types.yaml#/definitions/uint32"
> minimum: 2100
> @@ -94,12 +83,6 @@ properties:
> description:
> Properties for single VSNVS regulator.
>
> - properties:
> - regulator-name:
> - pattern: "^vsnvs$"
> - description:
> - should be "vsnvs"
> -
> unevaluatedProperties: false
>
> additionalProperties: false
> --
> 2.20.1
>