Re: [PATCH] dt-bindings: Add missing 'unevaluatedProperties' to regulator nodes

Bartosz Golaszewski
2022-12-07T12:37:57Z

On Tue, Dec 6, 2022 at 10:18 PM Rob Herring <robh@kernel.org> wrote:
>
> 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: Rob Herring <robh@kernel.org>
> ---
> I'd suggest that Mark take this if that's okay with Lee. Or I can take
> it.
>
> Documentation/devicetree/bindings/mfd/max77650.yaml | 2 --
> .../devicetree/bindings/mfd/mediatek,mt6360.yaml | 9 ---------
> .../bindings/power/supply/mt6360_charger.yaml | 2 +-
> .../bindings/regulator/max77650-regulator.yaml | 1 +
> .../devicetree/bindings/regulator/max8660.yaml | 6 +-----
> .../bindings/regulator/mt6360-regulator.yaml | 12 +++---------
> 6 files changed, 6 insertions(+), 26 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/max77650.yaml b/Documentation/devicetree/bindings/mfd/max77650.yaml
> index b0a0f0d3d9d4..4181174fcf58 100644
> --- a/Documentation/devicetree/bindings/mfd/max77650.yaml
> +++ b/Documentation/devicetree/bindings/mfd/max77650.yaml
> @@ -100,14 +100,12 @@ examples:
> compatible = "maxim,max77650-regulator";
>
> max77650_ldo: regulator-ldo {
> - regulator-compatible = "ldo";
> regulator-name = "max77650-ldo";
> regulator-min-microvolt = <1350000>;
> regulator-max-microvolt = <2937500>;
> };
>
> max77650_sbb0: regulator-sbb0 {
> - regulator-compatible = "sbb0";
> regulator-name = "max77650-sbb0";
> regulator-min-microvolt = <800000>;
> regulator-max-microvolt = <1587500>;

Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>