[PATCH 1/3] staging: rtl8723bs: rename rt_channel_info members to snake_case

Marcos Andrade posted 3 patches 1 week, 6 days ago
[PATCH 1/3] staging: rtl8723bs: rename rt_channel_info members to snake_case
Posted by Marcos Andrade 1 week, 6 days ago
Rename the CamelCase member names ChannelNum and ScanType of struct
rt_channel_info in rtw_mlme_ext.h to their snake_case equivalents
channel_num and scan_type.

While at it, fix a pre-existing whitespace issue inside the struct
flagged by checkpatch.pl.

This change brings the code in line with the kernel coding style
as described in Documentation/process/coding-style.rst.

No functional changes intended.

Signed-off-by: Marcos Andrade <marcosandrade95963@gmail.com>
---
 drivers/staging/rtl8723bs/include/rtw_mlme_ext.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
index afa5631a441a..998841ae5655 100644
--- a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
+++ b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
@@ -322,8 +322,8 @@ struct mlme_ext_info {
 
 /*  The channel information about this channel including joining, scanning, and power constraints. */
 struct rt_channel_info {
-	u8 		ChannelNum;		/*  The channel number. */
-	enum rt_scan_type	ScanType;		/*  Scan type such as passive or active scan. */
+	u8 channel_num;		/*  The channel number. */
+	enum rt_scan_type scan_type;		/*  Scan type such as passive or active scan. */
 };
 
 int rtw_ch_set_search_ch(struct rt_channel_info *ch_set, const u32 ch);
-- 
2.53.0
Re: [PATCH 1/3] staging: rtl8723bs: rename rt_channel_info members to snake_case
Posted by Dan Carpenter 1 week, 6 days ago
On Sat, Mar 21, 2026 at 03:13:40PM +0000, Marcos Andrade wrote:
> Rename the CamelCase member names ChannelNum and ScanType of struct
> rt_channel_info in rtw_mlme_ext.h to their snake_case equivalents
> channel_num and scan_type.
> 
> While at it, fix a pre-existing whitespace issue inside the struct
> flagged by checkpatch.pl.
> 
> This change brings the code in line with the kernel coding style
> as described in Documentation/process/coding-style.rst.
> 
> No functional changes intended.
> 
> Signed-off-by: Marcos Andrade <marcosandrade95963@gmail.com>
> ---

This breaks the build.

regards,
dan carpenter