Re: [PATCH v2 7/7] arm64: dts: qcom: milos-fairphone-fp6: Enable WiFi

Dmitry Baryshkov
2026-04-03T19:35:13Z

On Fri, Apr 03, 2026 at 03:52:53PM +0200, Luca Weiss wrote:
> Configure and enable the WiFi node, and add the required pinctrl to
> provide the sleep clock from the PMK8550 (PMK7635) to WCN6755.
>
> Thanks to Alexander Koskovich for helping with the bringup, adding
> the missing pinctrl to make the WPSS stop crashing.
>
> Link: https://lore.kernel.org/linux-arm-msm/DBF7OWAWQ94M.FSCP4DPF8ZJY@fairphone.com/
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
> arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts | 19 ++++++++++++++++++-
> 1 file changed, 18 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts b/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts
> index db72418b7195..d8ac495ca7c8 100644
> --- a/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts
> +++ b/arch/arm64/boot/dts/qcom/milos-fairphone-fp6.dts
> @@ -242,7 +242,7 @@ wcn6750-pmu {
>
> clocks = <&rpmhcc RPMH_RF_CLK1>;
>
> - pinctrl-0 = <&bluetooth_enable_default>;
> + pinctrl-0 = <&bluetooth_enable_default>, <&pmk8550_sleep_clk_default>;
> pinctrl-names = "default";
>
> regulators {
> @@ -766,6 +766,17 @@ &pmiv0104_eusb2_repeater {
> qcom,tune-usb2-preem = /bits/ 8 <0x6>;
> };
>
> +&pmk8550_gpios {
> + pmk8550_sleep_clk_default: sleep-clk-default-state {
> + pins = "gpio5";
> + function = "func1";
> + input-disable;
> + output-enable;

Hmm, if it's a sleep_clk, should it not be handled via the power
sequencer?

> + bias-disable;
> + power-source = <0>;
> + };
> +};
> +
> &pmr735b_gpios {
> s1j_enable_default: s1j-enable-default-state {
> pins = "gpio1";
> @@ -1049,3 +1060,9 @@ &usb_1_hsphy {
>
> status = "okay";
> };
> +
> +&wifi {
> + qcom,calibration-variant = "Fairphone_Gen_6";
> +
> + status = "okay";
> +};
>
> --
> 2.53.0
>

--
With best wishes
Dmitry