Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <a1429a6a-1688-c30a-4fb2-3d575ffecbf1@linux.microsoft.com>
Date:   Fri, 21 Oct 2022 11:04:49 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.3.3
Subject: Re: [PATCH v4 08/11] staging: r8188eu: replace leading spaces by tabs
To:     Deepak R Varma <drv@mailo.com>, outreachy@lists.linux.dev,
        Larry.Finger@lwfinger.net, phil@philpotter.co.uk,
        paskripkin@gmail.com, gregkh@linuxfoundation.org,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
        saurabh.truth@gmail.com
References: <cover.1666299151.git.drv@mailo.com>
 <c32b702c61ea3367d60f0a4c2443093d6ce45a69.1666299151.git.drv@mailo.com>
Content-Language: en-US
From:   Praveen Kumar <kumarpraveen@linux.microsoft.com>
In-Reply-To: <c32b702c61ea3367d60f0a4c2443093d6ce45a69.1666299151.git.drv@mailo.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

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@mailo.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.

> +	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.