[PATCH v2 0/4] call rtw_IOL_append_*_cmd directly

daniel watson posted 4 patches 3 years, 10 months ago
There is a newer version of this series
drivers/staging/r8188eu/core/rtw_iol.c    |  8 ++++----
drivers/staging/r8188eu/include/rtw_iol.h | 24 ++++++++---------------
2 files changed, 12 insertions(+), 20 deletions(-)
[PATCH v2 0/4] call rtw_IOL_append_*_cmd directly
Posted by daniel watson 3 years, 10 months ago
From: Daniel Watson <ozzloy@challenge-bot.com>

This patchset calls the rtw_IOL_append_*_cmd group of functions
directly and removes the #define wrappers.

Daniel Watson (4):
  staging: r8188eu: call rtw_IOL_append_WB_cmd directly
  staging: r8188eu: call rtw_IOL_append_WW_cmd directly
  staging: r8188eu: call rtw_IOL_append_WD_cmd directly
  staging: r8188eu: call rtw_IOL_append_WRF_cmd directly

 drivers/staging/r8188eu/core/rtw_iol.c    |  8 ++++----
 drivers/staging/r8188eu/include/rtw_iol.h | 24 ++++++++---------------
 2 files changed, 12 insertions(+), 20 deletions(-)

-- 
2.34.1
Re: [PATCH v2 0/4] call rtw_IOL_append_*_cmd directly
Posted by Dan Carpenter 3 years, 10 months ago
On Tue, Jun 14, 2022 at 05:49:24PM -0700, daniel watson wrote:
> From: Daniel Watson <ozzloy@challenge-bot.com>
> 
> This patchset calls the rtw_IOL_append_*_cmd group of functions
> directly and removes the #define wrappers.
> 
> Daniel Watson (4):
>   staging: r8188eu: call rtw_IOL_append_WB_cmd directly
>   staging: r8188eu: call rtw_IOL_append_WW_cmd directly
>   staging: r8188eu: call rtw_IOL_append_WD_cmd directly
>   staging: r8188eu: call rtw_IOL_append_WRF_cmd directly
> 

Looks good, thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>

regards,
dan carpenter
Re: [PATCH v2 0/4] call rtw_IOL_append_*_cmd directly
Posted by daniel watson 3 years, 10 months ago
On Wed, Jun 15, 2022 at 11:37:15AM +0300, Dan Carpenter wrote:
> On Tue, Jun 14, 2022 at 05:49:24PM -0700, daniel watson wrote:
> > From: Daniel Watson <ozzloy@challenge-bot.com>
> > 
> > This patchset calls the rtw_IOL_append_*_cmd group of functions
> > directly and removes the #define wrappers.
> > 
> 
> Looks good, thanks!
> 
> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
> 
> regards,
> dan carpenter
>

Thanks!  I noticed a typo in the first commit message.
I'll send a v3 to fix it.