Albert Herranz wrote: > Larry Finger wrote: >> Albert Herranz wrote: > > b43_new_kidx_api() (defined in xmit.h) needs struct b43_wldev defined because it dereferences it. > With this patch, struct b43_wl (defined in b43.h) needs struct b43_rxhdr_fw4 and struct b43_txhdr (defined in xmit.h). > > So we have b43_wldev -> b43_rxhdr_fw4, b43_txhdr -> b43_wl (at least). > > Moving the definition of struct b43_wl after the definition of struct b43_wldev and placing the inclusion of xmit.h between them gets rid of the generated dependencies. > > The patch hints too to other possible solution. > >> /* >> * Include goes here to avoid a dependency problem. >> * A better fix would be to integrate xmit.h into b43.h. >> */ >> #include "xmit.h" > > Do you have any other solutions in mind? No. I missed the hint. At first I thought there was a white-space problem that caused b43_wl to be replaced. When that was not the case, I posed my question. All seems OK. Larry -- 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