Re: [PATCH 2.6.17 0/9] NetXen: 1G/10G Ethernet Driver - patch for big-endian systems
From: Francois Romieu <romieu@fr.zoreil.com>
Date: 2006-08-24 21:44:01
wen xiong [off-list ref] : [...]
quoted hunk ↗ jump to hunk
diff -Nuar old/drivers/net/netxen/netxen_nic_hw.c new/drivers/net/netxen/netxen_nic_hw.c--- old/drivers/net/netxen/netxen_nic_hw.c 2006-08-23 12:58:43.000000000 -0500 +++ new/drivers/net/netxen/netxen_nic_hw.c 2006-08-23 13:15:19.000000000 -0500@@ -313,7 +313,8 @@ } } CMD_DESC_TCP_HDR_OFFSET_WRT(desc, skb->h.raw - skb->data); - desc->ip_hdr_offset = skb->nh.raw - skb->data; + desc->length_tcp_hdr=cpu_to_le32(desc->length_tcp_hdr);
s/=/ = / (several occurences) [...]
quoted hunk ↗ jump to hunk
diff -Nuar old/drivers/net/netxen/netxen_nic_init.c new/drivers/net/netxen/netxen_nic_init.c--- old/drivers/net/netxen/netxen_nic_init.c 2006-08-23 12:58:43.000000000 -0500 +++ new/drivers/net/netxen/netxen_nic_init.c 2006-08-23 13:15:19.000000000 -0500@@ -494,7 +494,7 @@ desc_head = recv_ctx->rcv_status_desc_head; desc = &desc_head[consumer]; - if ((desc->owner & STATUS_OWNER_HOST)) + if (((le16_to_cpu(desc->owner)) & STATUS_OWNER_HOST))
Would it make a difference to swab the constant part, i.e.: if (desc->owner & cpu_to_le16(STATUS_OWNER_HOST)) -- Ueimor