* Re: [PATCH] staging: rtl8723bs: fix whitespace issues in hal_com.c
2026-02-02 2:18 [PATCH] staging: rtl8723bs: fix whitespace issues in hal_com.c Ö. Efe D.
@ 2026-02-02 7:08 ` M.samet Duman
2026-02-02 9:34 ` Dan Carpenter
2026-02-02 9:38 ` Dan Carpenter
2026-02-07 13:18 ` Greg Kroah-Hartman
2 siblings, 1 reply; 7+ messages in thread
From: M.samet Duman @ 2026-02-02 7:08 UTC (permalink / raw)
To: Ö. Efe D.
Cc: Greg Kroah-Hartman, Michael Straube, linux-staging,
Ö. Efe D. , Öner Efe Dasgüney
Hello Ömer,
I wanted to share a brief observation regarding the patch titled:
“Remove spaces before tabs in comments to fix checkpatch warnings”
Looking at the diff, the change is limited to whitespace alignment inside comments. There is no functional change, no meaningful impact on readability, and no effect on build or runtime behavior.
Given that scope, describing the change as a “fix” feels slightly misleading. From a reader’s point of view, this appears closer to a cosmetic or comment cleanup driven by checkpatch preferences rather than a fix addressing an actual issue.
This is not to say the change is invalid, but clearer framing would help set expectations when scanning patches on the list and later when looking at history or blame output.
Thanks for sharing the patch.
> 2026. 2. 2. 오전 5:19, Ö. Efe D. <skylinesytbe@gmail.com> 작성:
>
> Remove spaces before tabs in comments to fix checkpatch warnings:
> "please, no space before tabs"
>
> Signed-off-by: Öner Efe Dasgüney <efedasguney@protonmail.com>
> ---
> drivers/staging/rtl8723bs/hal/hal_com.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
> index 70b5b289f9cb..10c34228a595 100644
> --- a/drivers/staging/rtl8723bs/hal/hal_com.c
> +++ b/drivers/staging/rtl8723bs/hal/hal_com.c
> @@ -362,8 +362,8 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
>
> if (bWIFICfg) { /* WMM */
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 0, 1, 0, 1, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 0, 1, 0, 1, 0, 0, 0, 0, 0 }; */
> /* 0:ep_0 num, 1:ep_1 num */
>
> pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[1];/* VO */
> @@ -379,8 +379,8 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
> } else { /* typical setting */
>
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 1, 1, 0, 0, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 1, 1, 0, 0, 0, 0, 0, 0, 0 }; */
> /* 0:ep_0 num, 1:ep_1 num */
>
> pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[0];/* VO */
> @@ -403,8 +403,8 @@ static void _ThreeOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
>
> if (bWIFICfg) { /* for WMM */
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 1, 2, 1, 0, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 1, 2, 1, 0, 0, 0, 0, 0, 0 }; */
> /* 0:H, 1:N, 2:L */
>
> pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[0];/* VO */
> @@ -420,8 +420,8 @@ static void _ThreeOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
> } else { /* typical setting */
>
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 2, 2, 1, 0, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 2, 2, 1, 0, 0, 0, 0, 0, 0 }; */
> /* 0:H, 1:N, 2:L */
>
> pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[0];/* VO */
> --
> 2.52.0
>
>
---
Best regards,
Mehmet Samet Duman
Author of Project Tick
https://projecttick.org
https://github.com/Project-Tick
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] staging: rtl8723bs: fix whitespace issues in hal_com.c
2026-02-02 2:18 [PATCH] staging: rtl8723bs: fix whitespace issues in hal_com.c Ö. Efe D.
2026-02-02 7:08 ` M.samet Duman
@ 2026-02-02 9:38 ` Dan Carpenter
2026-02-07 13:18 ` Greg Kroah-Hartman
2 siblings, 0 replies; 7+ messages in thread
From: Dan Carpenter @ 2026-02-02 9:38 UTC (permalink / raw)
To: Ö. Efe D.
Cc: Greg Kroah-Hartman, Michael Straube, linux-staging,
Ö. Efe D., Öner Efe Dasgüney
On Mon, Feb 02, 2026 at 03:18:45AM +0100, Ö. Efe D. wrote:
> Remove spaces before tabs in comments to fix checkpatch warnings:
> "please, no space before tabs"
>
> Signed-off-by: Öner Efe Dasgüney <efedasguney@protonmail.com>
> ---
> drivers/staging/rtl8723bs/hal/hal_com.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
> index 70b5b289f9cb..10c34228a595 100644
> --- a/drivers/staging/rtl8723bs/hal/hal_com.c
> +++ b/drivers/staging/rtl8723bs/hal/hal_com.c
> @@ -362,8 +362,8 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
>
> if (bWIFICfg) { /* WMM */
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 0, 1, 0, 1, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 0, 1, 0, 1, 0, 0, 0, 0, 0 }; */
These things are supposed to be aligned so that the commas on the top
line are the same as on the bottom line. It's a table. But it's messed
up both before and after.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] staging: rtl8723bs: fix whitespace issues in hal_com.c
2026-02-02 2:18 [PATCH] staging: rtl8723bs: fix whitespace issues in hal_com.c Ö. Efe D.
2026-02-02 7:08 ` M.samet Duman
2026-02-02 9:38 ` Dan Carpenter
@ 2026-02-07 13:18 ` Greg Kroah-Hartman
2 siblings, 0 replies; 7+ messages in thread
From: Greg Kroah-Hartman @ 2026-02-07 13:18 UTC (permalink / raw)
To: Ö. Efe D.
Cc: Michael Straube, linux-staging, Ö. Efe D.,
Öner Efe Dasgüney
On Mon, Feb 02, 2026 at 03:18:45AM +0100, Ö. Efe D. wrote:
> Remove spaces before tabs in comments to fix checkpatch warnings:
> "please, no space before tabs"
>
> Signed-off-by: Öner Efe Dasgüney <efedasguney@protonmail.com>
> ---
> drivers/staging/rtl8723bs/hal/hal_com.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
> index 70b5b289f9cb..10c34228a595 100644
> --- a/drivers/staging/rtl8723bs/hal/hal_com.c
> +++ b/drivers/staging/rtl8723bs/hal/hal_com.c
> @@ -362,8 +362,8 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
>
> if (bWIFICfg) { /* WMM */
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 0, 1, 0, 1, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 0, 1, 0, 1, 0, 0, 0, 0, 0 }; */
> /* 0:ep_0 num, 1:ep_1 num */
>
> pdvobjpriv->Queue2Pipe[0] = pdvobjpriv->RtOutPipe[1];/* VO */
> @@ -379,8 +379,8 @@ static void _TwoOutPipeMapping(struct adapter *padapter, bool bWIFICfg)
> } else { /* typical setting */
>
>
> - /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> - /* 1, 1, 0, 0, 0, 0, 0, 0, 0 }; */
> + /* BK, BE, VI, VO, BCN, CMD, MGT, HIGH, HCCA */
> + /* 1, 1, 0, 0, 0, 0, 0, 0, 0 }; */
As Dan said, this patch is correct in that it fixes the coding style
issue, but now the table looks just as bad as before. How about fixing
this up properly so that it both looks correct, AND fixes the coding
style warning at the same time?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread