[PATCH 0/1] staging: rtl8723bs: verify boolean and logics and rename dot11AuthAlgrthm

Nguyen Duc Thinh posted 1 patch 2 months ago
There is a newer version of this series
drivers/staging/rtl8723bs/core/rtw_ap.c       | 28 ++++----
drivers/staging/rtl8723bs/core/rtw_btcoex.c   |  2 +-
.../staging/rtl8723bs/core/rtw_ieee80211.c    |  2 +-
.../staging/rtl8723bs/core/rtw_ioctl_set.c    | 40 +++++------
drivers/staging/rtl8723bs/core/rtw_mlme.c     | 16 ++---
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 22 +++---
drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |  2 +-
drivers/staging/rtl8723bs/core/rtw_recv.c     | 70 +++++++++----------
drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  2 +-
.../staging/rtl8723bs/core/rtw_wlan_util.c    |  8 +--
drivers/staging/rtl8723bs/core/rtw_xmit.c     | 36 +++++-----
drivers/staging/rtl8723bs/hal/hal_com.c       |  2 +-
drivers/staging/rtl8723bs/include/rtw_mlme.h  | 12 ++--
.../staging/rtl8723bs/include/rtw_security.h  | 14 ++--
.../staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 32 ++++-----
drivers/staging/rtl8723bs/os_dep/os_intfs.c   |  2 +-
16 files changed, 145 insertions(+), 145 deletions(-)
[PATCH 0/1] staging: rtl8723bs: verify boolean and logics and rename dot11AuthAlgrthm
Posted by Nguyen Duc Thinh 2 months ago
This series cleans up the rtl8723bs driver by renaming the CamelCase variable dot11AuthAlgrthm to dot11_auth_algrthm and refactoring boolean comparisons to follow Linux coding standards. Both changes were verified by building individual object files and running checkpatch.pl.

Nguyen Duc Thinh (1):
  staging: rtl8723bs: verify boolean and logics and rename
    dot11AuthAlgrthm

 drivers/staging/rtl8723bs/core/rtw_ap.c       | 28 ++++----
 drivers/staging/rtl8723bs/core/rtw_btcoex.c   |  2 +-
 .../staging/rtl8723bs/core/rtw_ieee80211.c    |  2 +-
 .../staging/rtl8723bs/core/rtw_ioctl_set.c    | 40 +++++------
 drivers/staging/rtl8723bs/core/rtw_mlme.c     | 16 ++---
 drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 22 +++---
 drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |  2 +-
 drivers/staging/rtl8723bs/core/rtw_recv.c     | 70 +++++++++----------
 drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  2 +-
 .../staging/rtl8723bs/core/rtw_wlan_util.c    |  8 +--
 drivers/staging/rtl8723bs/core/rtw_xmit.c     | 36 +++++-----
 drivers/staging/rtl8723bs/hal/hal_com.c       |  2 +-
 drivers/staging/rtl8723bs/include/rtw_mlme.h  | 12 ++--
 .../staging/rtl8723bs/include/rtw_security.h  | 14 ++--
 .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 32 ++++-----
 drivers/staging/rtl8723bs/os_dep/os_intfs.c   |  2 +-
 16 files changed, 145 insertions(+), 145 deletions(-)

-- 
2.47.3
Re: [PATCH 0/1] staging: rtl8723bs: verify boolean and logics and rename dot11AuthAlgrthm
Posted by Luka Gejak 2 months ago
On Sat Apr 11, 2026 at 5:53 AM CEST, Nguyen Duc Thinh wrote:
> This series cleans up the rtl8723bs driver by renaming the CamelCase variable dot11AuthAlgrthm to dot11_auth_algrthm and refactoring boolean comparisons to follow Linux coding standards. Both changes were verified by building individual object files and running checkpatch.pl.
Please wrap all text to 72 characters. Also you don't need cover letter
for 1 patch, so unless you properly split this into 2 patches you don't 
need cover letter, but more on that in inline review of the patch.
>
> Nguyen Duc Thinh (1):
>   staging: rtl8723bs: verify boolean and logics and rename
>     dot11AuthAlgrthm
>
>  drivers/staging/rtl8723bs/core/rtw_ap.c       | 28 ++++----
>  drivers/staging/rtl8723bs/core/rtw_btcoex.c   |  2 +-
>  .../staging/rtl8723bs/core/rtw_ieee80211.c    |  2 +-
>  .../staging/rtl8723bs/core/rtw_ioctl_set.c    | 40 +++++------
>  drivers/staging/rtl8723bs/core/rtw_mlme.c     | 16 ++---
>  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 22 +++---
>  drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |  2 +-
>  drivers/staging/rtl8723bs/core/rtw_recv.c     | 70 +++++++++----------
>  drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  2 +-
>  .../staging/rtl8723bs/core/rtw_wlan_util.c    |  8 +--
>  drivers/staging/rtl8723bs/core/rtw_xmit.c     | 36 +++++-----
>  drivers/staging/rtl8723bs/hal/hal_com.c       |  2 +-
>  drivers/staging/rtl8723bs/include/rtw_mlme.h  | 12 ++--
>  .../staging/rtl8723bs/include/rtw_security.h  | 14 ++--
>  .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 32 ++++-----
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c   |  2 +-
>  16 files changed, 145 insertions(+), 145 deletions(-)