[dpdk-dev] [PATCH 01/34] e1000/base: update readme and copyright

2015-10-14 Thread Wenzhuo Lu
Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/README | 4 ++-- drivers/net/e1000/base/e1000_80003es2lan.c | 2 +- drivers/net/e1000/base/e1000_80003es2lan.h | 2 +- drivers/net/e1000/base/e1000_82540.c | 2 +- drivers/net/e1000/base/e1000_82541.c | 2 +- drivers

[dpdk-dev] [PATCH 02/34] e1000/base: add new devices

2015-10-14 Thread Wenzhuo Lu
Add some new i218 devices. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.c | 4 drivers/net/e1000/base/e1000_defines.h | 2 ++ drivers/net/e1000/base/e1000_hw.h | 4 drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ 4 files changed, 24

[dpdk-dev] [PATCH 03/34] e1000/base: fix issue with link flap on 82579

2015-10-14 Thread Wenzhuo Lu
wake mechanism. To fix the issue, we are disabling the Phase Locked Loop shutdown in 100M Low Power Idle. This solution will cause an increase of power in 100M EEE link. It may cost additional 28mW in this specific mode. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11

[dpdk-dev] [PATCH 05/34] e1000/base: redundant PHY power down for i210

2015-10-14 Thread Wenzhuo Lu
The wrong bit is being used in PHYREG16 for PHY power down. In addition, the use of PHYREG 16 is unnecessary if bit 11 of PHYREG 0 is used. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 15 --- drivers/net/e1000/base/e1000_phy.h | 1 - 2 files changed, 16

[dpdk-dev] [PATCH 06/34] e1000/base: add return value to the functions of setting receive address register

2015-10-14 Thread Wenzhuo Lu
-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82542.c | 6 -- drivers/net/e1000/base/e1000_api.c | 6 -- drivers/net/e1000/base/e1000_api.h | 2 +- drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ drivers/net

[dpdk-dev] [PATCH 04/34] e1000/base: fix issue with jumbo frame CRC failures in client

2015-10-14 Thread Wenzhuo Lu
This is a patch to change the value of register 776.20[11:2] for jumbo mode from 0x1A to 0x1F. This is to enlarge the gap between read and write pointers in the TX Fifo. And replace the magic number with a macro by the way. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h

[dpdk-dev] [PATCH 07/34] e1000/base: add defaults for i210 TX/RX PBSIZE

2015-10-14 Thread Wenzhuo Lu
These are the defaults for the packet buffer size registers that need to be explicitly set back if someone changes them and comes back to a normal driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000

[dpdk-dev] [PATCH 08/34] e1000/base: remove E1000_WRITE_FLUSH for DH89XXCC_SGMII after commencing HW reset

2015-10-14 Thread Wenzhuo Lu
- removed the condition. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c index 3dc8066..ab547ca 100644 --- a/drivers

[dpdk-dev] [PATCH 10/34] e1000/base: change invariant return to not use variables

2015-10-14 Thread Wenzhuo Lu
Although this change should be optimized out by the compiler, just return a constant directly rather than declare a variable. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/net/e1000

[dpdk-dev] [PATCH 09/34] e1000/base: add evaluation of e1000_nvm_read return value

2015-10-14 Thread Wenzhuo Lu
, it only covers the code used by the e1000e driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_manage.c | 5 - drivers/net/e1000/base/e1000_nvm.c| 3 +++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_manage.c b/drivers/net/e1000

[dpdk-dev] [PATCH 11/34] e1000/base: add return value handler when check manage mode

2015-10-14 Thread Wenzhuo Lu
Adding code, where missing, to handle the case when hw->nvm.ops.read returns an error value. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82571.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_82571.c b/drivers/net/e1000/b

[dpdk-dev] [PATCH 12/34] e1000/base: add return value handler for ESB2 controller init and reset

2015-10-14 Thread Wenzhuo Lu
or write failure consistent between hw_init and hw_reset. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_80003es2lan.c | 31 -- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/drivers/net/e1000/base/e1000_80003es2lan.c b/drivers/net/e1000/base

[dpdk-dev] [PATCH 13/34] e1000/base: add support for inverted format ETrackId

2015-10-14 Thread Wenzhuo Lu
There are some images which contain ETrackID in inverted format. This patch allows reading this format. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_nvm.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_nvm.c b/drivers/net

[dpdk-dev] [PATCH 14/34] e1000/base: add EEARBC_I210 for i210

2015-10-14 Thread Wenzhuo Lu
EEARBC has changed on i210. It means EEARBC has a different address on i210 than on other NICs. So, add a new entity named EEARBC_I210 to the register list and make sure the right one is being used on i210. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 17

[dpdk-dev] [PATCH 15/34] e1000/base: apply paranoia to macro arguments

2015-10-14 Thread Wenzhuo Lu
Macro arguments need to be in parens since we can pass in expressions. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.h | 16 drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_regs.h | 4 ++-- 3 files changed, 11 insertions(+), 11

[dpdk-dev] [PATCH 16/34] e1000/base: add flags to set eee advertisement modes

2015-10-14 Thread Wenzhuo Lu
!!! REQUIRES DRIVER CHANGES !!! Change e1000_set_eee_i350 and e1000_set_eee_i354 to have flags allowing changes in the advertised EEE speeds. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 34 +++--- drivers/net/e1000/base/e1000_82575.h | 4

[dpdk-dev] [PATCH 17/34] e1000/base: prevent ulp flow if cable connected

2015-10-14 Thread Wenzhuo Lu
Enabling ulp on link down when cable is connect caused an infinite loop of linkup/down indications in the NDIS driver. After discussed, correct flow is to enable ULP only when cable is disconnected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 3 +++ 1 file changed, 3

[dpdk-dev] [PATCH 18/34] e1000/base: fix TIPG value for non 10 half duplex mode

2015-10-14 Thread Wenzhuo Lu
TIPG value is increased when setting speed to 10 half to prevent packet loss. However, it was never decreased again when speed changes. This caused performance issues in the NDIS driver. Fix this to restore TIPG to default value on non 10 half. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000

[dpdk-dev] [PATCH 19/34] e1000/base: add return value for resume workaround

2015-10-14 Thread Wenzhuo Lu
Add u32 return value to function e1000_resume_workarounds_pchlan, so that calling function can detect PHY access failure during resuming flow. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11 ++- drivers/net/e1000/base/e1000_ich8lan.h | 2 +- 2 files changed

[dpdk-dev] [PATCH 20/34] e1000/base: fix link detect flow

2015-10-14 Thread Wenzhuo Lu
In case that auto-negotiate is not enabled, call e1000_setup_copper_link_generic instead of e1000_phy_setup_autoneg. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base

[dpdk-dev] [PATCH 22/34] e1000/base: add bit for disable packetbuffer read

2015-10-14 Thread Wenzhuo Lu
Added bit FEXTNVM7[18], that controls disabling MAC packet buffer read. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.h b/drivers/net/e1000/base/e1000_ich8lan.h index f5d8ab1

[dpdk-dev] [PATCH 23/34] e1000/base: K1 flow fixes

2015-10-14 Thread Wenzhuo Lu
This patch is for the following updates to the K1 configurations: Tx idle period for entering K1 should be 128 ns. Minimum Tx idle period in K1 should be 256 ns. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 47 +- drivers/net/e1000/base

[dpdk-dev] [PATCH 25/34] e1000/base: set correct value of beacon duration

2015-10-14 Thread Wenzhuo Lu
Fix for I217 Packet Loss issue - The Management Engine sets the FEXTNVM4 Beacon Duration incorrectly. This fix ensures that the correct value will always be set. Correct value for this field is 8 usec. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 14 ++ 1

[dpdk-dev] [PATCH 27/34] e1000/base: fix for i354 88E1112 PHY using AutoMedia Detect

2015-10-14 Thread Wenzhuo Lu
link type is controlled by the PHY settings. If the link is copper, as the M88E1112 page address is set to 1, it should be set back to 0 before checking this link. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 23 +++ 1 file changed, 15 insert

[dpdk-dev] [PATCH 28/34] e1000/base: increase timeout of polling bit RSPCIPHY in check_reset_block

2015-10-14 Thread Wenzhuo Lu
Previously, in check_reset_block RSPCIPHY was polled for 100 ms before determining that the ME veto is set. This needed to be increased to 300 ms. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[dpdk-dev] [PATCH 21/34] e1000/base: cleanup NAHUM6LP_HW tags

2015-10-14 Thread Wenzhuo Lu
Remove all NAHUM6LP_HW tags. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_hw.h | 6 +++--- drivers/net/e1000/base/e1000_ich8lan.c | 9 + drivers/net/e1000/base/e1000_ich8lan.h | 29 ++--- drivers/net/e1000/base/e1000_osdep.h | 1 - drivers

[dpdk-dev] [PATCH 30/34] e1000/base: use the correct i210 register for EEMNGCTL

2015-10-14 Thread Wenzhuo Lu
The i210 has two EEPROM access registers that are located in non-standard offsets: EEARBC and EEMNGCTL. EEARBC was fixed previously and EEMNGCTL should also be corrected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 30 ++ drivers/net/e1000

[dpdk-dev] [PATCH 31/34] e1000/base: move the print to the right position

2015-10-14 Thread Wenzhuo Lu
This info need not to be always printed. Move it into the if. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/e1000/base/e1000_phy.c b/drivers/net/e1000/base/e1000_phy.c index 6bbb379..d43b7ce

[dpdk-dev] [PATCH 32/34] e1000/base: synchronization of MAC-PHY interface only on non- ME systems

2015-10-14 Thread Wenzhuo Lu
On power up, the MAC - PHY interface needs to be set to PCIe, even if cable is disconnected. In ME systems, the ME handles this on exit from Sx(Sticky mode) state. In non-ME, the driver handles it. Added a check for non-ME system to the driver code that handles that. Signed-off-by: Wenzhuo Lu

[dpdk-dev] [PATCH 33/34] e1000/base: fix to enable both ulp and EEE in Sx state

2015-10-14 Thread Wenzhuo Lu
This patch implements a modified flow that allows both ULP and EEE in Sx(Sticky mode). Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.c b/drivers/net/e1000/base/e1000_ich8lan.c

[dpdk-dev] [PATCH 34/34] e1000/base: some minor change

2015-10-14 Thread Wenzhuo Lu
Some minor code change. No functionality impact. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 27 --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + 2 files changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/net/e1000/base

[dpdk-dev] [PATCH 29/34] e1000/base: implement 88E1543 PHY initialization

2015-10-14 Thread Wenzhuo Lu
The initializtion process for 88E1543 PHY. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 106 - drivers/net/e1000/base/e1000_82575.h | 1 + drivers/net/e1000/base/e1000_defines.h | 1 + 3 files changed, 107 insertions(+), 1 deletion

[dpdk-dev] [PATCH v2 01/35] e1000/base: update readme and copyright

2015-10-15 Thread Wenzhuo Lu
Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/README | 4 ++-- drivers/net/e1000/base/e1000_80003es2lan.c | 2 +- drivers/net/e1000/base/e1000_80003es2lan.h | 2 +- drivers/net/e1000/base/e1000_82540.c | 2 +- drivers/net/e1000/base/e1000_82541.c | 2 +- drivers

[dpdk-dev] [PATCH v2 02/35] e1000/base: add new devices

2015-10-15 Thread Wenzhuo Lu
Add some new i218 devices. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.c | 4 drivers/net/e1000/base/e1000_defines.h | 2 ++ drivers/net/e1000/base/e1000_hw.h | 4 drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ 4 files changed, 24

[dpdk-dev] [PATCH v2 03/35] e1000/base: fix issue with link flap on 82579

2015-10-15 Thread Wenzhuo Lu
wake mechanism. To fix the issue, we are disabling the Phase Locked Loop shutdown in 100M Low Power Idle. This solution will cause an increase of power in 100M EEE link. It may cost additional 28mW in this specific mode. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11

[dpdk-dev] [PATCH v2 04/35] e1000/base: fix issue with jumbo frame CRC failures in client

2015-10-15 Thread Wenzhuo Lu
This is a patch to change the value of register 776.20[11:2] for jumbo mode from 0x1A to 0x1F. This is to enlarge the gap between read and write pointers in the TX Fifo. And replace the magic number with a macro by the way. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h

[dpdk-dev] [PATCH v2 05/35] e1000/base: redundant PHY power down for i210

2015-10-15 Thread Wenzhuo Lu
The wrong bit is being used in PHYREG16 for PHY power down. In addition, the use of PHYREG 16 is unnecessary if bit 11 of PHYREG 0 is used. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 15 --- drivers/net/e1000/base/e1000_phy.h | 1 - 2 files changed, 16

[dpdk-dev] [PATCH v2 06/35] e1000/base: add return value to the functions of setting receive address register

2015-10-15 Thread Wenzhuo Lu
-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82542.c | 6 -- drivers/net/e1000/base/e1000_api.c | 6 -- drivers/net/e1000/base/e1000_api.h | 2 +- drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ drivers/net

