[v3] arm64: dts: mediatek: mt6358: Merge ldo_vcn33_* regulators

Message ID 20230913084457.1638690-1-wenst@chromium.org
State New
Headers
Series [v3] arm64: dts: mediatek: mt6358: Merge ldo_vcn33_* regulators |

Commit Message

Chen-Yu Tsai Sept. 13, 2023, 8:44 a.m. UTC
  The ldo_vcn33_bt and ldo_vcn33_wifi regulators are actually the same
regulator, having the same voltage setting and output pin. There are
simply two enable bits that are ORed together to enable the regulator.

Having two regulators representing the same output pin is misleading
from a design matching standpoint, and also error-prone in driver
implementations.

Now that the bindings have these two merged, merge them in the device
tree as well. Neither vcn33 regulators are referenced in upstream
device trees. As far as hardware designs go, none of the Chromebooks
using MT8183 w/ MT6358 use this output.

Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
This was part of the series "regulator: mt6358: Remove bogus regulators
and improvements". Since the driver parts related to this patch have all
been picked up, so I'm sending this separately.

 arch/arm64/boot/dts/mediatek/mt6358.dtsi | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)
  

Comments

Chen-Yu Tsai Nov. 27, 2023, 3:07 a.m. UTC | #1
Hi Matthias, Angelo,

On Wed, Sep 13, 2023 at 4:45 PM Chen-Yu Tsai <wenst@chromium.org> wrote:
>
> The ldo_vcn33_bt and ldo_vcn33_wifi regulators are actually the same
> regulator, having the same voltage setting and output pin. There are
> simply two enable bits that are ORed together to enable the regulator.
>
> Having two regulators representing the same output pin is misleading
> from a design matching standpoint, and also error-prone in driver
> implementations.
>
> Now that the bindings have these two merged, merge them in the device
> tree as well. Neither vcn33 regulators are referenced in upstream
> device trees. As far as hardware designs go, none of the Chromebooks
> using MT8183 w/ MT6358 use this output.
>
> Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

Please remember to queue this up. All the driver changes were merged in
the last cycle.

ChenYu

> ---
> This was part of the series "regulator: mt6358: Remove bogus regulators
> and improvements". Since the driver parts related to this patch have all
> been picked up, so I'm sending this separately.
>
>  arch/arm64/boot/dts/mediatek/mt6358.dtsi | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt6358.dtsi b/arch/arm64/boot/dts/mediatek/mt6358.dtsi
> index 96ba8490b629..ad5e4022e06d 100644
> --- a/arch/arm64/boot/dts/mediatek/mt6358.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt6358.dtsi
> @@ -292,15 +292,8 @@ mt6358_vrf18_reg: ldo_vrf18 {
>                                 regulator-enable-ramp-delay = <120>;
>                         };
>
> -                       mt6358_vcn33_bt_reg: ldo_vcn33_bt {
> -                               regulator-name = "vcn33_bt";
> -                               regulator-min-microvolt = <3300000>;
> -                               regulator-max-microvolt = <3500000>;
> -                               regulator-enable-ramp-delay = <270>;
> -                       };
> -
> -                       mt6358_vcn33_wifi_reg: ldo_vcn33_wifi {
> -                               regulator-name = "vcn33_wifi";
> +                       mt6358_vcn33_reg: ldo_vcn33 {
> +                               regulator-name = "vcn33";
>                                 regulator-min-microvolt = <3300000>;
>                                 regulator-max-microvolt = <3500000>;
>                                 regulator-enable-ramp-delay = <270>;
> --
> 2.42.0.283.g2d96d420d3-goog
>
  
AngeloGioacchino Del Regno Nov. 27, 2023, 12:57 p.m. UTC | #2
On Wed, 13 Sep 2023 16:44:56 +0800, Chen-Yu Tsai wrote:
 > The ldo_vcn33_bt and ldo_vcn33_wifi regulators are actually the same
 > regulator, having the same voltage setting and output pin. There are
 > simply two enable bits that are ORed together to enable the regulator.
 >
 > Having two regulators representing the same output pin is misleading
 > from a design matching standpoint, and also error-prone in driver
 > implementations.
 >
 > [...]

Applied, thanks!

[1/1] arm64: dts: mediatek: mt6358: Merge ldo_vcn33_* regulators
       commit: 2a19d6609722b40d0f44337d56ee75a69a69aeab

Regards,
Angelo
  

Patch

diff --git a/arch/arm64/boot/dts/mediatek/mt6358.dtsi b/arch/arm64/boot/dts/mediatek/mt6358.dtsi
index 96ba8490b629..ad5e4022e06d 100644
--- a/arch/arm64/boot/dts/mediatek/mt6358.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt6358.dtsi
@@ -292,15 +292,8 @@  mt6358_vrf18_reg: ldo_vrf18 {
 				regulator-enable-ramp-delay = <120>;
 			};
 
-			mt6358_vcn33_bt_reg: ldo_vcn33_bt {
-				regulator-name = "vcn33_bt";
-				regulator-min-microvolt = <3300000>;
-				regulator-max-microvolt = <3500000>;
-				regulator-enable-ramp-delay = <270>;
-			};
-
-			mt6358_vcn33_wifi_reg: ldo_vcn33_wifi {
-				regulator-name = "vcn33_wifi";
+			mt6358_vcn33_reg: ldo_vcn33 {
+				regulator-name = "vcn33";
 				regulator-min-microvolt = <3300000>;
 				regulator-max-microvolt = <3500000>;
 				regulator-enable-ramp-delay = <270>;