On Sat, Dec 22, 2012 at 02:27:45PM -0500, Kevin McKinney wrote: > This patch replaces "ULONG" with "unsigned long" > in Protocol.h > > Signed-off-by: Kevin McKinney <[email protected]> > --- > drivers/staging/bcm/Protocol.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/bcm/Protocol.h b/drivers/staging/bcm/Protocol.h > index 040689b..7070ce6 100644 > --- a/drivers/staging/bcm/Protocol.h > +++ b/drivers/staging/bcm/Protocol.h > @@ -113,8 +113,8 @@ typedef enum _E_SERVICEFLOW_CS_SPEC_ { > typedef struct _TCP_HEADER { > unsigned short usSrcPort; > unsigned short usDestPort; > - ULONG ulSeqNumber; > - ULONG ulAckNumber; > + unsigned long ulSeqNumber; > + unsigned long ulAckNumber;
BCM doesn't work on 64 bits. This might be one of the reasons (we already looked at another problem earlier). I'm pretty sure that the SeqNumber and AckNumber from the TCP header are both 32 bits. I can't test this code myself. regards, dan carpenter _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
