[dpdk-dev,1/2] net/virtio: fix link autoneg value

Message ID 20180316105155.38843-1-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Ferruh Yigit March 16, 2018, 10:51 a.m. UTC
  link_autoneg shold use ETH_LINK_FIXED, not ETH_LINK_SPEED_FIXED

Fixes: 138f367aec66 ("net/virtio: use linkstatus helper functions")
Cc: stephen@networkplumber.org

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 drivers/net/virtio/virtio_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Ferruh Yigit March 16, 2018, 11:06 a.m. UTC | #1
On 3/16/2018 10:51 AM, Ferruh Yigit wrote:
> link_autoneg shold use ETH_LINK_FIXED, not ETH_LINK_SPEED_FIXED
> 
> Fixes: 138f367aec66 ("net/virtio: use linkstatus helper functions")
> Cc: stephen@networkplumber.org
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

Series
Squashed into relevant commit in next-net, thanks.
  

Patch

diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
index 92a85e2c2..4dddb1c80 100644
--- a/drivers/net/virtio/virtio_ethdev.c
+++ b/drivers/net/virtio/virtio_ethdev.c
@@ -2001,7 +2001,7 @@  virtio_dev_link_update(struct rte_eth_dev *dev, __rte_unused int wait_to_complet
 	memset(&link, 0, sizeof(link));
 	link.link_duplex = ETH_LINK_FULL_DUPLEX;
 	link.link_speed  = ETH_SPEED_NUM_10G;
-	link.link_autoneg = ETH_LINK_SPEED_FIXED;
+	link.link_autoneg = ETH_LINK_FIXED;
 
 	if (hw->started == 0) {
 		link.link_status = ETH_LINK_DOWN;