Re: [PATCH 2/6] staging: rtl8723bs: fix position of opening braces

[Sayyad Abid]

On Mon, Sep 9, 2024 at 2:03 AM Philipp Hortmann
<philipp.g.hortmann@xxxxxxxxx> wrote:
>
> On 9/8/24 12:11, Sayyad Abid wrote:
> > This patch fixes the coding style issue of opening bracket "{" being on
> > the next line.
> >
> > Signed-off-by: Sayyad Abid <sayyad.abid16@xxxxxxxxx>
> >
> > ---
> >   drivers/staging/rtl8723bs/include/rtw_security.h | 6 ++----
> >   1 file changed, 2 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8723bs/include/rtw_security.h b/drivers/staging/rtl8723bs/include/rtw_security.h
> > index 487d69460cec..3ccbccf92417 100644
> > --- a/drivers/staging/rtl8723bs/include/rtw_security.h
> > +++ b/drivers/staging/rtl8723bs/include/rtw_security.h
> > @@ -170,8 +170,7 @@ struct security_priv {
> >
> >   #define GET_ENCRY_ALGO(psecuritypriv, psta, encry_algo, bmcst)\
> >   do {\
> > -     switch (psecuritypriv->dot11AuthAlgrthm)\
> > -     {\
> > +     switch (psecuritypriv->dot11AuthAlgrthm) {\
> >               case dot11AuthAlgrthm_Open:\
> >               case dot11AuthAlgrthm_Shared:\
> >               case dot11AuthAlgrthm_Auto:\
> > @@ -191,8 +190,7 @@ do {\
> >
> >   #define SET_ICE_IV_LEN(iv_len, icv_len, encrypt)\
> >   do {\
> > -     switch (encrypt)\
> > -     {\
> > +     switch (encrypt) {\
> >               case _WEP40_:\
> >               case _WEP104_:\
> >                       iv_len = 4;\
>
>
> Hi Sayyad,
>
> I cannot apply this patch. Please rebase.
>
Okay, will do this.
> Thanks for your support.
>
> Bye Philipp
Thank You!


-- 
Abid