netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] e1000: endian fixes
@ 2006-05-15 22:54 Alexey Dobriyan
  2006-05-23 21:30 ` Auke Kok
  0 siblings, 1 reply; 2+ messages in thread
From: Alexey Dobriyan @ 2006-05-15 22:54 UTC (permalink / raw)
  To: Andrew Morton; +Cc: cramerj, netdev, Jeff Garzik

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---

 drivers/net/e1000/e1000_main.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

--- 1/drivers/net/e1000/e1000_main.c
+++ 1/drivers/net/e1000/e1000_main.c
@@ -2451,7 +2451,7 @@ e1000_tso(struct e1000_adapter *adapter,
 
 		hdr_len = ((skb->h.raw - skb->data) + (skb->h.th->doff << 2));
 		mss = skb_shinfo(skb)->tso_size;
-		if (skb->protocol == ntohs(ETH_P_IP)) {
+		if (skb->protocol == htons(ETH_P_IP)) {
 			skb->nh.iph->tot_len = 0;
 			skb->nh.iph->check = 0;
 			skb->h.th->check =
@@ -2930,7 +2930,7 @@ e1000_xmit_frame(struct sk_buff *skb, st
 	/* Old method was to assume IPv4 packet by default if TSO was enabled.
 	 * 82571 hardware supports TSO capabilities for IPv6 as well...
 	 * no longer assume, we must. */
-	if (likely(skb->protocol == ntohs(ETH_P_IP)))
+	if (likely(skb->protocol == htons(ETH_P_IP)))
 		tx_flags |= E1000_TX_FLAGS_IPV4;
 
 	e1000_tx_queue(adapter, tx_ring, tx_flags,


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] e1000: endian fixes
  2006-05-15 22:54 [PATCH] e1000: endian fixes Alexey Dobriyan
@ 2006-05-23 21:30 ` Auke Kok
  0 siblings, 0 replies; 2+ messages in thread
From: Auke Kok @ 2006-05-23 21:30 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: Alexey Dobriyan, Andrew Morton, netdev, Jesse Brandeburg


Ack, please apply and add my acked-by/signed-off-by.

Cheers,

Auke


---
Signed-off-by: Auke Kok <auke-jan.h.kok@intel.com>


Alexey Dobriyan wrote:
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> ---
> 
>  drivers/net/e1000/e1000_main.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> --- 1/drivers/net/e1000/e1000_main.c
> +++ 1/drivers/net/e1000/e1000_main.c
> @@ -2451,7 +2451,7 @@ e1000_tso(struct e1000_adapter *adapter,
>  
>  		hdr_len = ((skb->h.raw - skb->data) + (skb->h.th->doff << 2));
>  		mss = skb_shinfo(skb)->tso_size;
> -		if (skb->protocol == ntohs(ETH_P_IP)) {
> +		if (skb->protocol == htons(ETH_P_IP)) {
>  			skb->nh.iph->tot_len = 0;
>  			skb->nh.iph->check = 0;
>  			skb->h.th->check =
> @@ -2930,7 +2930,7 @@ e1000_xmit_frame(struct sk_buff *skb, st
>  	/* Old method was to assume IPv4 packet by default if TSO was enabled.
>  	 * 82571 hardware supports TSO capabilities for IPv6 as well...
>  	 * no longer assume, we must. */
> -	if (likely(skb->protocol == ntohs(ETH_P_IP)))
> +	if (likely(skb->protocol == htons(ETH_P_IP)))
>  		tx_flags |= E1000_TX_FLAGS_IPV4;
>  
>  	e1000_tx_queue(adapter, tx_ring, tx_flags,
> 
> -
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-05-23 21:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-05-15 22:54 [PATCH] e1000: endian fixes Alexey Dobriyan
2006-05-23 21:30 ` Auke Kok

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).