diff options
author | Steve Glendinning <steve.glendinning@smsc.com> | 2009-03-20 04:14:53 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-03-20 04:14:53 -0400 |
commit | cd7a3b75ba7af7d14a264fe3d414fcc74307f748 (patch) | |
tree | 7d8eac62d9dabccbbfb39828fb5db4811c9d91e1 /drivers | |
parent | 8fd17f2e6c973efdb2e3fcb5026d204441dd01fd (diff) |
smsc9420: fix big endian rx checksum offload
The cpu_to_le16 here looks suspicious to me, I don't think we need it
because put_unaligned_le16 also does this.
I don't currently have any big endian hardware with a PCI bus available
to test on, so I haven't been able to verify this.
Signed-off-by: Steve Glendinning <steve.glendinning@smsc.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/smsc9420.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/smsc9420.c b/drivers/net/smsc9420.c index 17560dbcc7ad..5959ae86e57d 100644 --- a/drivers/net/smsc9420.c +++ b/drivers/net/smsc9420.c | |||
@@ -807,7 +807,7 @@ static void smsc9420_rx_handoff(struct smsc9420_pdata *pd, const int index, | |||
807 | if (pd->rx_csum) { | 807 | if (pd->rx_csum) { |
808 | u16 hw_csum = get_unaligned_le16(skb_tail_pointer(skb) + | 808 | u16 hw_csum = get_unaligned_le16(skb_tail_pointer(skb) + |
809 | NET_IP_ALIGN + packet_length + 4); | 809 | NET_IP_ALIGN + packet_length + 4); |
810 | put_unaligned_le16(cpu_to_le16(hw_csum), &skb->csum); | 810 | put_unaligned_le16(hw_csum, &skb->csum); |
811 | skb->ip_summed = CHECKSUM_COMPLETE; | 811 | skb->ip_summed = CHECKSUM_COMPLETE; |
812 | } | 812 | } |
813 | 813 | ||