diff options
| author |   Rahul Lakkireddy <rahul.lakkireddy@chelsio.com> | 2019-12-30 18:14:08 +0530 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-12-30 14:40:42 -0800 |
| commit | 0caeaf6ad532f9be5a768a158627cb31921cc8b7 (patch) | |
| tree | 84a55bc3e51ec51d3948cd22ba140567c5890f79 /drivers/net/ethernet/chelsio/cxgb4/cxgb4.h | |
| parent | Merge branch 'mlxsw-fixes' (diff) | |
| download | linux-dev-0caeaf6ad532f9be5a768a158627cb31921cc8b7.tar.xz linux-dev-0caeaf6ad532f9be5a768a158627cb31921cc8b7.zip | |
cxgb4/cxgb4vf: fix flow control display for auto negotiation
As per 802.3-2005, Section Two, Annex 28B, Table 28B-2 [1], when
_only_ Rx pause is enabled, both symmetric and asymmetric pause
towards local device must be enabled. Also, firmware returns the local
device's flow control pause params as part of advertised capabilities
and negotiated params as part of current link attributes. So, fix up
ethtool's flow control pause params fetch logic to read from acaps,
instead of linkattr.
[1] https://standards.ieee.org/standard/802_3-2005.html
Fixes: c3168cabe1af ("cxgb4/cxgbvf: Handle 32-bit fw port capabilities")
Signed-off-by: Surendra Mobiya <surendra@chelsio.com>
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/chelsio/cxgb4/cxgb4.h')
| -rw-r--r-- | drivers/net/ethernet/chelsio/cxgb4/cxgb4.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h b/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h index a70ac2097892..becee29f5df7 100644 --- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h +++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4.h @@ -504,6 +504,7 @@ struct link_config { enum cc_pause requested_fc; /* flow control user has requested */ enum cc_pause fc; /* actual link flow control */ + enum cc_pause advertised_fc; /* actual advertised flow control */ enum cc_fec requested_fec; /* Forward Error Correction: */ enum cc_fec fec; /* requested and actual in use */ |