On Thu, Jan 3, 2013 at 9:01 AM, Dan Carpenter <[email protected]> wrote: > 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.
Yeah, this is one struct that I was looking at to address the 64 bit problem. I will change these to 32 bit int and have these changes tested. If it works, I will submit another patch. Thanks, Kevin _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
