Re: [PATCH] staging: rtl8723bs: remove braces around single statements
From: Samuel Abraham
Date: Wed Mar 26 2025 - 05:00:38 EST
On Wed, Mar 26, 2025 at 9:26 AM Julia Lawall <julia.lawall@xxxxxxxx> wrote:
>
>
>
> On Tue, 25 Mar 2025, Abraham Samuel Adekunle wrote:
>
> > The code contains braces around single statements in the if blocks
> > which are unnecessary according to the Linux kernel coding style.
> >
> > Remove the braces to improve readability and maintain consistency.
> >
> > Reported by checkpatch:
> >
> > WARNING: braces {} are not necessary for single statement blocks
> >
> > Signed-off-by: Abraham Samuel Adekunle <abrahamadekunle50@xxxxxxxxx>
> > ---
> > drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 7 +++----
> > 1 file changed, 3 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> > index c60e179bb2e1..b17b295e8d3c 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> > @@ -56,9 +56,9 @@ int _ips_leave(struct adapter *padapter)
> > pwrpriv->ips_leave_cnts++;
> >
> > result = rtw_ips_pwr_up(padapter);
> > - if (result == _SUCCESS) {
> > + if (result == _SUCCESS)
> > pwrpriv->rf_pwrstate = rf_on;
> > - }
> > +
>
> I'm not sure you need to add a blank line here?
>
> julia
>
Thank you for your review. I will effect the change