[dpdk-dev] [PATCH v2 07/35] e1000/base: add defaults for i210 TX/RX PBSIZE

2015-10-15 Thread Wenzhuo Lu
These are the defaults for the packet buffer size registers that need to be explicitly set back if someone changes them and comes back to a normal driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000

[dpdk-dev] [PATCH v2 08/35] e1000/base: remove E1000_WRITE_FLUSH for DH89XXCC_SGMII after commencing HW reset

2015-10-15 Thread Wenzhuo Lu
- removed the condition. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c index 3dc8066..ab547ca 100644 --- a/drivers

[dpdk-dev] [PATCH v2 10/35] e1000/base: change invariant return to not use variables

2015-10-15 Thread Wenzhuo Lu
Although this change should be optimized out by the compiler, just return a constant directly rather than declare a variable. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/net/e1000

[dpdk-dev] [PATCH v2 00/35] update e1000 base code

2015-10-15 Thread Wenzhuo Lu
the correct i210 register for EEMNGCTL *move the print to the right position *synchronization of MAC-PHY interface only on non- ME systems *fix to enable both ulp and EEE in Sx state V2: Add the new devices to the DPDK PCI device list. Wenzhuo Lu (35): e1000/base: update readme and copyright e1000/base

[dpdk-dev] [PATCH v2 12/35] e1000/base: add return value handler for ESB2 controller init and reset

2015-10-15 Thread Wenzhuo Lu
or write failure consistent between hw_init and hw_reset. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_80003es2lan.c | 31 -- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/drivers/net/e1000/base/e1000_80003es2lan.c b/drivers/net/e1000/base

[dpdk-dev] [PATCH v2 09/35] e1000/base: add evaluation of e1000_nvm_read return value

2015-10-15 Thread Wenzhuo Lu
, it only covers the code used by the e1000e driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_manage.c | 5 - drivers/net/e1000/base/e1000_nvm.c| 3 +++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_manage.c b/drivers/net/e1000

[dpdk-dev] [PATCH v2 11/35] e1000/base: add return value handler when check manage mode

2015-10-15 Thread Wenzhuo Lu
Adding code, where missing, to handle the case when hw->nvm.ops.read returns an error value. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82571.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_82571.c b/drivers/net/e1000/b

[dpdk-dev] [PATCH v2 13/35] e1000/base: add support for inverted format ETrackId

2015-10-15 Thread Wenzhuo Lu
There are some images which contain ETrackID in inverted format. This patch allows reading this format. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_nvm.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_nvm.c b/drivers/net

[dpdk-dev] [PATCH v2 15/35] e1000/base: apply paranoia to macro arguments

2015-10-15 Thread Wenzhuo Lu
Macro arguments need to be in parens since we can pass in expressions. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.h | 16 drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_regs.h | 4 ++-- 3 files changed, 11 insertions(+), 11

[dpdk-dev] [PATCH v2 16/35] e1000/base: add flags to set eee advertisement modes

2015-10-15 Thread Wenzhuo Lu
!!! REQUIRES DRIVER CHANGES !!! Change e1000_set_eee_i350 and e1000_set_eee_i354 to have flags allowing changes in the advertised EEE speeds. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 34 +++--- drivers/net/e1000/base/e1000_82575.h | 4

[dpdk-dev] [PATCH v2 17/35] e1000/base: prevent ulp flow if cable connected

2015-10-15 Thread Wenzhuo Lu
Enabling ulp on link down when cable is connect caused an infinite loop of linkup/down indications in the NDIS driver. After discussed, correct flow is to enable ULP only when cable is disconnected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 3 +++ 1 file changed, 3

[dpdk-dev] [PATCH v2 19/35] e1000/base: add return value for resume workaround

2015-10-15 Thread Wenzhuo Lu
Add u32 return value to function e1000_resume_workarounds_pchlan, so that calling function can detect PHY access failure during resuming flow. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11 ++- drivers/net/e1000/base/e1000_ich8lan.h | 2 +- 2 files changed

[dpdk-dev] [PATCH v2 20/35] e1000/base: fix link detect flow

2015-10-15 Thread Wenzhuo Lu
In case that auto-negotiate is not enabled, call e1000_setup_copper_link_generic instead of e1000_phy_setup_autoneg. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base

[dpdk-dev] [PATCH v2 21/35] e1000/base: cleanup NAHUM6LP_HW tags

2015-10-15 Thread Wenzhuo Lu
Remove all NAHUM6LP_HW tags. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_hw.h | 6 +++--- drivers/net/e1000/base/e1000_ich8lan.c | 9 + drivers/net/e1000/base/e1000_ich8lan.h | 29 ++--- drivers/net/e1000/base/e1000_osdep.h | 1 - drivers

[dpdk-dev] [PATCH v2 22/35] e1000/base: add bit for disable packetbuffer read

2015-10-15 Thread Wenzhuo Lu
Added bit FEXTNVM7[18], that controls disabling MAC packet buffer read. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.h b/drivers/net/e1000/base/e1000_ich8lan.h index f5d8ab1

[dpdk-dev] [PATCH v2 24/35] e1000/base: remove FIXME comment

2015-10-15 Thread Wenzhuo Lu
The "FIXME" comment is revomed from e1000_acquire_swfw_sync_80003es2lan but forgotten being removed from e1000_acquire_swfw_sync_82575 while the similar changes were made to both. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 2 +- 1 file changed, 1 inser

[dpdk-dev] [PATCH v2 26/35] e1000/base: disable extension header parsing for IPv6

2015-10-15 Thread Wenzhuo Lu
All 1G Server products need to have IPv6 extension headers turned off. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c

[dpdk-dev] [PATCH v2 27/35] e1000/base: fix for i354 88E1112 PHY using AutoMedia Detect

2015-10-15 Thread Wenzhuo Lu
link type is controlled by the PHY settings. If the link is copper, as the M88E1112 page address is set to 1, it should be set back to 0 before checking this link. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 23 +++ 1 file changed, 15 insert

[dpdk-dev] [PATCH v2 28/35] e1000/base: increase timeout of polling bit RSPCIPHY in check_reset_block

2015-10-15 Thread Wenzhuo Lu
Previously, in check_reset_block RSPCIPHY was polled for 100 ms before determining that the ME veto is set. This needed to be increased to 300 ms. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[dpdk-dev] [PATCH v2 23/35] e1000/base: K1 flow fixes

2015-10-15 Thread Wenzhuo Lu
This patch is for the following updates to the K1 configurations: Tx idle period for entering K1 should be 128 ns. Minimum Tx idle period in K1 should be 256 ns. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 47 +- drivers/net/e1000/base

[dpdk-dev] [PATCH v2 25/35] e1000/base: set correct value of beacon duration

2015-10-15 Thread Wenzhuo Lu
Fix for I217 Packet Loss issue - The Management Engine sets the FEXTNVM4 Beacon Duration incorrectly. This fix ensures that the correct value will always be set. Correct value for this field is 8 usec. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 14 ++ 1

[dpdk-dev] [PATCH v2 29/35] e1000/base: implement 88E1543 PHY initialization

2015-10-15 Thread Wenzhuo Lu
The initializtion process for 88E1543 PHY. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 106 - drivers/net/e1000/base/e1000_82575.h | 1 + drivers/net/e1000/base/e1000_defines.h | 1 + 3 files changed, 107 insertions(+), 1 deletion

[dpdk-dev] [PATCH v2 30/35] e1000/base: use the correct i210 register for EEMNGCTL

2015-10-15 Thread Wenzhuo Lu
The i210 has two EEPROM access registers that are located in non-standard offsets: EEARBC and EEMNGCTL. EEARBC was fixed previously and EEMNGCTL should also be corrected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 30 ++ drivers/net/e1000

[dpdk-dev] [PATCH v2 31/35] e1000/base: move the print to the right position

2015-10-15 Thread Wenzhuo Lu
This info need not to be always printed. Move it into the if. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/e1000/base/e1000_phy.c b/drivers/net/e1000/base/e1000_phy.c index 6bbb379..d43b7ce

[dpdk-dev] [PATCH v2 33/35] e1000/base: fix to enable both ulp and EEE in Sx state

2015-10-15 Thread Wenzhuo Lu
This patch implements a modified flow that allows both ULP and EEE in Sx(Sticky mode). Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.c b/drivers/net/e1000/base/e1000_ich8lan.c

[dpdk-dev] [PATCH v2 34/35] e1000/base: some minor change

2015-10-15 Thread Wenzhuo Lu
Some minor code change. No functionality impact. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 27 --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + 2 files changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/net/e1000/base

[dpdk-dev] [PATCH v2 35/35] e1000: add new devices

2015-10-15 Thread Wenzhuo Lu
Add the new e1000 devices to the DPDK PCI device list. Signed-off-by: Wenzhuo Lu --- lib/librte_eal/common/include/rte_pci_dev_ids.h | 5 + 1 file changed, 5 insertions(+) diff --git a/lib/librte_eal/common/include/rte_pci_dev_ids.h b/lib/librte_eal/common/include/rte_pci_dev_ids.h index

[dpdk-dev] [PATCH v3 01/36] e1000/base: update readme and copyright

2015-10-16 Thread Wenzhuo Lu
Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/README | 4 ++-- drivers/net/e1000/base/e1000_80003es2lan.c | 2 +- drivers/net/e1000/base/e1000_80003es2lan.h | 2 +- drivers/net/e1000/base/e1000_82540.c | 2 +- drivers/net/e1000/base/e1000_82541.c | 2 +- drivers

[dpdk-dev] [PATCH v3 02/36] e1000/base: add new devices

2015-10-16 Thread Wenzhuo Lu
Add some new i218 devices. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.c | 4 drivers/net/e1000/base/e1000_defines.h | 2 ++ drivers/net/e1000/base/e1000_hw.h | 4 drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ 4 files changed, 24

[dpdk-dev] [PATCH v3 04/36] e1000/base: fix issue with jumbo frame CRC failures in client

2015-10-16 Thread Wenzhuo Lu
This is a patch to change the value of register 776.20[11:2] for jumbo mode from 0x1A to 0x1F. This is to enlarge the gap between read and write pointers in the TX Fifo. And replace the magic number with a macro by the way. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h

[dpdk-dev] [PATCH v3 03/36] e1000/base: fix issue with link flap on 82579

2015-10-16 Thread Wenzhuo Lu
wake mechanism. To fix the issue, we are disabling the Phase Locked Loop shutdown in 100M Low Power Idle. This solution will cause an increase of power in 100M EEE link. It may cost additional 28mW in this specific mode. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11

[dpdk-dev] [PATCH v3 05/36] e1000/base: redundant PHY power down for i210

2015-10-16 Thread Wenzhuo Lu
The wrong bit is being used in PHYREG16 for PHY power down. In addition, the use of PHYREG 16 is unnecessary if bit 11 of PHYREG 0 is used. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 15 --- drivers/net/e1000/base/e1000_phy.h | 1 - 2 files changed, 16

[dpdk-dev] [PATCH v3 06/36] e1000/base: add return value to the functions of setting receive address register

2015-10-16 Thread Wenzhuo Lu
-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82542.c | 6 -- drivers/net/e1000/base/e1000_api.c | 6 -- drivers/net/e1000/base/e1000_api.h | 2 +- drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_ich8lan.c | 18 ++ drivers/net

[dpdk-dev] [PATCH v3 07/36] e1000/base: add defaults for i210 TX/RX PBSIZE

2015-10-16 Thread Wenzhuo Lu
These are the defaults for the packet buffer size registers that need to be explicitly set back if someone changes them and comes back to a normal driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_defines.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000

[dpdk-dev] [PATCH v3 08/36] e1000/base: remove E1000_WRITE_FLUSH for DH89XXCC_SGMII after commencing HW reset

2015-10-16 Thread Wenzhuo Lu
- removed the condition. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c index 3dc8066..ab547ca 100644 --- a/drivers

[dpdk-dev] [PATCH v3 09/36] e1000/base: add evaluation of e1000_nvm_read return value

2015-10-16 Thread Wenzhuo Lu
, it only covers the code used by the e1000e driver. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_manage.c | 5 - drivers/net/e1000/base/e1000_nvm.c| 3 +++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_manage.c b/drivers/net/e1000

[dpdk-dev] [PATCH v3 10/36] e1000/base: change invariant return to not use variables

2015-10-16 Thread Wenzhuo Lu
Although this change should be optimized out by the compiler, just return a constant directly rather than declare a variable. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/net/e1000

[dpdk-dev] [PATCH v3 13/36] e1000/base: add support for inverted format ETrackId

2015-10-16 Thread Wenzhuo Lu
There are some images which contain ETrackID in inverted format. This patch allows reading this format. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_nvm.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_nvm.c b/drivers/net

[dpdk-dev] [PATCH v3 12/36] e1000/base: add return value handler for ESB2 controller init and reset

2015-10-16 Thread Wenzhuo Lu
or write failure consistent between hw_init and hw_reset. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_80003es2lan.c | 31 -- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/drivers/net/e1000/base/e1000_80003es2lan.c b/drivers/net/e1000/base

