On Tue, Feb 05, 2008 at 12:02:00AM +0000, Ben Dooks wrote: > This patch splits the receive status in 8bit wide fields and convert the > packet length from little endian to CPU byte order. > > Signed-off-by: Laurent Pinchart <[EMAIL PROTECTED]> > Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> > > Index: linux-2.6.24-git5-dm9k/drivers/net/dm9000.c > =================================================================== > --- linux-2.6.24-git5-dm9k.orig/drivers/net/dm9000.c > +++ linux-2.6.24-git5-dm9k/drivers/net/dm9000.c > @@ -867,7 +867,8 @@ dm9000_timer(unsigned long data) > } > > struct dm9000_rxhdr { > - u16 RxStatus; > + u8 RxPktReady; > + u8 RxStatus; > u16 RxLen;
thgis should be __le16. Also please install sparse and do a make C=2 CHECKFLAGS="-D__CHECK_ENDIAN__" run over the driver to make sure all hw structures are properly annotated and you do byteswaps consistantly. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html