On Sun, Jun 05, 2011 at 12:23:25PM -0700, julie Sullivan wrote:
> >
> > #include <dngl_stats.h>
> >
> > #include <dhd.h>
> > #include <linux/ieee80211.h>
> > -typedef const struct si_pub si_t;
> > +
> > +struct si_pub;
> >
> > #include <dngl_stats.h>
> > #include <dhd.h>
> 
> Is there any reason why dngl_stats.h and dhd.h are included twice in wl_iw.c?

No, no good reason.  The double inclusion seems to have been aournd for quite a
while.  A few others got cleaned up, but this one got missed.

> (if not, maybe you corrected this in another patch, I couldn't get the
> series to apply cleanly so there was no easy way of checking. If so
> sorry for the noise)

Nope, not fixed yet.  We'll get it cleaned up in a subsequent patch.

- Henry


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

Reply via email to