Unverified Commit 2a17ddfd authored by Rob Herring's avatar Rob Herring Committed by Mark Brown
Browse files

dt-bindings: Add missing 'unevaluatedProperties' to regulator nodes



Several regulator schemas are missing 'unevaluatedProperties' constraint
which means any extra properties are allowed. Upon adding the
constraint, there's numerous warnings from using the deprecated
'regulator-compatible' property. Remove the usage as examples shouldn't
be using long since deprecated properties.

Signed-off-by: default avatarRob Herring <robh@kernel.org>
Acked-by: default avatarLee Jones <lee@kernel.org>
Acked-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
Acked-by: default avatarBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Link: https://lore.kernel.org/r/20221206211554.92005-1-robh@kernel.org


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent cf34ac6a
Loading
Loading
Loading
Loading
+0 −2
Original line number Original line Diff line number Diff line
@@ -100,14 +100,12 @@ examples:
                compatible = "maxim,max77650-regulator";
                compatible = "maxim,max77650-regulator";


                max77650_ldo: regulator-ldo {
                max77650_ldo: regulator-ldo {
                    regulator-compatible = "ldo";
                    regulator-name = "max77650-ldo";
                    regulator-name = "max77650-ldo";
                    regulator-min-microvolt = <1350000>;
                    regulator-min-microvolt = <1350000>;
                    regulator-max-microvolt = <2937500>;
                    regulator-max-microvolt = <2937500>;
                };
                };


                max77650_sbb0: regulator-sbb0 {
                max77650_sbb0: regulator-sbb0 {
                    regulator-compatible = "sbb0";
                    regulator-name = "max77650-sbb0";
                    regulator-name = "max77650-sbb0";
                    regulator-min-microvolt = <800000>;
                    regulator-min-microvolt = <800000>;
                    regulator-max-microvolt = <1587500>;
                    regulator-max-microvolt = <1587500>;
+0 −9
Original line number Original line Diff line number Diff line
@@ -83,7 +83,6 @@ examples:
                richtek,vinovp-microvolt = <14500000>;
                richtek,vinovp-microvolt = <14500000>;


                otg_vbus_regulator: usb-otg-vbus-regulator {
                otg_vbus_regulator: usb-otg-vbus-regulator {
                    regulator-compatible = "usb-otg-vbus";
                    regulator-name = "usb-otg-vbus";
                    regulator-name = "usb-otg-vbus";
                    regulator-min-microvolt = <4425000>;
                    regulator-min-microvolt = <4425000>;
                    regulator-max-microvolt = <5825000>;
                    regulator-max-microvolt = <5825000>;
@@ -145,7 +144,6 @@ examples:
                compatible = "mediatek,mt6360-regulator";
                compatible = "mediatek,mt6360-regulator";
                LDO_VIN3-supply = <&BUCK2>;
                LDO_VIN3-supply = <&BUCK2>;
                buck1 {
                buck1 {
                    regulator-compatible = "BUCK1";
                    regulator-name = "mt6360,buck1";
                    regulator-name = "mt6360,buck1";
                    regulator-min-microvolt = <300000>;
                    regulator-min-microvolt = <300000>;
                    regulator-max-microvolt = <1300000>;
                    regulator-max-microvolt = <1300000>;
@@ -154,7 +152,6 @@ examples:
                         MT6360_OPMODE_ULP>;
                         MT6360_OPMODE_ULP>;
                };
                };
                BUCK2: buck2 {
                BUCK2: buck2 {
                    regulator-compatible = "BUCK2";
                    regulator-name = "mt6360,buck2";
                    regulator-name = "mt6360,buck2";
                    regulator-min-microvolt = <300000>;
                    regulator-min-microvolt = <300000>;
                    regulator-max-microvolt = <1300000>;
                    regulator-max-microvolt = <1300000>;
@@ -163,7 +160,6 @@ examples:
                         MT6360_OPMODE_ULP>;
                         MT6360_OPMODE_ULP>;
                };
                };
                ldo6 {
                ldo6 {
                    regulator-compatible = "LDO6";
                    regulator-name = "mt6360,ldo6";
                    regulator-name = "mt6360,ldo6";
                    regulator-min-microvolt = <500000>;
                    regulator-min-microvolt = <500000>;
                    regulator-max-microvolt = <2100000>;
                    regulator-max-microvolt = <2100000>;
@@ -171,7 +167,6 @@ examples:
                         MT6360_OPMODE_LP>;
                         MT6360_OPMODE_LP>;
                };
                };
                ldo7 {
                ldo7 {
                    regulator-compatible = "LDO7";
                    regulator-name = "mt6360,ldo7";
                    regulator-name = "mt6360,ldo7";
                    regulator-min-microvolt = <500000>;
                    regulator-min-microvolt = <500000>;
                    regulator-max-microvolt = <2100000>;
                    regulator-max-microvolt = <2100000>;
@@ -179,7 +174,6 @@ examples:
                         MT6360_OPMODE_LP>;
                         MT6360_OPMODE_LP>;
                };
                };
                ldo1 {
                ldo1 {
                    regulator-compatible = "LDO1";
                    regulator-name = "mt6360,ldo1";
                    regulator-name = "mt6360,ldo1";
                    regulator-min-microvolt = <1200000>;
                    regulator-min-microvolt = <1200000>;
                    regulator-max-microvolt = <3600000>;
                    regulator-max-microvolt = <3600000>;
@@ -187,7 +181,6 @@ examples:
                         MT6360_OPMODE_LP>;
                         MT6360_OPMODE_LP>;
                };
                };
                ldo2 {
                ldo2 {
                    regulator-compatible = "LDO2";
                    regulator-name = "mt6360,ldo2";
                    regulator-name = "mt6360,ldo2";
                    regulator-min-microvolt = <1200000>;
                    regulator-min-microvolt = <1200000>;
                    regulator-max-microvolt = <3600000>;
                    regulator-max-microvolt = <3600000>;
@@ -195,7 +188,6 @@ examples:
                         MT6360_OPMODE_LP>;
                         MT6360_OPMODE_LP>;
                };
                };
                ldo3 {
                ldo3 {
                    regulator-compatible = "LDO3";
                    regulator-name = "mt6360,ldo3";
                    regulator-name = "mt6360,ldo3";
                    regulator-min-microvolt = <1200000>;
                    regulator-min-microvolt = <1200000>;
                    regulator-max-microvolt = <3600000>;
                    regulator-max-microvolt = <3600000>;
@@ -203,7 +195,6 @@ examples:
                         MT6360_OPMODE_LP>;
                         MT6360_OPMODE_LP>;
                };
                };
                ldo5 {
                ldo5 {
                    regulator-compatible = "LDO5";
                    regulator-name = "mt6360,ldo5";
                    regulator-name = "mt6360,ldo5";
                    regulator-min-microvolt = <2700000>;
                    regulator-min-microvolt = <2700000>;
                    regulator-max-microvolt = <3600000>;
                    regulator-max-microvolt = <3600000>;
+1 −1
Original line number Original line Diff line number Diff line
@@ -26,6 +26,7 @@ properties:
    type: object
    type: object
    description: OTG boost regulator.
    description: OTG boost regulator.
    $ref: /schemas/regulator/regulator.yaml#
    $ref: /schemas/regulator/regulator.yaml#
    unevaluatedProperties: false


required:
required:
  - compatible
  - compatible
@@ -39,7 +40,6 @@ examples:
      richtek,vinovp-microvolt = <14500000>;
      richtek,vinovp-microvolt = <14500000>;


      otg_vbus_regulator: usb-otg-vbus-regulator {
      otg_vbus_regulator: usb-otg-vbus-regulator {
        regulator-compatible = "usb-otg-vbus";
        regulator-name = "usb-otg-vbus";
        regulator-name = "usb-otg-vbus";
        regulator-min-microvolt = <4425000>;
        regulator-min-microvolt = <4425000>;
        regulator-max-microvolt = <5825000>;
        regulator-max-microvolt = <5825000>;
+1 −0
Original line number Original line Diff line number Diff line
@@ -26,6 +26,7 @@ properties:
patternProperties:
patternProperties:
  "^regulator-(ldo|sbb[0-2])$":
  "^regulator-(ldo|sbb[0-2])$":
    $ref: "regulator.yaml#"
    $ref: "regulator.yaml#"
    unevaluatedProperties: false


required:
required:
  - compatible
  - compatible
+1 −5
Original line number Original line Diff line number Diff line
@@ -26,6 +26,7 @@ properties:
    patternProperties:
    patternProperties:
      "regulator-.+":
      "regulator-.+":
        $ref: "regulator.yaml#"
        $ref: "regulator.yaml#"
        unevaluatedProperties: false


    additionalProperties: false
    additionalProperties: false


@@ -43,31 +44,26 @@ examples:


        regulators {
        regulators {
          regulator-V3 {
          regulator-V3 {
            regulator-compatible= "V3(DCDC)";
            regulator-min-microvolt = <725000>;
            regulator-min-microvolt = <725000>;
            regulator-max-microvolt = <1800000>;
            regulator-max-microvolt = <1800000>;
          };
          };


          regulator-V4 {
          regulator-V4 {
            regulator-compatible= "V4(DCDC)";
            regulator-min-microvolt = <725000>;
            regulator-min-microvolt = <725000>;
            regulator-max-microvolt = <1800000>;
            regulator-max-microvolt = <1800000>;
          };
          };


          regulator-V5 {
          regulator-V5 {
            regulator-compatible= "V5(LDO)";
            regulator-min-microvolt = <1700000>;
            regulator-min-microvolt = <1700000>;
            regulator-max-microvolt = <2000000>;
            regulator-max-microvolt = <2000000>;
          };
          };


          regulator-V6 {
          regulator-V6 {
            regulator-compatible= "V6(LDO)";
            regulator-min-microvolt = <1800000>;
            regulator-min-microvolt = <1800000>;
            regulator-max-microvolt = <3300000>;
            regulator-max-microvolt = <3300000>;
          };
          };


          regulator-V7 {
          regulator-V7 {
            regulator-compatible= "V7(LDO)";
            regulator-min-microvolt = <1800000>;
            regulator-min-microvolt = <1800000>;
            regulator-max-microvolt = <3300000>;
            regulator-max-microvolt = <3300000>;
          };
          };
Loading