On 22.01.2018 12:22, Ajay Singh wrote:
> Fix the "do not add new typedefs" issue found by checkpatch.pl
> script
> 
>From 
>https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-format
the the message body line is wrapped at 75 columns.


> Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
> Reviewed-by: Claudiu Beznea <claudiu.bez...@microchip.com>
> ---
>  drivers/staging/wilc1000/wilc_wlan_if.h | 72 
> ++++++++++++++++-----------------
>  1 file changed, 36 insertions(+), 36 deletions(-)
> 
> diff --git a/drivers/staging/wilc1000/wilc_wlan_if.h 
> b/drivers/staging/wilc1000/wilc_wlan_if.h
> index 1641ddb..e186509 100644
> --- a/drivers/staging/wilc1000/wilc_wlan_if.h
> +++ b/drivers/staging/wilc1000/wilc_wlan_if.h
> @@ -114,28 +114,28 @@ enum {
>       G_MIXED_11B_2_MODE,             /* 1,2,5,11,6,12,24 otherwise all on */
>  };
>  
> -typedef enum {
> +enum {
>       G_SHORT_PREAMBLE        = 0,    /* Short Preamble */
>       G_LONG_PREAMBLE         = 1,    /* Long Preamble */
>       G_AUTO_PREAMBLE         = 2,    /* Auto Preamble Selection */
> -} G_PREAMBLE_T;
> +};
>  
>  #define MAC_CONNECTED                1
>  #define MAC_DISCONNECTED     0
>  
>  #define SCAN_DONE            TRUE
> -typedef enum {
> +enum {
>       PASSIVE_SCAN            = 0,
>       ACTIVE_SCAN             = 1,
> -} SCANTYPE_T;
> +};
>  
> -typedef enum {
> +enum {
>       NO_POWERSAVE            = 0,
>       MIN_FAST_PS             = 1,
>       MAX_FAST_PS             = 2,
>       MIN_PSPOLL_PS           = 3,
>       MAX_PSPOLL_PS           = 4
> -} USER_PS_MODE_T;
> +};
>  
>  enum chip_ps_states {
>       CHIP_WAKEDUP            = 0,
> @@ -153,7 +153,7 @@ enum bus_release {
>       RELEASE_ALLOW_SLEEP     = 1,
>  };
>  
> -typedef enum {
> +enum {
>       NO_SECURITY             = 0,
>       WEP_40                  = 0x3,
>       WEP_104                 = 0x7,
> @@ -163,7 +163,7 @@ typedef enum {
>       WPA2_AES                = 0x31,
>       WPA2_TKIP               = 0x51,
>       WPA2_AES_TKIP           = 0x71, /* Aes or Tkip */
> -} SECURITY_T;
> +};
>  
>  enum AUTHTYPE {
>       OPEN_SYSTEM             = 1,
> @@ -178,88 +178,88 @@ enum SITESURVEY {
>       SITE_SURVEY_OFF         = 2
>  };
>  
> -typedef enum {
> +enum {
>       NORMAL_ACK              = 0,
>       NO_ACK,
> -} ACK_POLICY_T;
> +};
>  
> -typedef enum {
> +enum {
>       DONT_RESET              = 0,
>       DO_RESET                = 1,
>       NO_REQUEST              = 2,
> -} RESET_REQ_T;
> +};
>  
> -typedef enum {
> +enum {
>       REKEY_DISABLE           = 1,
>       REKEY_TIME_BASE,
>       REKEY_PKT_BASE,
>       REKEY_TIME_PKT_BASE
> -} RSNA_REKEY_POLICY_T;
> +};
>  
> -typedef enum {
> +enum {
>       FILTER_NO               = 0x00,
>       FILTER_AP_ONLY          = 0x01,
>       FILTER_STA_ONLY         = 0x02
> -} SCAN_CLASS_FITLER_T;
> +};
>  
> -typedef enum {
> +enum {
>       PRI_HIGH_RSSI           = 0x00,
>       PRI_LOW_RSSI            = 0x04,
>       PRI_DETECT              = 0x08
> -} SCAN_PRI_T;
> +};
>  
> -typedef enum {
> +enum {
>       CH_FILTER_OFF           = 0x00,
>       CH_FILTER_ON            = 0x10
> -} CH_FILTER_T;
> +};
>  
> -typedef enum {
> +enum {
>       AUTO_PROT               = 0,    /* Auto */
>       NO_PROT,                        /* Do not use any protection */
>       ERP_PROT,                       /* Protect all ERP frame exchanges */
>       HT_PROT,                        /* Protect all HT frame exchanges  */
>       GF_PROT,                        /* Protect all GF frame exchanges  */
> -} N_PROTECTION_MODE_T;
> +};
>  
> -typedef enum {
> +enum {
>       G_SELF_CTS_PROT,
>       G_RTS_CTS_PROT,
> -} G_PROTECTION_MODE_T;
> +};
>  
> -typedef enum {
> +enum {
>       HT_MIXED_MODE           = 1,
>       HT_ONLY_20MHZ_MODE,
>       HT_ONLY_20_40MHZ_MODE,
> -} N_OPERATING_MODE_T;
> +};
>  
> -typedef enum {
> +enum {
>       NO_DETECT               = 0,
>       DETECT_ONLY             = 1,
>       DETECT_PROTECT          = 2,
>       DETECT_PROTECT_REPORT   = 3,
> -} N_OBSS_DETECTION_T;
> +};
>  
> -typedef enum {
> +enum {
>       RTS_CTS_NONHT_PROT      = 0,    /* RTS-CTS at non-HT rate */
>       FIRST_FRAME_NONHT_PROT,         /* First frame at non-HT rate */
>       LSIG_TXOP_PROT,                 /* LSIG TXOP Protection */
>       FIRST_FRAME_MIXED_PROT,         /* First frame at Mixed format */
> -} N_PROTECTION_TYPE_T;
> +};
>  
> -typedef enum {
> +enum {
>       STATIC_MODE             = 1,
>       DYNAMIC_MODE            = 2,
>       MIMO_MODE               = 3,    /* power save disable */
> -} N_SMPS_MODE_T;
> +};
>  
> -typedef enum {
> +enum {
>       DISABLE_SELF_CTS,
>       ENABLE_SELF_CTS,
>       DISABLE_TX_ABORT,
>       ENABLE_TX_ABORT,
>       HW_TRIGGER_ABORT,
>       SW_TRIGGER_ABORT,
> -} TX_ABORT_OPTION_T;
> +};
>  
>  enum wid_type {
>       WID_CHAR                = 0,
> @@ -281,7 +281,7 @@ struct wid {
>       s8 *val;
>  };
>  
> -typedef enum {
> +enum {
>       WID_NIL                         = 0xffff,
>  
>       /*
> @@ -889,7 +889,7 @@ typedef enum {
>       /* Miscellaneous WIDs */
>       WID_ALL                         = 0x7FFE,
>       WID_MAX                         = 0xFFFF
> -} WID_T;
> +};
>  
>  struct wilc;
>  int wilc_wlan_init(struct net_device *dev);
> 

Reply via email to