staging: r8188eu: clean up the OnDeAuth function

[PATCH 0/9] staging: r8188eu: clean up the OnDeAuth function

Posted by Martin Kaiser 3 years, 5 months ago

This series cleans up the OnDeAuth function and tries to replace
driver-specific parsing code with helpers from ieee80211.h.

Martin Kaiser (9):
  staging: r8188eu: replace one GetAddr3Ptr call
  staging: r8188eu: get reason code from mgmt struct
  staging: r8188eu: clarify the bBusyTraffic assignment
  staging: r8188eu: use sa instead of Addr2
  staging: r8188eu: get bssid from mgmt struct
  staging: r8188eu: exit for deauth from unknown station
  staging: r8188eu: remove unnecessary return
  staging: r8188eu: summarize two flags checks
  staging: r8188eu: ignore_received_deauth is a boolean

 drivers/staging/r8188eu/core/rtw_mlme_ext.c | 45 ++++++++++-----------
 1 file changed, 21 insertions(+), 24 deletions(-)

-- 
2.30.2

Re: [PATCH 0/9] staging: r8188eu: clean up the OnDeAuth function

Posted by Philipp Hortmann 3 years, 5 months ago

On 10/15/22 17:24, Martin Kaiser wrote:
> This series cleans up the OnDeAuth function and tries to replace
> driver-specific parsing code with helpers from ieee80211.h.
> 
> Martin Kaiser (9):
>    staging: r8188eu: replace one GetAddr3Ptr call
>    staging: r8188eu: get reason code from mgmt struct
>    staging: r8188eu: clarify the bBusyTraffic assignment
>    staging: r8188eu: use sa instead of Addr2
>    staging: r8188eu: get bssid from mgmt struct
>    staging: r8188eu: exit for deauth from unknown station
>    staging: r8188eu: remove unnecessary return
>    staging: r8188eu: summarize two flags checks
>    staging: r8188eu: ignore_received_deauth is a boolean
> 
>   drivers/staging/r8188eu/core/rtw_mlme_ext.c | 45 ++++++++++-----------
>   1 file changed, 21 insertions(+), 24 deletions(-)
> 

Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150

Re: [PATCH 0/9] staging: r8188eu: clean up the OnDeAuth function

Posted by Pavel Skripkin 3 years, 5 months ago

Hi Martin,

Martin Kaiser <martin@kaiser.cx> says:
> This series cleans up the OnDeAuth function and tries to replace
> driver-specific parsing code with helpers from ieee80211.h.
> 
> Martin Kaiser (9):
>    staging: r8188eu: replace one GetAddr3Ptr call
>    staging: r8188eu: get reason code from mgmt struct
>    staging: r8188eu: clarify the bBusyTraffic assignment
>    staging: r8188eu: use sa instead of Addr2
>    staging: r8188eu: get bssid from mgmt struct
>    staging: r8188eu: exit for deauth from unknown station
>    staging: r8188eu: remove unnecessary return
>    staging: r8188eu: summarize two flags checks
>    staging: r8188eu: ignore_received_deauth is a boolean
> 
>   drivers/staging/r8188eu/core/rtw_mlme_ext.c | 45 ++++++++++-----------
>   1 file changed, 21 insertions(+), 24 deletions(-)
> 

works for me, thanks

Acked-by: Pavel Skripkin <paskripkin@gmail.com>




With regards,
Pavel Skripkin