> From: Francesco Dolcini <francesco@xxxxxxxxxx>
> Sent: Wednesday, December 6, 2023 4:01 AM
> To: David Lin <yu-hao.lin@xxxxxxx>
> Cc: linux-wireless@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> briannorris@xxxxxxxxxxxx; kvalo@xxxxxxxxxx; francesco@xxxxxxxxxx; Pete
> Hsieh <tsung-hsien.hsieh@xxxxxxx>
> Subject: [EXT] Re: [PATCH] wifi: mwifiex: added extra delay for firmware ready.
>
> Caution: This is an external email. Please take care when clicking links or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
>
>
> Hello David,
>
> thanks for your patch. Planning to run some test on this over the next days and
> we'll provide some actual feedback if this really solves the issue we are
> affected by.
>
> Just a couple of nitpicky comments on the actual patch.
>
>
> On the commit message you should use imperative mood, e.g.
>
> `wifi: mwifiex: add extra delay for firmware ready`
>
> with no period at the end of the line.
>
Thanks. I will fix it in patch v2.
> On Tue, Nov 28, 2023 at 04:25:44PM +0800, David Lin wrote:
> > For SDIO IW416, in a corner case FW may return ready before complete
> > full initialization.
> > Command timeout may occur at driver load after reboot.
> > Workaround by adding 100ms delay at checking FW status.
> >
> > Signed-off-by: David Lin <yu-hao.lin@xxxxxxx>
>
> Add
>
> Cc: stable@...
>
Thanks. I will add "cc: stable" in patch v2.
> > ---
> > drivers/net/wireless/marvell/mwifiex/sdio.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/net/wireless/marvell/mwifiex/sdio.c
> > b/drivers/net/wireless/marvell/mwifiex/sdio.c
> > index 6462a0ffe698..744e9403430a 100644
> > --- a/drivers/net/wireless/marvell/mwifiex/sdio.c
> > +++ b/drivers/net/wireless/marvell/mwifiex/sdio.c
> > @@ -783,6 +783,9 @@ static int mwifiex_check_fw_status(struct
> mwifiex_adapter *adapter,
> > ret = -1;
> > }
> >
> > + if (!ret)
> > + msleep(100);
> > +
>
> you could just add the delay after
>
> if (firmware_stat == FIRMWARE_READY_SDIO) {
>
> this would be more read-able to me. Adding also a short comment like
>
> /* Firmware might pretend to be ready, when it's not.
> * Wait a little bit more as a workaround */
>
O.K.
>
> Francesco
