net/mlx5_core: Use port number in the query port mtu helpers
authorMajd Dibbiny <majd@mellanox.com>
Thu, 4 Jun 2015 16:30:43 +0000 (19:30 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 4 Jun 2015 23:41:01 +0000 (16:41 -0700)
Extend the function prototypes for max and operational mtu to take the
local port number. In the Ethernet driver is this hard coded to one,
since ConnectX4 Ethernet devices are always function-per-port.
The IB driver also serves older devices (ConnectIB) which isn't such,
and hence the part can vary.

Signed-off-by: Majd Dibbiny <majd@mellanox.com>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlx5/core/en_main.c
drivers/net/ethernet/mellanox/mlx5/core/port.c
include/linux/mlx5/driver.h

index edba09cca600ce9a4b412ad9eebcd082da9876de..7348c5173aa976f96aed8f0d29e148bb9700054f 100644 (file)
@@ -1386,7 +1386,7 @@ int mlx5e_open_locked(struct net_device *netdev)
                return err;
        }
 
-       err = mlx5_query_port_oper_mtu(mdev, &actual_mtu);
+       err = mlx5_query_port_oper_mtu(mdev, &actual_mtu, 1);
        if (err) {
                netdev_err(netdev, "%s: mlx5_query_port_oper_mtu failed %d\n",
                           __func__, err);
@@ -1614,7 +1614,7 @@ static int mlx5e_change_mtu(struct net_device *netdev, int new_mtu)
        int max_mtu;
        int err = 0;
 
-       err = mlx5_query_port_max_mtu(mdev, &max_mtu);
+       err = mlx5_query_port_max_mtu(mdev, &max_mtu, 1);
        if (err)
                return err;
 
index 7d3d0f9f328dd23855ae170f36c778f0c19abf33..d9498aae5ab5b2e12d82750b33bc99c479a17e4c 100644 (file)
@@ -213,7 +213,8 @@ int mlx5_query_port_status(struct mlx5_core_dev *dev, u8 *status)
 }
 
 static int mlx5_query_port_mtu(struct mlx5_core_dev *dev,
-                              int *admin_mtu, int *max_mtu, int *oper_mtu)
+                              int *admin_mtu, int *max_mtu, int *oper_mtu,
+                              u8 local_port)
 {
        u32 in[MLX5_ST_SZ_DW(pmtu_reg)];
        u32 out[MLX5_ST_SZ_DW(pmtu_reg)];
@@ -221,7 +222,7 @@ static int mlx5_query_port_mtu(struct mlx5_core_dev *dev,
 
        memset(in, 0, sizeof(in));
 
-       MLX5_SET(pmtu_reg, in, local_port, 1);
+       MLX5_SET(pmtu_reg, in, local_port, local_port);
 
        err = mlx5_core_access_reg(dev, in, sizeof(in), out,
                                   sizeof(out), MLX5_REG_PMTU, 0, 0);
@@ -253,14 +254,16 @@ int mlx5_set_port_mtu(struct mlx5_core_dev *dev, int mtu)
 }
 EXPORT_SYMBOL_GPL(mlx5_set_port_mtu);
 
-int mlx5_query_port_max_mtu(struct mlx5_core_dev *dev, int *max_mtu)
+int mlx5_query_port_max_mtu(struct mlx5_core_dev *dev, int *max_mtu,
+                           u8 local_port)
 {
-       return mlx5_query_port_mtu(dev, NULL, max_mtu, NULL);
+       return mlx5_query_port_mtu(dev, NULL, max_mtu, NULL, local_port);
 }
 EXPORT_SYMBOL_GPL(mlx5_query_port_max_mtu);
 
-int mlx5_query_port_oper_mtu(struct mlx5_core_dev *dev, int *oper_mtu)
+int mlx5_query_port_oper_mtu(struct mlx5_core_dev *dev, int *oper_mtu,
+                            u8 local_port)
 {
-       return mlx5_query_port_mtu(dev, NULL, NULL, oper_mtu);
+       return mlx5_query_port_mtu(dev, NULL, NULL, oper_mtu, local_port);
 }
 EXPORT_SYMBOL_GPL(mlx5_query_port_oper_mtu);
index b90fb9336d212025024f754a495fdff2d32ff162..cd09784b699989d03d92eec00e991a7c2863acff 100644 (file)
@@ -751,8 +751,10 @@ int mlx5_set_port_status(struct mlx5_core_dev *dev,
 int mlx5_query_port_status(struct mlx5_core_dev *dev, u8 *status);
 
 int mlx5_set_port_mtu(struct mlx5_core_dev *dev, int mtu);
-int mlx5_query_port_max_mtu(struct mlx5_core_dev *dev, int *max_mtu);
-int mlx5_query_port_oper_mtu(struct mlx5_core_dev *dev, int *oper_mtu);
+int mlx5_query_port_max_mtu(struct mlx5_core_dev *dev, int *max_mtu,
+                           u8 local_port);
+int mlx5_query_port_oper_mtu(struct mlx5_core_dev *dev, int *oper_mtu,
+                            u8 local_port);
 
 int mlx5_debug_eq_add(struct mlx5_core_dev *dev, struct mlx5_eq *eq);
 void mlx5_debug_eq_remove(struct mlx5_core_dev *dev, struct mlx5_eq *eq);