[dpdk-dev,2/2] net/mlx5: fix missing errno in probe code (minor)

Message ID 20180524121658.22663-2-adrien.mazarguil@6wind.com (mailing list archive)
State Accepted, archived
Delegated to: Shahaf Shuler
Headers

Checks

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

Commit Message

Adrien Mazarguil May 24, 2018, 12:17 p.m. UTC
  Fixes: b43802b4bdfe ("net/mlx5: support 16 hardware priorities")
Cc: Xueming Li <xuemingl@mellanox.com>

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
---
 drivers/net/mlx5/mlx5.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Yongseok Koh May 24, 2018, 6:30 p.m. UTC | #1
> On May 24, 2018, at 5:17 AM, Adrien Mazarguil <adrien.mazarguil@6wind.com> wrote:
> 
> Fixes: b43802b4bdfe ("net/mlx5: support 16 hardware priorities")
> Cc: Xueming Li <xuemingl@mellanox.com>
> 
> Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
> ---
Acked-by: Yongseok Koh <yskoh@mellanox.com>
 
Thanks
  

Patch

diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c
index bbe78de28..36d72f757 100644
--- a/drivers/net/mlx5/mlx5.c
+++ b/drivers/net/mlx5/mlx5.c
@@ -1141,6 +1141,7 @@  mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
 		if (verb_priorities < MLX5_VERBS_FLOW_PRIO_8) {
 			DRV_LOG(ERR, "port %u wrong Verbs flow priorities: %u",
 				eth_dev->data->port_id, verb_priorities);
+			err = ENOTSUP;
 			goto port_error;
 		}
 		priv->config.max_verbs_prio = verb_priorities;