[PATCH v4 0/3] staging: rtl8723bs: coding style and refactoring cleanups

Khushal Chitturi posted 3 patches 2 weeks, 2 days ago
There is a newer version of this series
drivers/staging/rtl8723bs/core/rtw_cmd.c      | 102 ++++++------
.../staging/rtl8723bs/core/rtw_ioctl_set.c    |  15 +-
drivers/staging/rtl8723bs/core/rtw_mlme.c     |  16 +-
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  10 +-
drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |   2 +-
drivers/staging/rtl8723bs/core/rtw_recv.c     |   4 +-
drivers/staging/rtl8723bs/core/rtw_xmit.c     | 149 +++++++++++-------
drivers/staging/rtl8723bs/hal/hal_btcoex.c    |   6 +-
.../staging/rtl8723bs/hal/rtl8723bs_xmit.c    |   4 +-
drivers/staging/rtl8723bs/include/rtw_mlme.h  |  31 ++--
.../staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   2 +-
drivers/staging/rtl8723bs/os_dep/os_intfs.c   |   8 +-
12 files changed, 194 insertions(+), 155 deletions(-)
[PATCH v4 0/3] staging: rtl8723bs: coding style and refactoring cleanups
Posted by Khushal Chitturi 2 weeks, 2 days ago
This patch series performs minor refactoring and coding style cleanups in
the rtl8723bs driver, focusing on rtw_xmit.c and addressing checkpatch.pl
warnings.

---
Changelog:
v3 -> v4: Updated rename scope, cleaned up formatting, comments, and
parentheses.
v2 -> v3: Resubmitted as a versioned series to avoid threading confusion.
v1 -> v2: Fixed incorrect commit messages.

Note: I noticed this driver uses a custom N_BYTE_ALIGMENT macro for memory
alignment. Would it be acceptable if I send a follow-up patch converting
this to PTR_ALIGN since it's more commonly used in the kernel?

Khushal Chitturi (3):
  staging: rtl8723bs: simplify boolean expressions
  staging: rtl8723bs: rename LinkDetectInfo and clean up related code
  staging: rtl8723bs: fix line length and alignment issues

 drivers/staging/rtl8723bs/core/rtw_cmd.c      | 102 ++++++------
 .../staging/rtl8723bs/core/rtw_ioctl_set.c    |  15 +-
 drivers/staging/rtl8723bs/core/rtw_mlme.c     |  16 +-
 drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  10 +-
 drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |   2 +-
 drivers/staging/rtl8723bs/core/rtw_recv.c     |   4 +-
 drivers/staging/rtl8723bs/core/rtw_xmit.c     | 149 +++++++++++-------
 drivers/staging/rtl8723bs/hal/hal_btcoex.c    |   6 +-
 .../staging/rtl8723bs/hal/rtl8723bs_xmit.c    |   4 +-
 drivers/staging/rtl8723bs/include/rtw_mlme.h  |  31 ++--
 .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   2 +-
 drivers/staging/rtl8723bs/os_dep/os_intfs.c   |   8 +-
 12 files changed, 194 insertions(+), 155 deletions(-)

-- 
2.52.0
Re: [PATCH v4 0/3] staging: rtl8723bs: coding style and refactoring cleanups
Posted by Greg KH 1 week, 4 days ago
On Thu, Jan 22, 2026 at 10:17:48PM +0530, Khushal Chitturi wrote:
> This patch series performs minor refactoring and coding style cleanups in
> the rtl8723bs driver, focusing on rtw_xmit.c and addressing checkpatch.pl
> warnings.
> 
> ---
> Changelog:
> v3 -> v4: Updated rename scope, cleaned up formatting, comments, and
> parentheses.
> v2 -> v3: Resubmitted as a versioned series to avoid threading confusion.
> v1 -> v2: Fixed incorrect commit messages.
> 
> Note: I noticed this driver uses a custom N_BYTE_ALIGMENT macro for memory
> alignment. Would it be acceptable if I send a follow-up patch converting
> this to PTR_ALIGN since it's more commonly used in the kernel?
> 
> Khushal Chitturi (3):
>   staging: rtl8723bs: simplify boolean expressions
>   staging: rtl8723bs: rename LinkDetectInfo and clean up related code
>   staging: rtl8723bs: fix line length and alignment issues
> 
>  drivers/staging/rtl8723bs/core/rtw_cmd.c      | 102 ++++++------
>  .../staging/rtl8723bs/core/rtw_ioctl_set.c    |  15 +-
>  drivers/staging/rtl8723bs/core/rtw_mlme.c     |  16 +-
>  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  10 +-
>  drivers/staging/rtl8723bs/core/rtw_pwrctrl.c  |   2 +-
>  drivers/staging/rtl8723bs/core/rtw_recv.c     |   4 +-
>  drivers/staging/rtl8723bs/core/rtw_xmit.c     | 149 +++++++++++-------
>  drivers/staging/rtl8723bs/hal/hal_btcoex.c    |   6 +-
>  .../staging/rtl8723bs/hal/rtl8723bs_xmit.c    |   4 +-
>  drivers/staging/rtl8723bs/include/rtw_mlme.h  |  31 ++--
>  .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   2 +-
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c   |   8 +-
>  12 files changed, 194 insertions(+), 155 deletions(-)
> 
> -- 
> 2.52.0
> 

Does not apply to my tree at all, please rebase and resend.

thanks,

greg k-h