Messages in this thread Patch in this message | | | Date | Sat, 21 Oct 2023 14:56:25 +0200 | From | Francesco Dolcini <> | Subject | Re: [PATCH] wifi: mwifiex: added delay after firmware downloaded. |
| |
Hello David, thanks for your patch.
On Fri, Oct 20, 2023 at 03:35:34AM +0000, David Lin wrote: > Added 100 ms delayed after firmware is downloaded and active in order to > avoid command timeout for the first command sent to firmware.
Please have a look at Documentation/process/submitting-patches.rst. In very brief: - you should use imperative mood to describe your changes - this is fixing a real issue, I believe this is worth a more complete commit message, including a log of the failure this is solving.
I am also inclined to think that this change should be backported and it likely would need a Fixes: tag.
> --- > drivers/net/wireless/marvell/mwifiex/main.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/wireless/marvell/mwifiex/main.c b/drivers/net/wireless/marvell/mwifiex/main.c > index d99127dc466e..a25e8bb8daec 100644 > --- a/drivers/net/wireless/marvell/mwifiex/main.c > +++ b/drivers/net/wireless/marvell/mwifiex/main.c > @@ -570,6 +570,7 @@ static int _mwifiex_fw_dpc(const struct firmware *firmware, void *context) > goto err_dnld_fw; > > mwifiex_dbg(adapter, MSG, "WLAN FW is active\n"); > + mdelay(100);
from my understanding there is already a function polling for the firmware to be active, e.g. mwifiex_check_fw_status(). Maybe is that function that needs to be fixed instead of adding an additional delay here?
Francesco
| |