[PATCH 05/12] staging: brcm80211: replaced typedef wlc_ap_info_t by struct wlc_ap_info

Roland Vossen rvossen at broadcom.com
Wed Dec 8 18:36:53 UTC 2010


Hello Dan,

Acknowledged. Indeed it is a better idea to delete this struct. I will do this in a subsequent patch and delete the wlc_mac80211.c comments as well. Greg, is that ok with you (so proceed with the current patch and you will get an additional patch from me) ? 

Thanks, Roland.

-----Original Message-----
From: Dan Carpenter [mailto:error27 at gmail.com] 
Sent: Tuesday, December 07, 2010 11:45 AM
To: Roland Vossen
Cc: gregkh at suse.de; devel at linuxdriverproject.org
Subject: Re: [PATCH 05/12] staging: brcm80211: replaced typedef wlc_ap_info_t by struct wlc_ap_info

On Tue, Dec 07, 2010 at 10:39:39PM +0300, Dan Carpenter wrote:
> > -	wlc_ap_info_t *ap;
> > +	struct wlc_ap_info *ap;
> > 
> 
> This isn't used any more.  It would be better to delete both the lines
> instead of updating them.
> 

Btw.  The bits that used to use it are in
drivers/staging/brcm80211/sys/wlc_mac80211.c, but they're commented out.
You could delete the comments as well.

regards,
dan carpenter







More information about the devel mailing list