From: Parav Pandit <pa...@nvidia.com>

SF's hardware function id is already stored in mlx5_sf. Reuse it,
instead of querying the hw table.

Signed-off-by: Parav Pandit <pa...@nvidia.com>
Signed-off-by: Saeed Mahameed <sae...@nvidia.com>
---
 drivers/net/ethernet/mellanox/mlx5/core/sf/devlink.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/sf/devlink.c 
b/drivers/net/ethernet/mellanox/mlx5/core/sf/devlink.c
index 60a6328a9ca0..52226d9b9a6d 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/sf/devlink.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/sf/devlink.c
@@ -270,15 +270,14 @@ static int mlx5_sf_add(struct mlx5_core_dev *dev, struct 
mlx5_sf_table *table,
 {
        struct mlx5_eswitch *esw = dev->priv.eswitch;
        struct mlx5_sf *sf;
-       u16 hw_fn_id;
        int err;
 
        sf = mlx5_sf_alloc(table, new_attr->sfnum, extack);
        if (IS_ERR(sf))
                return PTR_ERR(sf);
 
-       hw_fn_id = mlx5_sf_sw_to_hw_id(dev, sf->id);
-       err = mlx5_esw_offloads_sf_vport_enable(esw, &sf->dl_port, hw_fn_id, 
new_attr->sfnum);
+       err = mlx5_esw_offloads_sf_vport_enable(esw, &sf->dl_port, sf->hw_fn_id,
+                                               new_attr->sfnum);
        if (err)
                goto esw_err;
        *new_port_index = sf->port_index;
-- 
2.30.2

Reply via email to