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 <[email protected]>
---
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];
+ 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
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 <[email protected]>
> ---
>
> 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.
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 <[email protected]>
> > ---
> >
> > 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.
>