mbox series

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

Message ID 20221015152440.232281-1-martin@kaiser.cx
Headers
Series staging: r8188eu: clean up the OnDeAuth function |

Message

Martin Kaiser Oct. 15, 2022, 3:24 p.m. UTC 
  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(-)

Comments

Pavel Skripkin Oct. 15, 2022, 3:56 p.m. UTC | #1 
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
Philipp Hortmann Oct. 15, 2022, 8:31 p.m. UTC | #2 
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