Re: [PATCH] mv643xx_eth: fix byte order when checksum offload is enabled

2008-01-20 Thread Byron Bradley
The Marvell Orion system on chips have an integrated mv643xx MAC. On these little endian ARM devices mv643xx will oops when checksum offload is enabled. Swapping the byte order of the protocol and checksum solves this problem. Signed-off-by: Byron Bradley <[EMAIL PROTECTED]> Cc: Dale Farnsworth <[

Re: [PATCH] mv643xx_eth: fix byte order when checksum offload is enabled

2008-01-19 Thread Dale Farnsworth
OK, after digesting Al Viro's comments on this, I agree with him and retract this. These multiple byte swaps sure are confusing. Byron, would you please resubmit? Thanks, -Dale On Sat, Jan 19, 2008 at 01:23:01PM -0700, Dale Farnsworth wrote: > From: Byron Bradley <[EMAIL PROTECTED]> > > The Ma

[PATCH] mv643xx_eth: fix byte order when checksum offload is enabled

2008-01-19 Thread Dale Farnsworth
From: Byron Bradley <[EMAIL PROTECTED]> The Marvell Orion system on chips have an integrated mv643xx MAC. On these little endian ARM devices mv643xx will oops when checksum offload is enabled. Swapping the byte order of the protocol and checksum solves this problem. Signed-off-by: Byron Bradley <