Re: [PATCH 12/12] wifi: mwifiex: drop asynchronous init waiting code

[Sascha Hauer <s.hauer@pengutronix.de>]

On Mon, Sep 09, 2024 at 07:14:11PM +0200, Francesco Dolcini wrote:
> On Mon, Aug 26, 2024 at 01:01:33PM +0200, Sascha Hauer wrote:
> > Historically all commands sent to the mwifiex driver have been
> > asynchronous. For this reason there is code that waits for the
> > last initialization command to complete before going on. Nowadays the
> > commands can be sent synchronously, meaning that they are completed
> > when the command call returns. This makes all the waiting code
> > unnecessary. It is removed in this patch.
> 
> I am not sure to understand this. Is the code to have asynchronous command gone
> or it is just not used anymore? In the code here you remove waiting for the
> firmware init to be complete, but from the patch is not clear why this is not
> needed anymore.  Maybe a specific commit you can reference in which such
> support was removed?

Commands can still be sent asynchronously by passing sync=false to
mwifiex_send_cmd(), but this is no longer done in the initialization
phase as of:

| commit 7bff9c974e1a70819c30c37d8ec0d84d456f8237
| Author: Stone Piao <piaoyun@marvell.com>
| Date:   Tue Sep 25 20:23:39 2012 -0700
| 
|     mwifiex: send firmware initialization commands synchronously
| 
|     The driver will send some commands to firmware during the
|     initialization. Currently these commands are sent asynchronously,
|     which means that we firstly insert all of them to a pre-allocated
|     command queue, and then start to process them one by one. The
|     command queue will soon be exhausted if we keep adding new
|     initialization commands.
| 
|     This issue can be resolved by sending initialization commands
|     synchronously because each command is consumed and the buffer is
|     recycled before queuing next command.
| 
|     Signed-off-by: Stone Piao <piaoyun@marvell.com>
|     Signed-off-by: Bing Zhao <bzhao@marvell.com>
|     Signed-off-by: John W. Linville <linville@tuxdriver.com>

I'll mention this commit in the commit message next round.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |