RE: [PATCH net-next v4 4/4] r8169: support new chips of RTL8111F

-----Original Message-----
From: Francois Romieu [mailto:romieu@fr.zoreil.com] 
Sent: Saturday, September 03, 2011 12:29 AM
To: Hayeswang
Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next v4 4/4] r8169: support new chips 
of RTL8111F

Hayes Wang [off-list ref] :

quoted

Support new chips of RTL8111F.

Amongst other things :o)

quoted

diff --git a/drivers/net/ethernet/realtek/r8169.c 

b/drivers/net/ethernet/realtek/r8169.c
index 175c769..8e6a200 100644

--- a/drivers/net/ethernet/realtek/r8169.c
+++ b/drivers/net/ethernet/realtek/r8169.c

[...]

quoted

@@ -711,7 +719,10 @@ MODULE_FIRMWARE(FIRMWARE_8168D_1);
 MODULE_FIRMWARE(FIRMWARE_8168D_2);
 MODULE_FIRMWARE(FIRMWARE_8168E_1);
 MODULE_FIRMWARE(FIRMWARE_8168E_2);
+MODULE_FIRMWARE(FIRMWARE_8168E_3);

This one is relevant for Linus's tree.

Don't worry about submitting again, I'll send it separately.

OK. I would adjust it again.

No opinion regarding the jumbo fixes patches I sent on 2011/07/17 ?

It seems fine except that you don't need to disable rx checksum when using jumbo
frame. The chips don't support tx checksum when using jumbo frame because of the
feature of early tx. The early tx means that the hw would start to send the
packet without loading the all data into fifo, so the software checksum is
necessary. However, there is no influence for rx, so you could still enable rx
checksum.
 
Best Regards,
Hayes