[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AS8PR04MB8404CFE51B0A3ABEA13808569213A@AS8PR04MB8404.eurprd04.prod.outlook.com>
Date: Thu, 10 Aug 2023 06:17:51 +0000
From: Sherry Sun <sherry.sun@....com>
To: Guido Günther <agx@...xcpu.org>,
Ulf Hansson <ulf.hansson@...aro.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
dl-linux-imx <linux-imx@....com>,
Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will@...nel.org>, Peng Fan <peng.fan@....com>,
Bjorn Andersson <quic_bjorande@...cinc.com>,
Arnd Bergmann <arnd@...db.de>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Konrad Dybcio <konrad.dybcio@...aro.org>,
Nícolas F. R. A. Prado
<nfraprado@...labora.com>,
Rafał Miłecki <rafal@...ecki.pl>,
"kernel@...i.sm" <kernel@...i.sm>,
"linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
David Heidelberg <david@...t.cz>,
Bough Chen <haibo.chen@....com>
Subject: RE: [PATCH v1 4/5] arm64: dts: imx8mq-librem5-devkit: Drop
power-supply
> -----Original Message-----
> From: Guido Günther <agx@...xcpu.org>
> Sent: 2023年8月10日 2:50
> To: Ulf Hansson <ulf.hansson@...aro.org>; Rob Herring
> <robh+dt@...nel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@...aro.org>; Conor Dooley <conor+dt@...nel.org>;
> Liam Girdwood <lgirdwood@...il.com>; Mark Brown
> <broonie@...nel.org>; Shawn Guo <shawnguo@...nel.org>; Sascha Hauer
> <s.hauer@...gutronix.de>; Pengutronix Kernel Team
> <kernel@...gutronix.de>; Fabio Estevam <festevam@...il.com>; dl-linux-
> imx <linux-imx@....com>; Catalin Marinas <catalin.marinas@....com>;
> Will Deacon <will@...nel.org>; Guido Günther <agx@...xcpu.org>; Peng Fan
> <peng.fan@....com>; Bjorn Andersson <quic_bjorande@...cinc.com>;
> Arnd Bergmann <arnd@...db.de>; Geert Uytterhoeven
> <geert+renesas@...der.be>; Konrad Dybcio <konrad.dybcio@...aro.org>;
> Nícolas F. R. A. Prado <nfraprado@...labora.com>; Rafał Miłecki
> <rafal@...ecki.pl>; kernel@...i.sm; linux-mmc@...r.kernel.org;
> devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; alsa-devel@...a-
> project.org; linux-arm-kernel@...ts.infradead.org; David Heidelberg
> <david@...t.cz>
> Subject: [PATCH v1 4/5] arm64: dts: imx8mq-librem5-devkit: Drop power-
> supply
>
> Use a power sequence instead. As this handles the wifi disable line rename
> the pincontrol accordingly. This also makes it similar to the Librem 5.
>
> Resolves following warning:
>
> arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dtb:
> mmc@...50000: Unevaluated properties are not allowed ('power-supply'
> was unexpected)
> from schema $id:
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicet
> ree.org%2Fschemas%2Fmmc%2Ffsl-imx-
> esdhc.yaml%23&data=05%7C01%7Csherry.sun%40nxp.com%7Cfefefef5326f
> 4205167c08db990988d7%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0
> %7C638272038447289369%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLj
> AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%
> 7C%7C&sdata=EpcdsbWNkU0%2FuYBx9tHlw9qoSo87XneMNxOHw5Z4%2FFA
> %3D&reserved=0
>
> Reported-by: David Heidelberg <david@...t.cz>
> Signed-off-by: Guido Günther <agx@...xcpu.org>
> ---
> .../boot/dts/freescale/imx8mq-librem5-devkit.dts | 13 ++++---------
> 1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
> b/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
> index be9ef5c271df..244075e2a599 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5-devkit.dts
> @@ -231,16 +231,11 @@ vibrator {
> vcc-supply = <®_3v3_p>;
> };
>
> - wifi_pwr_en: regulator-wifi-en {
> - compatible = "regulator-fixed";
> + wifi_pwrseq: pwrseq {
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_wifi_pwr_en>;
> - regulator-name = "WIFI_EN";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> - gpio = <&gpio3 5 GPIO_ACTIVE_HIGH>;
> - enable-active-high;
> - regulator-always-on;
> + compatible = "mmc-pwrseq-simple";
> + reset-gpios = <&gpio3 5 GPIO_ACTIVE_LOW>;
> };
> };
>
> @@ -1032,7 +1027,7 @@ &usdhc2 {
> pinctrl-2 = <&pinctrl_usdhc2_200mhz>;
> bus-width = <4>;
> vmmc-supply = <®_usdhc2_vmmc>;
> - power-supply = <&wifi_pwr_en>;
> + mmc-pwrseq = <&wifi_pwrseq>;
Hi Guido,
Normally we should use vmmc-supply to control the M.2 pin56(W_DISABLE1#), which controls the WLAN radio power.
And use the reset-gpios of mmc-pwrseq to control the M.2 pin23(SDIO_RESET#), it can independently reset the Wi-Fi radio.
Best Regards
Sherry
Powered by blists - more mailing lists