From beb4fe27998936c069c9b52b89ad65c4c697fc46 Mon Sep 17 00:00:00 2001 From: Chen-Yu Tsai Date: Thu, 14 May 2026 17:15:16 +0800 Subject: [PATCH] regulator: dt-bindings: mt6359: Deprecate bogus vcn33_[12]_* split regulators vcn33_[12]_bt and vcn33_[12]_wifi refer to the same output. There are two enable bits in the registers so that BT and WiFi drivers can toggle them separately without any coordination. If either bit is set, then the regulator output is enabled. Deprecate the existing regulators, and add proper regulators matching the outputs: vcn33_1 and vcn33_2. Signed-off-by: Chen-Yu Tsai Reviewed-by: Krzysztof Kozlowski Link: https://patch.msgid.link/20260514091520.2718987-4-wenst@chromium.org Signed-off-by: Mark Brown --- .../bindings/regulator/mt6359-regulator.yaml | 29 +++++++++---------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/Documentation/devicetree/bindings/regulator/mt6359-regulator.yaml b/Documentation/devicetree/bindings/regulator/mt6359-regulator.yaml index ac925334ae83..e3c20412c8a7 100644 --- a/Documentation/devicetree/bindings/regulator/mt6359-regulator.yaml +++ b/Documentation/devicetree/bindings/regulator/mt6359-regulator.yaml @@ -30,7 +30,7 @@ patternProperties: $ref: regulator.yaml# unevaluatedProperties: false - "^ldo_vcn(18|13|33_1_bt|13_1_wifi|33_2_bt|33_2_wifi)$": + "^ldo_vcn(18|13|33_[12])$": type: object $ref: regulator.yaml# unevaluatedProperties: false @@ -55,6 +55,15 @@ patternProperties: $ref: regulator.yaml# unevaluatedProperties: false + "^ldo_vcn33_[12]_(bt|wifi)$": + type: object + $ref: regulator.yaml# + description: + vcn33_[12]_(bt|wifi) are incorrect representations. + Use vcn33_[12] instead. + deprecated: true + unevaluatedProperties: false + additionalProperties: false examples: @@ -203,13 +212,8 @@ examples: regulator-min-microvolt = <900000>; regulator-max-microvolt = <1300000>; }; - mt6359_vcn33_1_bt_ldo_reg: ldo_vcn33_1_bt { - regulator-name = "vcn33_1_bt"; - regulator-min-microvolt = <2800000>; - regulator-max-microvolt = <3500000>; - }; - mt6359_vcn33_1_wifi_ldo_reg: ldo_vcn33_1_wifi { - regulator-name = "vcn33_1_wifi"; + mt6359_vcn33_1_bt_ldo_reg: ldo_vcn33_1 { + regulator-name = "vcn33_1"; regulator-min-microvolt = <2800000>; regulator-max-microvolt = <3500000>; }; @@ -270,13 +274,8 @@ examples: regulator-min-microvolt = <2500000>; regulator-max-microvolt = <3300000>; }; - mt6359_vcn33_2_bt_ldo_reg: ldo_vcn33_2_bt { - regulator-name = "vcn33_2_bt"; - regulator-min-microvolt = <2800000>; - regulator-max-microvolt = <3500000>; - }; - mt6359_vcn33_2_wifi_ldo_reg: ldo_vcn33_2_wifi { - regulator-name = "vcn33_2_wifi"; + mt6359_vcn33_2_bt_ldo_reg: ldo_vcn33_2 { + regulator-name = "vcn33_2"; regulator-min-microvolt = <2800000>; regulator-max-microvolt = <3500000>; };