On Thu, Jan 3, 2013 at 9:01 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> 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 <klmckinney1@xxxxxxxxx>
>> ---
>> 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
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