[dpdk-dev] [PATCH v3 14/36] e1000/base: add EEARBC_I210 for i210

2015-10-16 Thread Wenzhuo Lu
EEARBC has changed on i210. It means EEARBC has a different address on i210 than on other NICs. So, add a new entity named EEARBC_I210 to the register list and make sure the right one is being used on i210. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 17

[dpdk-dev] [PATCH v3 15/36] e1000/base: apply paranoia to macro arguments

2015-10-16 Thread Wenzhuo Lu
Macro arguments need to be in parens since we can pass in expressions. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_api.h | 16 drivers/net/e1000/base/e1000_hw.h | 2 +- drivers/net/e1000/base/e1000_regs.h | 4 ++-- 3 files changed, 11 insertions(+), 11

[dpdk-dev] [PATCH v3 11/36] e1000/base: add return value handler when check manage mode

2015-10-16 Thread Wenzhuo Lu
Adding code, where missing, to handle the case when hw->nvm.ops.read returns an error value. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82571.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/e1000/base/e1000_82571.c b/drivers/net/e1000/b

[dpdk-dev] [PATCH v3 18/36] e1000/base: fix TIPG value for non 10 half duplex mode

2015-10-16 Thread Wenzhuo Lu
TIPG value is increased when setting speed to 10 half to prevent packet loss. However, it was never decreased again when speed changes. This caused performance issues in the NDIS driver. Fix this to restore TIPG to default value on non 10 half. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000

[dpdk-dev] [PATCH v3 19/36] e1000/base: add return value for resume workaround

2015-10-16 Thread Wenzhuo Lu
Add u32 return value to function e1000_resume_workarounds_pchlan, so that calling function can detect PHY access failure during resuming flow. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 11 ++- drivers/net/e1000/base/e1000_ich8lan.h | 2 +- 2 files changed

[dpdk-dev] [PATCH v3 20/36] e1000/base: fix link detect flow

2015-10-16 Thread Wenzhuo Lu
In case that auto-negotiate is not enabled, call e1000_setup_copper_link_generic instead of e1000_phy_setup_autoneg. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/net/e1000/base

[dpdk-dev] [PATCH v3 17/36] e1000/base: prevent ulp flow if cable connected

2015-10-16 Thread Wenzhuo Lu
Enabling ulp on link down when cable is connect caused an infinite loop of linkup/down indications in the NDIS driver. After discussed, correct flow is to enable ULP only when cable is disconnected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 3 +++ 1 file changed, 3

[dpdk-dev] [PATCH v3 22/36] e1000/base: add bit for disable packetbuffer read

2015-10-16 Thread Wenzhuo Lu
Added bit FEXTNVM7[18], that controls disabling MAC packet buffer read. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.h b/drivers/net/e1000/base/e1000_ich8lan.h index f5d8ab1

[dpdk-dev] [PATCH v3 23/36] e1000/base: K1 flow fixes

2015-10-16 Thread Wenzhuo Lu
This patch is for the following updates to the K1 configurations: Tx idle period for entering K1 should be 128 ns. Minimum Tx idle period in K1 should be 256 ns. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 47 +- drivers/net/e1000/base

[dpdk-dev] [PATCH v3 24/36] e1000/base: remove FIXME comment

2015-10-16 Thread Wenzhuo Lu
The "FIXME" comment is revomed from e1000_acquire_swfw_sync_80003es2lan but forgotten being removed from e1000_acquire_swfw_sync_82575 while the similar changes were made to both. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 2 +- 1 file changed, 1 inser

[dpdk-dev] [PATCH v3 25/36] e1000/base: set correct value of beacon duration

2015-10-16 Thread Wenzhuo Lu
Fix for I217 Packet Loss issue - The Management Engine sets the FEXTNVM4 Beacon Duration incorrectly. This fix ensures that the correct value will always be set. Correct value for this field is 8 usec. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 14 ++ 1

[dpdk-dev] [PATCH v3 26/36] e1000/base: disable extension header parsing for IPv6

2015-10-16 Thread Wenzhuo Lu
All 1G Server products need to have IPv6 extension headers turned off. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c

[dpdk-dev] [PATCH v3 27/36] e1000/base: fix for i354 88E1112 PHY using AutoMedia Detect

2015-10-16 Thread Wenzhuo Lu
link type is controlled by the PHY settings. If the link is copper, as the M88E1112 page address is set to 1, it should be set back to 0 before checking this link. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_82575.c | 23 +++ 1 file changed, 15 insert

