All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Staging: wlan-ng: removed unused code
@ 2016-08-08 17:46 Claudiu Beznea
  2016-08-21 15:34 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Claudiu Beznea @ 2016-08-08 17:46 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, Claudiu Beznea

This patch removes unused code from wlan-ng driver.

Signed-off-by: Claudiu Beznea <claudiu.beznea@gmail.com>
---
 drivers/staging/wlan-ng/p80211types.h | 43 -----------------------------------
 1 file changed, 43 deletions(-)

diff --git a/drivers/staging/wlan-ng/p80211types.h b/drivers/staging/wlan-ng/p80211types.h
index 8cb4fc6..256c742 100644
--- a/drivers/staging/wlan-ng/p80211types.h
+++ b/drivers/staging/wlan-ng/p80211types.h
@@ -205,20 +205,11 @@ typedef struct p80211enumpair {
 	char *name;
 } p80211enumpair_t;
 
-typedef struct p80211enum {
-	int nitems;
-	p80211enumpair_t *list;
-} p80211enum_t;
-
 /*----------------------------------------------------------------*/
 /* The following structure types are used to store data items in */
 /*  messages. */
 
 /* Template pascal string */
-typedef struct p80211pstr {
-	u8 len;
-} __packed p80211pstr_t;
-
 typedef struct p80211pstrd {
 	u8 len;
 	u8 data[0];
@@ -301,14 +292,6 @@ typedef struct p80211item_pstr32 {
 	p80211pstr32_t data;
 } __packed p80211item_pstr32_t;
 
-/* message data item for OCTETSTR, DISPLAYSTR */
-typedef struct p80211item_pstr255 {
-	u32 did;
-	u16 status;
-	u16 len;
-	p80211pstr255_t data;
-} __packed p80211item_pstr255_t;
-
 /* message data item for UNK 392, namely mib items */
 typedef struct p80211item_unk392 {
 	u32 did;
@@ -346,30 +329,4 @@ typedef void (*p80211_fromtext_t) (struct catlistitem *, u32 did, u8 *itembuf,
 				   char *textbuf);
 typedef u32(*p80211_valid_t) (struct catlistitem *, u32 did, u8 *itembuf);
 
-/*----------------------------------------------------------------*/
-/* Enumeration Lists */
-/*  The following are the external declarations */
-/*  for all enumerations  */
-
-extern p80211enum_t MKENUMNAME(truth);
-extern p80211enum_t MKENUMNAME(ifstate);
-extern p80211enum_t MKENUMNAME(powermgmt);
-extern p80211enum_t MKENUMNAME(bsstype);
-extern p80211enum_t MKENUMNAME(authalg);
-extern p80211enum_t MKENUMNAME(phytype);
-extern p80211enum_t MKENUMNAME(temptype);
-extern p80211enum_t MKENUMNAME(regdomain);
-extern p80211enum_t MKENUMNAME(ccamode);
-extern p80211enum_t MKENUMNAME(diversity);
-extern p80211enum_t MKENUMNAME(scantype);
-extern p80211enum_t MKENUMNAME(resultcode);
-extern p80211enum_t MKENUMNAME(reason);
-extern p80211enum_t MKENUMNAME(status);
-extern p80211enum_t MKENUMNAME(msgcode);
-extern p80211enum_t MKENUMNAME(msgitem_status);
-
-extern p80211enum_t MKENUMNAME(lnxroam_reason);
-
-extern p80211enum_t MKENUMNAME(p2preamble);
-
 #endif /* _P80211TYPES_H */
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread
* Re: [PATCH] Staging: wlan-ng: removed unused code
  2016-08-08 17:46 [PATCH] Staging: wlan-ng: removed unused code Claudiu Beznea
@ 2016-08-21 15:34 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2016-08-21 15:34 UTC (permalink / raw)
  To: Claudiu Beznea; +Cc: devel, linux-kernel

On Mon, Aug 08, 2016 at 08:46:20PM +0300, Claudiu Beznea wrote:
> This patch removes unused code from wlan-ng driver.
> 
> Signed-off-by: Claudiu Beznea <claudiu.beznea@gmail.com>
> ---
>  drivers/staging/wlan-ng/p80211types.h | 43 -----------------------------------
>  1 file changed, 43 deletions(-)
> 
> diff --git a/drivers/staging/wlan-ng/p80211types.h b/drivers/staging/wlan-ng/p80211types.h
> index 8cb4fc6..256c742 100644
> --- a/drivers/staging/wlan-ng/p80211types.h
> +++ b/drivers/staging/wlan-ng/p80211types.h
> @@ -205,20 +205,11 @@ typedef struct p80211enumpair {
>  	char *name;
>  } p80211enumpair_t;
>  
> -typedef struct p80211enum {
> -	int nitems;
> -	p80211enumpair_t *list;
> -} p80211enum_t;
> -
>  /*----------------------------------------------------------------*/
>  /* The following structure types are used to store data items in */
>  /*  messages. */
>  
>  /* Template pascal string */
> -typedef struct p80211pstr {
> -	u8 len;
> -} __packed p80211pstr_t;
> -
>  typedef struct p80211pstrd {
>  	u8 len;
>  	u8 data[0];
> @@ -301,14 +292,6 @@ typedef struct p80211item_pstr32 {
>  	p80211pstr32_t data;
>  } __packed p80211item_pstr32_t;
>  
> -/* message data item for OCTETSTR, DISPLAYSTR */
> -typedef struct p80211item_pstr255 {
> -	u32 did;
> -	u16 status;
> -	u16 len;
> -	p80211pstr255_t data;
> -} __packed p80211item_pstr255_t;
> -
>  /* message data item for UNK 392, namely mib items */
>  typedef struct p80211item_unk392 {
>  	u32 did;
> @@ -346,30 +329,4 @@ typedef void (*p80211_fromtext_t) (struct catlistitem *, u32 did, u8 *itembuf,
>  				   char *textbuf);
>  typedef u32(*p80211_valid_t) (struct catlistitem *, u32 did, u8 *itembuf);
>  
> -/*----------------------------------------------------------------*/
> -/* Enumeration Lists */
> -/*  The following are the external declarations */
> -/*  for all enumerations  */
> -
> -extern p80211enum_t MKENUMNAME(truth);
> -extern p80211enum_t MKENUMNAME(ifstate);
> -extern p80211enum_t MKENUMNAME(powermgmt);
> -extern p80211enum_t MKENUMNAME(bsstype);
> -extern p80211enum_t MKENUMNAME(authalg);
> -extern p80211enum_t MKENUMNAME(phytype);
> -extern p80211enum_t MKENUMNAME(temptype);
> -extern p80211enum_t MKENUMNAME(regdomain);
> -extern p80211enum_t MKENUMNAME(ccamode);
> -extern p80211enum_t MKENUMNAME(diversity);
> -extern p80211enum_t MKENUMNAME(scantype);
> -extern p80211enum_t MKENUMNAME(resultcode);
> -extern p80211enum_t MKENUMNAME(reason);
> -extern p80211enum_t MKENUMNAME(status);
> -extern p80211enum_t MKENUMNAME(msgcode);
> -extern p80211enum_t MKENUMNAME(msgitem_status);
> -
> -extern p80211enum_t MKENUMNAME(lnxroam_reason);
> -
> -extern p80211enum_t MKENUMNAME(p2preamble);
> -
>  #endif /* _P80211TYPES_H */
> -- 
> 1.9.1

I don't see any "code" removed, only unused typedefs and externs.

Please be more specific in your changelog comments.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-08-21 21:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-08 17:46 [PATCH] Staging: wlan-ng: removed unused code Claudiu Beznea
2016-08-21 15:34 ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.