Re: [Outreachy kernel] [PATCH v3] staging: rtl8192u: ieee80211: Remove typedef to structure _bandwidth_autoswitch

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Sat, Feb 25, 2017 at 09:34:41PM +0530, sayli karnik wrote:
> Using typedef for a structure type is not suggested in Linux kernel coding
> style guidelines. So remove typedef from the _bandwidth_autoswitch structure.
> The typedef pointer pbandwidth_autoswitch is not used anywhere, so omit
> it. Also rename the structure to bandwidth_autoswitch since an underscore
> is normally prepended to indicate a hidden name, which is not the case
> here.
> 
> Signed-off-by: sayli karnik <karniksayli1995@gmail.com>
> ---
> Changes in v2:
> Renamed the structure from _bandwidth_autoswitch to bandwidth_autoswitch
> Changes in v3:
> Changed spacing after 'long' to as it was before
> 
>  drivers/staging/rtl8192u/ieee80211/ieee80211.h | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h
> index 176083e..c1fa229 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h
> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h
> @@ -1461,13 +1461,12 @@ struct tx_pending {
>  	struct ieee80211_txb *txb;
>  };
>  
> -typedef struct _bandwidth_autoswitch {
> +struct bandwidth_autoswitch {
>  	long threshold_20Mhzto40Mhz;
> -	long	threshold_40Mhzto20Mhz;
> +	long	threshold_40Mhzto20Mhz;

Why are you still changing this line?

Please do not do that.

thanks,

greg k-h