[PATCH 0/2] staging: rtl8723bs: add braces to if/else statements

[PATCH 0/2] staging: rtl8723bs: add braces to if/else statements

[Siwanan Bungtong]

This series adds missing braces to if/else statements in rtl8723bs
to comply with the kernel coding style and improve code readability.

No functional changes.

Siwanan Bungtong (2):
  staging: rtl8723bs: add braces to if/else arms in rtw_efuse.c
  staging: rtl8723bs: add braces to if/else arms in HalBtc8723b1Ant.c

 drivers/staging/rtl8723bs/core/rtw_efuse.c      | 4 ++--
 drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c | 7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)

-- 
2.53.0

[PATCH 1/2] staging: rtl8723bs: add braces to if/else arms in rtw_efuse.c

[Siwanan Bungtong]

Add missing braces to if/else statements to comply with kernel
coding style and improve readability.

No functional changes.

Signed-off-by: Siwanan Bungtong <horstaufmental@gmail.com>
---
 drivers/staging/rtl8723bs/core/rtw_efuse.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_efuse.c b/drivers/staging/rtl8723bs/core/rtw_efuse.c
index 099320e4eb50..7600b723fda3 100644
--- a/drivers/staging/rtl8723bs/core/rtw_efuse.c
+++ b/drivers/staging/rtl8723bs/core/rtw_efuse.c
@@ -76,9 +76,9 @@ u16		Address)
 				break;
 		}
 		return rtw_read8(Adapter, EFUSE_CTRL);
-	} else
+	} else {
 		return 0xFF;
-
+	}
 } /* EFUSE_Read1Byte */
 
 /*  11/16/2008 MH Read one byte from real Efuse. */
-- 
2.53.0

[PATCH 2/2] staging: rtl8723bs: add braces to if/else arms in HalBtc8723b1Ant.c

[Siwanan Bungtong]

Add missing braces to if/else statements to comply with kernel
coding style and improve readability.

No functional changes.

Signed-off-by: Siwanan Bungtong <horstaufmental@gmail.com>
---
 drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c b/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
index b3e34f97cfc6..9bef096164fa 100644
--- a/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
+++ b/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
@@ -1485,12 +1485,13 @@ static void halbtc8723b1ant_PsTdmaCheckForPowerSaveState(
 	if (lpsMode) {	/*  already under LPS state */
 		if (bNewPsState) {
 			/*  keep state under LPS, do nothing. */
-		} else /*  will leave LPS state, turn off psTdma first */
+		} else { /*  will leave LPS state, turn off psTdma first */
 			halbtc8723b1ant_PsTdma(pBtCoexist, NORMAL_EXEC, false, 0);
+		}
 	} else {						/*  NO PS state */
-		if (bNewPsState) /*  will enter LPS state, turn off psTdma first */
+		if (bNewPsState) { /*  will enter LPS state, turn off psTdma first */
 			halbtc8723b1ant_PsTdma(pBtCoexist, NORMAL_EXEC, false, 0);
-		else {
+		} else {
 			/*  keep state under NO PS state, do nothing. */
 		}
 	}
-- 
2.53.0

Re: [PATCH 0/2] staging: rtl8723bs: add braces to if/else statements

[Nikolay Kulikov]

On Sat, May 02, 2026 at 03:31:19AM +0700, Siwanan Bungtong wrote:
> This series adds missing braces to if/else statements in rtl8723bs
> to comply with the kernel coding style and improve code readability.
> 
> No functional changes.
> 
> Siwanan Bungtong (2):
>   staging: rtl8723bs: add braces to if/else arms in rtw_efuse.c
>   staging: rtl8723bs: add braces to if/else arms in HalBtc8723b1Ant.c
> 
>  drivers/staging/rtl8723bs/core/rtw_efuse.c      | 4 ++--
>  drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c | 7 ++++---
>  2 files changed, 6 insertions(+), 5 deletions(-)
> 

This is not applicable to staging-next, please rebase.

Thanks,
Nikolay

Re: [PATCH 1/2] staging: rtl8723bs: add braces to if/else arms in rtw_efuse.c

[Greg KH]

On Sat, May 02, 2026 at 03:31:20AM +0700, Siwanan Bungtong wrote:
> Add missing braces to if/else statements to comply with kernel
> coding style and improve readability.
> 
> No functional changes.
> 
> Signed-off-by: Siwanan Bungtong <horstaufmental@gmail.com>
> ---
>  drivers/staging/rtl8723bs/core/rtw_efuse.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723bs/core/rtw_efuse.c b/drivers/staging/rtl8723bs/core/rtw_efuse.c
> index 099320e4eb50..7600b723fda3 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_efuse.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_efuse.c
> @@ -76,9 +76,9 @@ u16		Address)
>  				break;
>  		}
>  		return rtw_read8(Adapter, EFUSE_CTRL);
> -	} else
> +	} else {
>  		return 0xFF;
> -
> +	}
>  } /* EFUSE_Read1Byte */
>  
>  /*  11/16/2008 MH Read one byte from real Efuse. */
> -- 
> 2.53.0
> 
> 

Does not apply to my tree :(