Commit 99a6993a authored by Jian Shen's avatar Jian Shen Committed by David S. Miller

net: hns3: cleanup of return values in hclge_init_client_instance()

Removes the goto and directly returns in case of errors as part of the
cleanup.
Signed-off-by: default avatarJian Shen <shenjian15@huawei.com>
Signed-off-by: default avatarPeng Li <lipeng321@huawei.com>
Signed-off-by: default avatarSalil Mehta <salil.mehta@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e63cd65f
...@@ -5297,7 +5297,7 @@ static int hclge_init_client_instance(struct hnae3_client *client, ...@@ -5297,7 +5297,7 @@ static int hclge_init_client_instance(struct hnae3_client *client,
vport->nic.client = client; vport->nic.client = client;
ret = client->ops->init_instance(&vport->nic); ret = client->ops->init_instance(&vport->nic);
if (ret) if (ret)
goto err; return ret;
if (hdev->roce_client && if (hdev->roce_client &&
hnae3_dev_roce_supported(hdev)) { hnae3_dev_roce_supported(hdev)) {
...@@ -5305,11 +5305,11 @@ static int hclge_init_client_instance(struct hnae3_client *client, ...@@ -5305,11 +5305,11 @@ static int hclge_init_client_instance(struct hnae3_client *client,
ret = hclge_init_roce_base_info(vport); ret = hclge_init_roce_base_info(vport);
if (ret) if (ret)
goto err; return ret;
ret = rc->ops->init_instance(&vport->roce); ret = rc->ops->init_instance(&vport->roce);
if (ret) if (ret)
goto err; return ret;
} }
break; break;
...@@ -5319,7 +5319,7 @@ static int hclge_init_client_instance(struct hnae3_client *client, ...@@ -5319,7 +5319,7 @@ static int hclge_init_client_instance(struct hnae3_client *client,
ret = client->ops->init_instance(&vport->nic); ret = client->ops->init_instance(&vport->nic);
if (ret) if (ret)
goto err; return ret;
break; break;
case HNAE3_CLIENT_ROCE: case HNAE3_CLIENT_ROCE:
...@@ -5331,18 +5331,16 @@ static int hclge_init_client_instance(struct hnae3_client *client, ...@@ -5331,18 +5331,16 @@ static int hclge_init_client_instance(struct hnae3_client *client,
if (hdev->roce_client && hdev->nic_client) { if (hdev->roce_client && hdev->nic_client) {
ret = hclge_init_roce_base_info(vport); ret = hclge_init_roce_base_info(vport);
if (ret) if (ret)
goto err; return ret;
ret = client->ops->init_instance(&vport->roce); ret = client->ops->init_instance(&vport->roce);
if (ret) if (ret)
goto err; return ret;
} }
} }
} }
return 0; return 0;
err:
return ret;
} }
static void hclge_uninit_client_instance(struct hnae3_client *client, static void hclge_uninit_client_instance(struct hnae3_client *client,
......
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