Re: [PATCH v2 14/15] arm64: dts: mediatek: mt7981b-openwrt-one: Enable wifi

[AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>]

Il 01/11/25 14:32, Sjoerd Simons ha scritto:
> Enable Dual-band WiFI 6 functionality on the Openwrt One
> 
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> Signed-off-by: Sjoerd Simons <sjoerd@collabora.com>
> ---
> V1 -> V2: Update eeprom node label
> ---
>   .../boot/dts/mediatek/mt7981b-openwrt-one.dts      | 24 ++++++++++++++++++++++
>   arch/arm64/boot/dts/mediatek/mt7981b.dtsi          |  2 +-
>   2 files changed, 25 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt7981b-openwrt-one.dts b/arch/arm64/boot/dts/mediatek/mt7981b-openwrt-one.dts
> index 90edb9f493c6d..b13f16d7816bf 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7981b-openwrt-one.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt7981b-openwrt-one.dts
> @@ -129,6 +129,22 @@ conf-pd {
>   			pins = "SPI2_CLK", "SPI2_MOSI", "SPI2_MISO";
>   		};
>   	};
> +
> +	wifi_dbdc_pins: wifi-dbdc-pins {
> +		mux {
> +			function = "eth";
> +			groups = "wf0_mode1";
> +		};
> +
> +		conf {
> +			pins = "WF_HB1", "WF_HB2", "WF_HB3", "WF_HB4",
> +			       "WF_HB0", "WF_HB0_B", "WF_HB5", "WF_HB6",
> +			       "WF_HB7", "WF_HB8", "WF_HB9", "WF_HB10",
> +			       "WF_TOP_CLK", "WF_TOP_DATA", "WF_XO_REQ",
> +			       "WF_CBA_RESETB", "WF_DIG_RESETB";
> +			drive-strength = <MTK_DRIVE_4mA>;

You forgot to address my comment here. drive-strength = <4>;

Regards,
Angelo

> +		};
> +	};
>   };
>   
>   &spi2 {
> @@ -200,6 +216,14 @@ &usb_phy {
>   	status = "okay";
>   };
>   
> +&wifi {
> +	nvmem-cells = <&wifi_factory_calibration>;
> +	nvmem-cell-names = "eeprom";
> +	pinctrl-names = "dbdc";
> +	pinctrl-0 = <&wifi_dbdc_pins>;
> +	status = "okay";
> +};
> +
>   &xhci {
>   	phys = <&u2port0 PHY_TYPE_USB2>;
>   	vusb33-supply = <&reg_3p3v>;
> diff --git a/arch/arm64/boot/dts/mediatek/mt7981b.dtsi b/arch/arm64/boot/dts/mediatek/mt7981b.dtsi
> index eb2effb3c1ed2..17dd13d4c0015 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7981b.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt7981b.dtsi
> @@ -490,7 +490,7 @@ wo_ccif0: syscon@151a5000 {
>   			interrupts = <GIC_SPI 211 IRQ_TYPE_LEVEL_HIGH>;
>   		};
>   
> -		wifi@18000000 {
> +		wifi: wifi@18000000 {
>   			compatible = "mediatek,mt7981-wmac";
>   			reg = <0 0x18000000 0 0x1000000>,
>   			      <0 0x10003000 0 0x1000>,
>