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:[email protected]] 
Sent: Tuesday, December 07, 2010 11:45 AM
To: Roland Vossen
Cc: [email protected]; [email protected]
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




_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to