[dpdk-dev] [PATCH v3 28/36] e1000/base: increase timeout of polling bit RSPCIPHY in check_reset_block

2015-10-16 Thread Wenzhuo Lu
Previously, in check_reset_block RSPCIPHY was polled for 100 ms before determining that the ME veto is set. This needed to be increased to 300 ms. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[dpdk-dev] [PATCH v3 30/36] e1000/base: use the correct i210 register for EEMNGCTL

2015-10-16 Thread Wenzhuo Lu
The i210 has two EEPROM access registers that are located in non-standard offsets: EEARBC and EEMNGCTL. EEARBC was fixed previously and EEMNGCTL should also be corrected. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_i210.c | 30 ++ drivers/net/e1000

[dpdk-dev] [PATCH v3 31/36] e1000/base: move the print to the right position

2015-10-16 Thread Wenzhuo Lu
This info need not to be always printed. Move it into the if. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_phy.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/e1000/base/e1000_phy.c b/drivers/net/e1000/base/e1000_phy.c index 6bbb379..d43b7ce

[dpdk-dev] [PATCH v3 32/36] e1000/base: synchronization of MAC-PHY interface only on non- ME systems

2015-10-16 Thread Wenzhuo Lu
On power up, the MAC - PHY interface needs to be set to PCIe, even if cable is disconnected. In ME systems, the ME handles this on exit from Sx(Sticky mode) state. In non-ME, the driver handles it. Added a check for non-ME system to the driver code that handles that. Signed-off-by: Wenzhuo Lu

[dpdk-dev] [PATCH v3 33/36] e1000/base: fix to enable both ulp and EEE in Sx state

2015-10-16 Thread Wenzhuo Lu
This patch implements a modified flow that allows both ULP and EEE in Sx(Sticky mode). Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/e1000/base/e1000_ich8lan.c b/drivers/net/e1000/base/e1000_ich8lan.c

[dpdk-dev] [PATCH v3 34/36] e1000/base: some minor change

2015-10-16 Thread Wenzhuo Lu
Some minor code change. No functionality impact. Signed-off-by: Wenzhuo Lu --- drivers/net/e1000/base/e1000_ich8lan.c | 27 --- drivers/net/e1000/base/e1000_ich8lan.h | 1 + 2 files changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/net/e1000/base

[dpdk-dev] [PATCH v3 35/36] e1000: add new devices

2015-10-16 Thread Wenzhuo Lu
Add the new e1000 devices to the DPDK PCI device list. Signed-off-by: Wenzhuo Lu --- lib/librte_eal/common/include/rte_pci_dev_ids.h | 5 + 1 file changed, 5 insertions(+) diff --git a/lib/librte_eal/common/include/rte_pci_dev_ids.h b/lib/librte_eal/common/include/rte_pci_dev_ids.h index

[dpdk-dev] [PATCH v3 36/36] doc: update release notes for e1000 base code update

2015-10-16 Thread Wenzhuo Lu
Signed-off-by: Wenzhuo Lu --- doc/guides/rel_notes/release_2_2.rst | 22 ++ 1 file changed, 22 insertions(+) diff --git a/doc/guides/rel_notes/release_2_2.rst b/doc/guides/rel_notes/release_2_2.rst index 5687676..36a9d69 100644 --- a/doc/guides/rel_notes/release_2_2.rst

[dpdk-dev] [PATCH v2 0/4] RSS enhancement on Intel x550 NIC

2015-10-16 Thread Wenzhuo Lu
. Wenzhuo Lu (4): ixgbe: 512 entries RSS table on x550 ixgbe: VF RSS config on x550 ixgbe: VF RSS reta/hash query and update doc: release notes update for RSS enhancement doc/guides/rel_notes/release_2_2.rst | 5 ++ drivers/net/ixgbe/ixgbe_ethdev.c | 102

[dpdk-dev] [PATCH v2 1/4] ixgbe: 512 entries RSS table on x550

2015-10-16 Thread Wenzhuo Lu
. This patch configs the entries beyond 128 on x550 to let RSS work well, and also update the query and update functions to support 512 entries. Signed-off-by: Wenzhuo Lu --- drivers/net/ixgbe/ixgbe_ethdev.c | 57 +--- drivers/net/ixgbe/ixgbe_ethdev.h | 4

<    1   2   3   4   5   >