[PATCH] arm64: dts: imx8mp-verdin: drop limit to sdio wi-fi frequency to 100 mhz

[PATCH] arm64: dts: imx8mp-verdin: drop limit to sdio wi-fi frequency to 100 mhz

[João Paulo Gonçalves]

From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

SDIO frequency was limited to workaround some instabilities, however it
turned out that the instability had nothing to do with the speed but was
because of an issue with the USDHC IP that was fixed in
commit 52e4c32baed2 ("mmc: sdhci-esdhc-imx: only enable DAT[0] and CMD line
auto tuning for SDIO device").

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Signed-off-by: João Paulo Gonçalves <joao.goncalves@toradex.com>
---
 arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
index efcab00c0142..cae06cb67cd3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
@@ -75,7 +75,6 @@ bluetooth {
 &usdhc1 {
 	bus-width = <4>;
 	keep-power-in-suspend;
-	max-frequency = <100000000>;
 	non-removable;
 	pinctrl-names = "default", "state_100mhz", "state_200mhz";
 	pinctrl-0 = <&pinctrl_usdhc1>, <&pinctrl_wifi_ctrl>;
-- 
2.34.1

RE: [PATCH] arm64: dts: imx8mp-verdin: drop limit to sdio wi-fi frequency to 100 mhz

[Peng Fan]

> Subject: [PATCH] arm64: dts: imx8mp-verdin: drop limit to sdio wi-fi
> frequency to 100 mhz
> 
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> SDIO frequency was limited to workaround some instabilities, however
> it turned out that the instability had nothing to do with the speed but
> was because of an issue with the USDHC IP that was fixed in commit
> 52e4c32baed2 ("mmc: sdhci-esdhc-imx: only enable DAT[0] and CMD
> line auto tuning for SDIO device").
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> Signed-off-by: João Paulo Gonçalves <joao.goncalves@toradex.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
> b/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
> index efcab00c0142..cae06cb67cd3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin-wifi.dtsi
> @@ -75,7 +75,6 @@ bluetooth {
>  &usdhc1 {
>  	bus-width = <4>;
>  	keep-power-in-suspend;
> -	max-frequency = <100000000>;
>  	non-removable;
>  	pinctrl-names = "default", "state_100mhz", "state_200mhz";
>  	pinctrl-0 = <&pinctrl_usdhc1>, <&pinctrl_wifi_ctrl>;
> --
> 2.34.1
> 
Reviewed-by: Peng Fan <peng.fan@nxp.com>

Re: [PATCH] arm64: dts: imx8mp-verdin: drop limit to sdio wi-fi frequency to 100 mhz

[Shawn Guo]

On Mon, Aug 05, 2024 at 02:48:14PM -0300, João Paulo Gonçalves wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> SDIO frequency was limited to workaround some instabilities, however it
> turned out that the instability had nothing to do with the speed but was
> because of an issue with the USDHC IP that was fixed in
> commit 52e4c32baed2 ("mmc: sdhci-esdhc-imx: only enable DAT[0] and CMD line
> auto tuning for SDIO device").
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> Signed-off-by: João Paulo Gonçalves <joao.goncalves@toradex.com>

Applied, thanks!