summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Yongjun <weiyongjun1@huawei.com>2016-10-22 14:31:06 +0000
committerDavid S. Miller <davem@davemloft.net>2016-10-26 17:20:36 -0400
commit92d230dd8cafac417e130e404d4b64eafe2271de (patch)
treeda6fcd23e2997a258665b22247852be29fb0ef01
parent44060abe1dd609d5efa641cce3784777ff5d507f (diff)
downloadlinux-stable-92d230dd8cafac417e130e404d4b64eafe2271de.tar.gz
linux-stable-92d230dd8cafac417e130e404d4b64eafe2271de.tar.bz2
linux-stable-92d230dd8cafac417e130e404d4b64eafe2271de.zip
rocker: fix error return code in rocker_world_check_init()
Fix to return error code -EINVAL from the error handling case instead of 0, as done elsewhere in this function. Fixes: e420114eef4a ("rocker: introduce worlds infrastructure") Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> Acked-by: Jiri Pirko <jiri@mellanox.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/ethernet/rocker/rocker_main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/rocker/rocker_main.c b/drivers/net/ethernet/rocker/rocker_main.c
index 5424fb341613..24b746406bc7 100644
--- a/drivers/net/ethernet/rocker/rocker_main.c
+++ b/drivers/net/ethernet/rocker/rocker_main.c
@@ -1471,7 +1471,7 @@ static int rocker_world_check_init(struct rocker_port *rocker_port)
if (rocker->wops) {
if (rocker->wops->mode != mode) {
dev_err(&rocker->pdev->dev, "hardware has ports in different worlds, which is not supported\n");
- return err;
+ return -EINVAL;
}
return 0;
}