diff options
author | Yaniv Rosner <yanivr@broadcom.com> | 2008-08-13 18:56:17 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-08-13 19:04:30 -0400 |
commit | 8c99e7b0436473593a68e740d1032909bc5335a1 (patch) | |
tree | 621557728da3ea1bb7cf4e4e08a4c184bf037b4e /drivers/net/bnx2x_link.h | |
parent | 57963ed94c27e94a7533434da5943195ea072a35 (diff) |
bnx2x: Pause settings
Pause settings
- 1G pause was not working due to missing write to the emac block
(TX_MODE_FLOW_EN)
- The flow control should use the negotiated result (after autoneg) so
we should save both the requested autoneg and the result
- The HW credits with flow control at 1G speed were not optimized and
caused low throughput
- It is recommended to turn off flow control if the MTU is bigger than
5000B due to internal buffers size
Signed-off-by: Yaniv Rosner <yanivr@broadcom.com>
Signed-off-by: Eilon Greenstein <eilong@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bnx2x_link.h')
-rw-r--r-- | drivers/net/bnx2x_link.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/bnx2x_link.h b/drivers/net/bnx2x_link.h index 714d37ac95de..b222552b868d 100644 --- a/drivers/net/bnx2x_link.h +++ b/drivers/net/bnx2x_link.h | |||
@@ -58,11 +58,13 @@ struct link_params { | |||
58 | 58 | ||
59 | u16 req_duplex; | 59 | u16 req_duplex; |
60 | u16 req_flow_ctrl; | 60 | u16 req_flow_ctrl; |
61 | u16 req_fc_auto_adv; /* Should be set to TX / BOTH when | ||
62 | req_flow_ctrl is set to AUTO */ | ||
61 | u16 req_line_speed; /* Also determine AutoNeg */ | 63 | u16 req_line_speed; /* Also determine AutoNeg */ |
62 | 64 | ||
63 | /* Device parameters */ | 65 | /* Device parameters */ |
64 | u8 mac_addr[6]; | 66 | u8 mac_addr[6]; |
65 | u16 mtu; | 67 | |
66 | 68 | ||
67 | 69 | ||
68 | /* shmem parameters */ | 70 | /* shmem parameters */ |