On Sat, 19 Apr 2014 16:57:45 -0400 Sean MacLennan <seanm@xxxxxxxx> wrote: > A sparse error fixup removed a htons() which is required for the > driver to function. This patch puts the htons() back and fixes the > sparse warning correctly by changing the left side cast. > > Signed-off-by: Sean MacLennan <seanm@xxxxxxxx> > --- > diff --git a/drivers/staging/rtl8192e/rtllib_tx.c > b/drivers/staging/rtl8192e/rtllib_tx.c index 11d0a9d..b7dd153 100644 > --- a/drivers/staging/rtl8192e/rtllib_tx.c > +++ b/drivers/staging/rtl8192e/rtllib_tx.c > @@ -171,7 +171,7 @@ inline int rtllib_put_snap(u8 *data, u16 h_proto) > snap->oui[1] = oui[1]; > snap->oui[2] = oui[2]; > > - *(u16 *)(data + SNAP_SIZE) = h_proto; > + *(__be16 *)(data + SNAP_SIZE) = htons(h_proto); > > return SNAP_SIZE + sizeof(u16); > } Any status on this patch? Cheers, Sean _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel