| Message ID | efc793c8-da17-e267-851e-f08f097843ee@web.de (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | [v2] arm64: dts: zcu100-revC: Give wifi some time after power-on | expand |
* Jan Kiszka <jan.kiszka@web.de> [190124 08:29]: > From: Jan Kiszka <jan.kiszka@siemens.com> > > Somewhere along recent changes to power control of the wl1831, power-on > became very unreliable on the Ultra96, failing like this: > > wl1271_sdio: probe of mmc2:0001:1 failed with error -16 > wl1271_sdio: probe of mmc2:0001:2 failed with error -16 > > After playing with some dt parameters and comparing to other users of > this chip, it turned out we need some power-on delay to make things > stable again. In contrast to those other users which define 200 ms, > Ultra96 is already happy with 10 ms. Good to hear this got sorted out, thanks everybody. Acked-by: Tony Lindgren <tony@atomide.com>
On 24. 01. 19 9:28, Jan Kiszka wrote: > From: Jan Kiszka <jan.kiszka@siemens.com> > > Somewhere along recent changes to power control of the wl1831, power-on > became very unreliable on the Ultra96, failing like this: > > wl1271_sdio: probe of mmc2:0001:1 failed with error -16 > wl1271_sdio: probe of mmc2:0001:2 failed with error -16 > > After playing with some dt parameters and comparing to other users of > this chip, it turned out we need some power-on delay to make things > stable again. In contrast to those other users which define 200 ms, > Ultra96 is already happy with 10 ms. > > Fixes: 5869ba0653b9 ("arm64: zynqmp: Add support for Xilinx zcu100-revC") > Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com> > Acked-by: Ulf Hansson <ulf.hansson@linaro.org> > --- > > Changes in v2: > - fix changelog > > arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts > index 13a0a028df98..e5699d0d91e4 100644 > --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts > +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts > @@ -101,6 +101,7 @@ > sdio_pwrseq: sdio-pwrseq { > compatible = "mmc-pwrseq-simple"; > reset-gpios = <&gpio 7 GPIO_ACTIVE_LOW>; /* WIFI_EN */ > + post-power-on-delay-ms = <10>; > }; > }; > > -- > 2.16.4 > Applied. Thanks, Michal
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts index 13a0a028df98..e5699d0d91e4 100644 --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts @@ -101,6 +101,7 @@ sdio_pwrseq: sdio-pwrseq { compatible = "mmc-pwrseq-simple"; reset-gpios = <&gpio 7 GPIO_ACTIVE_LOW>; /* WIFI_EN */ + post-power-on-delay-ms = <10>; }; };