diff options
| author | Govindarajulu Varadarajan <_govind@gmx.com> | 2014-06-23 14:37:59 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-06-24 01:32:19 +0400 |
| commit | 10cc88446cec4eee8e2efab24ad387d52ef1f4fb (patch) | |
| tree | eff9e24156cad2aa7286ebc4a753e5c50b710045 /drivers/net/ethernet/cisco | |
| parent | e0f31d8498676fda36289603a054d0d490aa2679 (diff) | |
| download | linux-10cc88446cec4eee8e2efab24ad387d52ef1f4fb.tar.xz | |
enic: fix return value in _vnic_dev_cmd
Hardware (in readq(&devcmd->args[0])) returns positive number in case of error.
But _vnic_dev_cmd should return a negative value in case of error.
Signed-off-by: Govindarajulu Varadarajan <_govind@gmx.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/cisco')
| -rw-r--r-- | drivers/net/ethernet/cisco/enic/vnic_dev.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/cisco/enic/vnic_dev.c b/drivers/net/ethernet/cisco/enic/vnic_dev.c index e86a45cb9e68..263081b8e636 100644 --- a/drivers/net/ethernet/cisco/enic/vnic_dev.c +++ b/drivers/net/ethernet/cisco/enic/vnic_dev.c @@ -312,12 +312,12 @@ static int _vnic_dev_cmd(struct vnic_dev *vdev, enum vnic_devcmd_cmd cmd, err = (int)readq(&devcmd->args[0]); if (err == ERR_EINVAL && cmd == CMD_CAPABILITY) - return err; + return -err; if (err != ERR_ECMDUNKNOWN || cmd != CMD_CAPABILITY) pr_err("Error %d devcmd %d\n", err, _CMD_N(cmd)); - return err; + return -err; } if (_CMD_DIR(cmd) & _CMD_DIR_READ) { |