Commit 72b8eaab authored by Tariq Toukan's avatar Tariq Toukan Committed by David S. Miller

net/mlx4: Replace ENOSYS with better fitting error codes

Conform the following warning:
WARNING: ENOSYS means 'invalid syscall nr' and nothing else.
Signed-off-by: default avatarTariq Toukan <tariqt@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 30357d7d
...@@ -1980,7 +1980,7 @@ static int mlx4_en_get_module_info(struct net_device *dev, ...@@ -1980,7 +1980,7 @@ static int mlx4_en_get_module_info(struct net_device *dev,
modinfo->eeprom_len = ETH_MODULE_SFF_8472_LEN; modinfo->eeprom_len = ETH_MODULE_SFF_8472_LEN;
break; break;
default: default:
return -ENOSYS; return -EINVAL;
} }
return 0; return 0;
......
...@@ -672,7 +672,7 @@ int mlx4_QUERY_FUNC_CAP(struct mlx4_dev *dev, u8 gen_or_port, ...@@ -672,7 +672,7 @@ int mlx4_QUERY_FUNC_CAP(struct mlx4_dev *dev, u8 gen_or_port,
MLX4_GET(field, outbox, QUERY_FUNC_CAP_PHYS_PORT_OFFSET); MLX4_GET(field, outbox, QUERY_FUNC_CAP_PHYS_PORT_OFFSET);
func_cap->physical_port = field; func_cap->physical_port = field;
if (func_cap->physical_port != gen_or_port) { if (func_cap->physical_port != gen_or_port) {
err = -ENOSYS; err = -EINVAL;
goto out; goto out;
} }
......
...@@ -838,7 +838,7 @@ static int mlx4_slave_cap(struct mlx4_dev *dev) ...@@ -838,7 +838,7 @@ static int mlx4_slave_cap(struct mlx4_dev *dev)
*/ */
if (hca_param.global_caps) { if (hca_param.global_caps) {
mlx4_err(dev, "Unknown hca global capabilities\n"); mlx4_err(dev, "Unknown hca global capabilities\n");
return -ENOSYS; return -EINVAL;
} }
mlx4_log_num_mgm_entry_size = hca_param.log_mc_entry_sz; mlx4_log_num_mgm_entry_size = hca_param.log_mc_entry_sz;
...@@ -896,7 +896,7 @@ static int mlx4_slave_cap(struct mlx4_dev *dev) ...@@ -896,7 +896,7 @@ static int mlx4_slave_cap(struct mlx4_dev *dev)
PF_CONTEXT_BEHAVIOUR_MASK) { PF_CONTEXT_BEHAVIOUR_MASK) {
mlx4_err(dev, "Unknown pf context behaviour %x known flags %x\n", mlx4_err(dev, "Unknown pf context behaviour %x known flags %x\n",
func_cap.pf_context_behaviour, PF_CONTEXT_BEHAVIOUR_MASK); func_cap.pf_context_behaviour, PF_CONTEXT_BEHAVIOUR_MASK);
return -ENOSYS; return -EINVAL;
} }
dev->caps.num_ports = func_cap.num_ports; dev->caps.num_ports = func_cap.num_ports;
...@@ -3492,7 +3492,7 @@ static int mlx4_load_one(struct pci_dev *pdev, int pci_dev_data, ...@@ -3492,7 +3492,7 @@ static int mlx4_load_one(struct pci_dev *pdev, int pci_dev_data,
mlx4_enable_msi_x(dev); mlx4_enable_msi_x(dev);
if ((mlx4_is_mfunc(dev)) && if ((mlx4_is_mfunc(dev)) &&
!(dev->flags & MLX4_FLAG_MSI_X)) { !(dev->flags & MLX4_FLAG_MSI_X)) {
err = -ENOSYS; err = -EOPNOTSUPP;
mlx4_err(dev, "INTx is not supported in multi-function mode, aborting\n"); mlx4_err(dev, "INTx is not supported in multi-function mode, aborting\n");
goto err_free_eq; goto err_free_eq;
} }
......
...@@ -1396,7 +1396,7 @@ static int remove_ok(struct res_common *res, enum mlx4_resource type, int extra) ...@@ -1396,7 +1396,7 @@ static int remove_ok(struct res_common *res, enum mlx4_resource type, int extra)
case RES_MTT: case RES_MTT:
return remove_mtt_ok((struct res_mtt *)res, extra); return remove_mtt_ok((struct res_mtt *)res, extra);
case RES_MAC: case RES_MAC:
return -ENOSYS; return -EOPNOTSUPP;
case RES_EQ: case RES_EQ:
return remove_eq_ok((struct res_eq *)res); return remove_eq_ok((struct res_eq *)res);
case RES_COUNTER: case RES_COUNTER:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment