Re: [PATCH] staging: rtl8723bs: os_dep: remove redundant else in rtw_dev_unload

[Luka Gejak]

On Sat Apr 18, 2026 at 7:09 PM CEST, Shyam Sunder Reddy Padira wrote:
> Remove the unnecessary else block following a break
> statment to simplify the control flow and improve
> code readability.
>
> Signed-off-by: Shyam Sunder Reddy Padira <shyamsunderreddypadira@xxxxxxxxx>
> ---
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 29939bf5a156..3207d6d2c727 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -988,12 +988,10 @@ void rtw_dev_unload(struct adapter *padapter)
>  			rtw_stop_drv_threads(padapter);
>  
>  		while (atomic_read(&pcmdpriv->cmdthd_running)) {
> -			if (cnt > 5) {
> +			if (cnt > 5)
>  				break;
> -			} else {
> -				cnt++;
> -				msleep(10);
> -			}
> +			cnt++;
> +			msleep(10);
>  		}
>  
>  		/* check the status of IPS */

LGTM.

Reviewed-by: Luka Gejak <luka.gejak@xxxxxxxxx>

Best regards,
Luka Gejak