Commit d41884ee authored by Huazhong Tan's avatar Huazhong Tan Committed by David S. Miller

net: hns3: narrow two local variable range in hclgevf_reset_prepare_wait()

Since variable send_msg and ret only used in if branch, so move
their definition into the if branch.
Signed-off-by: default avatarHuazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c1f1f16c
...@@ -1788,10 +1788,10 @@ static int hclgevf_reset_prepare_wait(struct hclgevf_dev *hdev) ...@@ -1788,10 +1788,10 @@ static int hclgevf_reset_prepare_wait(struct hclgevf_dev *hdev)
{ {
#define HCLGEVF_RESET_SYNC_TIME 100 #define HCLGEVF_RESET_SYNC_TIME 100
if (hdev->reset_type == HNAE3_VF_FUNC_RESET) {
struct hclge_vf_to_pf_msg send_msg; struct hclge_vf_to_pf_msg send_msg;
int ret = 0; int ret;
if (hdev->reset_type == HNAE3_VF_FUNC_RESET) {
hclgevf_build_send_msg(&send_msg, HCLGE_MBX_RESET, 0); hclgevf_build_send_msg(&send_msg, HCLGE_MBX_RESET, 0);
ret = hclgevf_send_mbx_msg(hdev, &send_msg, true, NULL, 0); ret = hclgevf_send_mbx_msg(hdev, &send_msg, true, NULL, 0);
if (ret) { if (ret) {
...@@ -1806,10 +1806,10 @@ static int hclgevf_reset_prepare_wait(struct hclgevf_dev *hdev) ...@@ -1806,10 +1806,10 @@ static int hclgevf_reset_prepare_wait(struct hclgevf_dev *hdev)
/* inform hardware that preparatory work is done */ /* inform hardware that preparatory work is done */
msleep(HCLGEVF_RESET_SYNC_TIME); msleep(HCLGEVF_RESET_SYNC_TIME);
hclgevf_reset_handshake(hdev, true); hclgevf_reset_handshake(hdev, true);
dev_info(&hdev->pdev->dev, "prepare reset(%d) wait done, ret:%d\n", dev_info(&hdev->pdev->dev, "prepare reset(%d) wait done\n",
hdev->reset_type, ret); hdev->reset_type);
return ret; return 0;
} }
static void hclgevf_dump_rst_info(struct hclgevf_dev *hdev) static void hclgevf_dump_rst_info(struct hclgevf_dev *hdev)
......
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