Re: [patch] dm9601: Fix receive MTU

2007-10-02 Thread Jeff Garzik

Peter Korsgaard wrote:

Please apply to 2.6.23.
---
dm9601 didn't take the ethernet header into account when calculating
RX MTU, causing packets bigger than 1486 to fail.

Signed-off-by: Peter Korsgaard [EMAIL PROTECTED]
---
 drivers/net/usb/dm9601.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-2.6.23-rc8/drivers/net/usb/dm9601.c
===
--- linux-2.6.23-rc8.orig/drivers/net/usb/dm9601.c
+++ linux-2.6.23-rc8/drivers/net/usb/dm9601.c
@@ -405,7 +405,7 @@
dev-net-ethtool_ops = dm9601_ethtool_ops;
dev-net-hard_header_len += DM_TX_OVERHEAD;
dev-hard_mtu = dev-net-mtu + dev-net-hard_header_len;
-   dev-rx_urb_size = dev-net-mtu + DM_RX_OVERHEAD;
+   dev-rx_urb_size = dev-net-mtu + ETH_HLEN + DM_RX_OVERHEAD;


applied.  in the future, please note (and avoid) a patch formatting 
error.  As noted in #14 of Documentation/SubmittingPatches, any comments 
on the patch like Please apply to 2.6.23 should go AFTER the patch 
description and --- separator.


Without hand editing, the normal tools everybody uses for importing 
patches would delete your dm9601 didn't take the ethernet header... 
description, leaving only the Please apply to 2.6.23. part and nothing 
else.


I did the hand-editing, but please try to follow the standard patch 
format in the future.



-
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[patch] dm9601: Fix receive MTU

2007-10-01 Thread Peter Korsgaard
Please apply to 2.6.23.
---
dm9601 didn't take the ethernet header into account when calculating
RX MTU, causing packets bigger than 1486 to fail.

Signed-off-by: Peter Korsgaard [EMAIL PROTECTED]
---
 drivers/net/usb/dm9601.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-2.6.23-rc8/drivers/net/usb/dm9601.c
===
--- linux-2.6.23-rc8.orig/drivers/net/usb/dm9601.c
+++ linux-2.6.23-rc8/drivers/net/usb/dm9601.c
@@ -405,7 +405,7 @@
dev-net-ethtool_ops = dm9601_ethtool_ops;
dev-net-hard_header_len += DM_TX_OVERHEAD;
dev-hard_mtu = dev-net-mtu + dev-net-hard_header_len;
-   dev-rx_urb_size = dev-net-mtu + DM_RX_OVERHEAD;
+   dev-rx_urb_size = dev-net-mtu + ETH_HLEN + DM_RX_OVERHEAD;
 
dev-mii.dev = dev-net;
dev-mii.mdio_read = dm9601_mdio_read;

-- 
Bye, Peter Korsgaard
-
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html