diff mbox series

net/txgbe: fix link macro

Message ID 20211101131932.3398023-1-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series net/txgbe: fix link macro |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/github-robot: build success github build: passed
ci/intel-Testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-x86_64-compile-testing fail Testing issues
ci/iol-x86_64-unit-testing fail Testing issues
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS

Commit Message

Ferruh Yigit Nov. 1, 2021, 1:19 p.m. UTC 
  Macro is changed unintentionally while adding RTE_ prefix, fixing the
original value.

Fixes: 295968d17407 ("ethdev: add namespace")

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

Comments

Jiawen Wu Nov. 2, 2021, 1:51 a.m. UTC | #1 
On November 1, 2021 9:20 PM, Ferruh Yigit wrote:
> Macro is changed unintentionally while adding RTE_ prefix, fixing the
original
> value.
> 
> Fixes: 295968d17407 ("ethdev: add namespace")
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  drivers/net/txgbe/txgbe_ethdev.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/txgbe/txgbe_ethdev.c
> b/drivers/net/txgbe/txgbe_ethdev.c
> index 169272ded5b0..fde9914e493f 100644
> --- a/drivers/net/txgbe/txgbe_ethdev.c
> +++ b/drivers/net/txgbe/txgbe_ethdev.c
> @@ -2693,7 +2693,7 @@ txgbe_dev_link_update_share(struct rte_eth_dev
> *dev,
>  	link.link_speed = RTE_ETH_SPEED_NUM_NONE;
>  	link.link_duplex = RTE_ETH_LINK_HALF_DUPLEX;
>  	link.link_autoneg = !(dev->data->dev_conf.link_speeds &
> -			RTE_ETH_LINK_AUTONEG);
> +			RTE_ETH_LINK_SPEED_FIXED);
> 
>  	hw->mac.get_link_status = true;
> 
> --
> 2.31.1

Acked-by: Jiawen Wu <jiawenwu@trustnetic.com>
Ferruh Yigit Nov. 2, 2021, 1:35 p.m. UTC | #2 
On 11/2/2021 1:51 AM, Jiawen Wu wrote:
> On November 1, 2021 9:20 PM, Ferruh Yigit wrote:
>> Macro is changed unintentionally while adding RTE_ prefix, fixing the
> original
>> value.
>>
>> Fixes: 295968d17407 ("ethdev: add namespace")
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 
> Acked-by: Jiawen Wu <jiawenwu@trustnetic.com>
> 

Applied to dpdk-next-net/main, thanks.
diff mbox series

Patch

diff --git a/drivers/net/txgbe/txgbe_ethdev.c b/drivers/net/txgbe/txgbe_ethdev.c
index 169272ded5b0..fde9914e493f 100644
--- a/drivers/net/txgbe/txgbe_ethdev.c
+++ b/drivers/net/txgbe/txgbe_ethdev.c
@@ -2693,7 +2693,7 @@  txgbe_dev_link_update_share(struct rte_eth_dev *dev,
 	link.link_speed = RTE_ETH_SPEED_NUM_NONE;
 	link.link_duplex = RTE_ETH_LINK_HALF_DUPLEX;
 	link.link_autoneg = !(dev->data->dev_conf.link_speeds &
-			RTE_ETH_LINK_AUTONEG);
+			RTE_ETH_LINK_SPEED_FIXED);
 
 	hw->mac.get_link_status = true;