Re: [PATCH v1] arm64: dts: qcom: Add fixed regulators for WLAN supplies

[Shawn Guo <shengchao.guo@oss.qualcomm.com>]

On Fri, May 01, 2026 at 01:19:18PM +0800, Qian Zhang wrote:
> Add GPIO-controlled fixed regulators for the WLAN power rails on the
> Arduino VENTUNO Q board:
> 
>   - wlan_reg_3v3: 3.3V supply controlled by GPIO54 (wlan_rf_kill)
>   - wlan_reg_1v8: 1.8V supply controlled by GPIO56 (wlan_en)
> 
> Both regulators are enabled at boot to support WLAN initialization.
> 
> Signed-off-by: Qian Zhang <qian.zhang@oss.qualcomm.com>

Nit: you might want to have subject prefix like below to tell the scope
of the change?

  arm64: dts: qcom: arduino-monza: 

> ---
> Changes in v1:

Patch version numbering generally starts from 1 rather than 0. You have
a particular reason for numbering from 0?

Shawn

>   - Clarified GPIO roles: GPIO54 for wlan_rf_kill, GPIO56 for wlan_en
>   - Improved commit message readability with bullet list format
> 
>  .../boot/dts/qcom/monaco-arduino-monza.dts    | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/monaco-arduino-monza.dts b/arch/arm64/boot/dts/qcom/monaco-arduino-monza.dts
> index ca14f0ea4dae..1a40ac5bb4bb 100644
> --- a/arch/arm64/boot/dts/qcom/monaco-arduino-monza.dts
> +++ b/arch/arm64/boot/dts/qcom/monaco-arduino-monza.dts
> @@ -153,6 +153,27 @@ vreg_nvme: regulator-3p3-m2 {
>  		enable-active-high;
>  		startup-delay-us = <20000>;
>  	};
> +
> +	wlan_reg_3v3: wlan-3v3-regulator {
> +		compatible = "regulator-fixed";
> +		regulator-name = "wlan_3v3";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		gpio = <&tlmm 54 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +		regulator-boot-on;
> +	};
> +
> +	wlan_reg_1v8: wlan-1v8-regulator {
> +		compatible = "regulator-fixed";
> +		regulator-name = "wlan_1v8";
> +		regulator-min-microvolt = <1800000>;
> +		regulator-max-microvolt = <1800000>;
> +		gpio = <&tlmm 56 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +		regulator-boot-on;
> +	};
> +
>  };
>  
>  &ethernet0 {
> -- 
> 2.34.1
> 
>