linux-kernel - Re: [PATCH v4 08/11] staging: r8188eu: replace leading spaces by tabs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Y1HaQmlhNGU0XSV1@debian-BULLSEYE-live-builder-AMD64>
Date:   Fri, 21 Oct 2022 05:01:14 +0530
From:   Deepak R Varma <drv@...lo.com>
To:     Praveen Kumar <kumarpraveen@...ux.microsoft.com>
Cc:     outreachy@...ts.linux.dev, Larry.Finger@...inger.net,
        phil@...lpotter.co.uk, paskripkin@...il.com,
        gregkh@...uxfoundation.org, linux-staging@...ts.linux.dev,
        linux-kernel@...r.kernel.org, saurabh.truth@...il.com
Subject: Re: [PATCH v4 08/11] staging: r8188eu: replace leading spaces by tabs

On Fri, Oct 21, 2022 at 11:04:49AM +0530, Praveen Kumar wrote:
> On 21-10-2022 03:01, Deepak R Varma wrote:
> > Spaces are prohibited as per the Linux coding style guidelines. Replace
> > those by tabs wherever possible to improve code alignment. Error
> > reported by checkpatch script.
> >
> > Signed-off-by: Deepak R Varma <drv@...lo.com>
> > ---
> >
> > Changes in v4:
> >    -- None.
> >
> > Changes in v3:
> >    1. Patch newly added in the patch set.
> >
> >  drivers/staging/r8188eu/include/rtw_cmd.h      | 10 +++++-----
> >  drivers/staging/r8188eu/include/rtw_mlme.h     |  2 +-
> >  drivers/staging/r8188eu/include/rtw_mlme_ext.h |  2 +-
> >  3 files changed, 7 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/staging/r8188eu/include/rtw_cmd.h b/drivers/staging/r8188eu/include/rtw_cmd.h
> > index 61fb523afb08..4b6a91aefc1e 100644
> > --- a/drivers/staging/r8188eu/include/rtw_cmd.h
> > +++ b/drivers/staging/r8188eu/include/rtw_cmd.h
> > @@ -592,14 +592,14 @@ struct setratable_parm {
> >  };
> >
> >  struct getratable_parm {
> > -                uint rsvd;
> > +	uint rsvd;
> >  };
> >
> >  struct getratable_rsp {
> > -        u8 ss_ForceUp[NumRates];
> > -        u8 ss_ULevel[NumRates];
> > -        u8 ss_DLevel[NumRates];
> > -        u8 count_judge[NumRates];
> > +	u8 ss_ForceUp[NumRates];
> > +	u8 ss_ULevel[NumRates];
> > +	u8 ss_DLevel[NumRates];
>
> Can this be reworked further like ss_ForceUp -> ss_force_up or ss_forceup ?
> Similarly for other attributes.

Absolutely; however, it will have to be in a separate patch since that change
will be "Avoid CamelCase", different from the current scope of this patch.
I am making note of this work as a separate patch. I will send that in as a
standalone patch as I would like to keep the current patch set size manageable
by me.

Thank you Praveen.
./drv

>
> > +	u8 count_judge[NumRates];
> >  };
> >
> >  /* to get TX,RX retry count */
> > diff --git a/drivers/staging/r8188eu/include/rtw_mlme.h b/drivers/staging/r8188eu/include/rtw_mlme.h
> > index 7658f864136e..ebf7168a7ef9 100644
> > --- a/drivers/staging/r8188eu/include/rtw_mlme.h
> > +++ b/drivers/staging/r8188eu/include/rtw_mlme.h
> > @@ -528,7 +528,7 @@ void rtw_indicate_scan_done(struct adapter *padapter);
> >  int rtw_restruct_sec_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie,
> >  			uint in_len);
> >  int rtw_restruct_wmm_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie,
> > -		        uint in_len, uint initial_out_len);
> > +			uint in_len, uint initial_out_len);
> >  void rtw_init_registrypriv_dev_network(struct adapter *adapter);
> >
> >  void rtw_update_registrypriv_dev_network(struct adapter *adapter);
> > diff --git a/drivers/staging/r8188eu/include/rtw_mlme_ext.h b/drivers/staging/r8188eu/include/rtw_mlme_ext.h
> > index be470f913a94..413b94e38744 100644
> > --- a/drivers/staging/r8188eu/include/rtw_mlme_ext.h
> > +++ b/drivers/staging/r8188eu/include/rtw_mlme_ext.h
> > @@ -449,7 +449,7 @@ void ERP_IE_handler(struct adapter *padapter, struct ndis_802_11_var_ie *pIE);
> >  void VCS_update(struct adapter *padapter, struct sta_info *psta);
> >
> >  void update_beacon_info(struct adapter *padapter, u8 *pframe, uint len,
> > -		        struct sta_info *psta);
> > +			struct sta_info *psta);
> >  int rtw_check_bcn_info(struct adapter  *Adapter, u8 *pframe, u32 packet_len);
> >  void update_IOT_info(struct adapter *padapter);
> >  void update_capinfo(struct adapter *adapter, u16 updatecap);
> > --
> > 2.30.2
> >
> >
> >
>
> Regards,
>
> ~Praveen.
>