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 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
