[v1] staging: r8188eu: another round of xmit cleanups

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

Posted by Martin Kaiser 2 years, 7 months ago

This series contains some more cleanups for the xmit code.

Amongst other things, we can reduce the data for the completion function
of the usb bulk urbs that we send out.

Martin Kaiser (7):
  staging: r8188eu: merge do_queue_select into its only caller
  staging: r8188eu: simplify rtw_alloc_xmitframe
  staging: r8188eu: remove unused frametag defines
  staging: r8188eu: xmit_buf's ff_hwaddr is not used
  staging: r8188eu: simplify xmit_buf flags
  staging: r8188eu: simplify rtw_get_ff_hwaddr
  staging: r8188eu: bagg_pkt parameter is not used

 drivers/staging/r8188eu/core/rtw_xmit.c       | 85 +++++--------------
 drivers/staging/r8188eu/hal/rtl8188eu_xmit.c  |  8 +-
 drivers/staging/r8188eu/include/rtw_xmit.h    | 17 +---
 .../staging/r8188eu/os_dep/usb_ops_linux.c    | 25 +-----
 4 files changed, 27 insertions(+), 108 deletions(-)

-- 
2.30.2

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

Posted by Philipp Hortmann 2 years, 7 months ago

On 2/7/23 20:23, Martin Kaiser wrote:
> This series contains some more cleanups for the xmit code.
> 
> Amongst other things, we can reduce the data for the completion function
> of the usb bulk urbs that we send out.
> 
> Martin Kaiser (7):
>    staging: r8188eu: merge do_queue_select into its only caller
>    staging: r8188eu: simplify rtw_alloc_xmitframe
>    staging: r8188eu: remove unused frametag defines
>    staging: r8188eu: xmit_buf's ff_hwaddr is not used
>    staging: r8188eu: simplify xmit_buf flags
>    staging: r8188eu: simplify rtw_get_ff_hwaddr
>    staging: r8188eu: bagg_pkt parameter is not used
> 
>   drivers/staging/r8188eu/core/rtw_xmit.c       | 85 +++++--------------
>   drivers/staging/r8188eu/hal/rtl8188eu_xmit.c  |  8 +-
>   drivers/staging/r8188eu/include/rtw_xmit.h    | 17 +---
>   .../staging/r8188eu/os_dep/usb_ops_linux.c    | 25 +-----
>   4 files changed, 27 insertions(+), 108 deletions(-)
> 
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150