[v1] staging: r8188eu: another round of cleanups

[PATCH 0/9] staging: r8188eu: another round of cleanups

Martin Kaiser posted 9 patches 3 years, 5 months ago

Diff against v1 v1 v1 v2
Download series mbox

There is a newer version of this series

drivers/staging/r8188eu/core/rtw_led.c        |  3 +-
drivers/staging/r8188eu/core/rtw_mlme.c       | 23 ++++++---------
drivers/staging/r8188eu/core/rtw_mlme_ext.c   | 28 ++++++++-----------
drivers/staging/r8188eu/core/rtw_recv.c       |  2 --
drivers/staging/r8188eu/hal/hal_intf.c        | 19 +++++--------
.../staging/r8188eu/hal/rtl8188e_hal_init.c   |  6 +---
drivers/staging/r8188eu/hal/rtl8188e_rxdesc.c | 15 ++++------
drivers/staging/r8188eu/include/rtw_mlme.h    |  7 -----
.../staging/r8188eu/include/rtw_mlme_ext.h    |  4 +--
drivers/staging/r8188eu/include/wifi.h        |  8 ------
10 files changed, 36 insertions(+), 79 deletions(-)

Expand all Fold all

[PATCH 0/9] staging: r8188eu: another round of cleanups

Posted by Martin Kaiser 3 years, 5 months ago

Here's another patch set with several cleanups throughout the driver.

Martin Kaiser (9):
  staging: r8188eu: don't store addba request
  staging: r8188eu: remove some obsolete comments
  staging: r8188eu: reorder assignments, clarify the header format
  staging: r8188eu: reformat a function header
  staging: r8188eu: remove state checks in rtw_led_control
  staging: r8188eu: clean up rtw_hal_init
  staging: r8188eu: remove get_fwstate
  staging: r8188eu: merge two rtw_free_network_nolock functions
  staging: r8188eu: remove checks in dump_mgntframe

 drivers/staging/r8188eu/core/rtw_led.c        |  3 +-
 drivers/staging/r8188eu/core/rtw_mlme.c       | 23 ++++++---------
 drivers/staging/r8188eu/core/rtw_mlme_ext.c   | 28 ++++++++-----------
 drivers/staging/r8188eu/core/rtw_recv.c       |  2 --
 drivers/staging/r8188eu/hal/hal_intf.c        | 19 +++++--------
 .../staging/r8188eu/hal/rtl8188e_hal_init.c   |  6 +---
 drivers/staging/r8188eu/hal/rtl8188e_rxdesc.c | 15 ++++------
 drivers/staging/r8188eu/include/rtw_mlme.h    |  7 -----
 .../staging/r8188eu/include/rtw_mlme_ext.h    |  4 +--
 drivers/staging/r8188eu/include/wifi.h        |  8 ------
 10 files changed, 36 insertions(+), 79 deletions(-)

-- 
2.30.2

Re: [PATCH 0/9] staging: r8188eu: another round of cleanups

Posted by Philipp Hortmann 3 years, 5 months ago

On 11/6/22 13:48, Martin Kaiser wrote:
> Here's another patch set with several cleanups throughout the driver.
> 
> Martin Kaiser (9):
>    staging: r8188eu: don't store addba request
>    staging: r8188eu: remove some obsolete comments
>    staging: r8188eu: reorder assignments, clarify the header format
>    staging: r8188eu: reformat a function header
>    staging: r8188eu: remove state checks in rtw_led_control
>    staging: r8188eu: clean up rtw_hal_init
>    staging: r8188eu: remove get_fwstate
>    staging: r8188eu: merge two rtw_free_network_nolock functions
>    staging: r8188eu: remove checks in dump_mgntframe
> 
>   drivers/staging/r8188eu/core/rtw_led.c        |  3 +-
>   drivers/staging/r8188eu/core/rtw_mlme.c       | 23 ++++++---------
>   drivers/staging/r8188eu/core/rtw_mlme_ext.c   | 28 ++++++++-----------
>   drivers/staging/r8188eu/core/rtw_recv.c       |  2 --
>   drivers/staging/r8188eu/hal/hal_intf.c        | 19 +++++--------
>   .../staging/r8188eu/hal/rtl8188e_hal_init.c   |  6 +---
>   drivers/staging/r8188eu/hal/rtl8188e_rxdesc.c | 15 ++++------
>   drivers/staging/r8188eu/include/rtw_mlme.h    |  7 -----
>   .../staging/r8188eu/include/rtw_mlme_ext.h    |  4 +--
>   drivers/staging/r8188eu/include/wifi.h        |  8 ------
>   10 files changed, 36 insertions(+), 79 deletions(-)
> 
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150