Re: [PATCH 1/2] staging: rtl8723bs: simplify LPS_RF_ON_check control flow
From: Greg Kroah-Hartman
Date: Thu May 21 2026 - 05:45:47 EST
On Tue, May 12, 2026 at 03:04:44AM +0000, Hungyu Lin wrote:
> Replace err + break pattern with direct returns to simplify
> the control flow and improve readability.
>
> No functional change intended.
>
> Signed-off-by: Hungyu Lin <dennylin0707@xxxxxxxxx>
> ---
> drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 24 ++++++++------------
> 1 file changed, 9 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> index ca1cb58fc801..7f297f2d2ff4 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> @@ -389,35 +389,29 @@ void rtw_set_ps_mode(struct adapter *padapter, u8 ps_mode,
>
> /*
> * Return:
> - *0: Leave OK
> - *-1: Timeout
> - *-2: Other error
> + * 0: Leave OK
> + * -1: Timeout
> + * -2: Other error
> */
This change was not documented above, and doesn't belong in this patch
:(