net: hns3: fix pause configure fail problem

When configure pause, current implementation returns directly
after setup PFC without setup BP, which is not sufficient.

So this patch fixes it, only return while setting PFC failed.

Fixes: 44e59e375bf7 ("net: hns3: do not return GE PFC setting err when initializing")
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: Peng Li <lipeng321@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_tm.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_tm.c
index aafc69f4bfdd6f5d2afce29a7b3affb80b2f036c..a7bbb6d3091a62b185ea28c8fa39a6da8f0a7072 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_tm.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_tm.c
@@ -1331,8 +1331,11 @@ int hclge_pause_setup_hw(struct hclge_dev *hdev, bool init)
        ret = hclge_pfc_setup_hw(hdev);
        if (init && ret == -EOPNOTSUPP)
                dev_warn(&hdev->pdev->dev, "GE MAC does not support pfc\n");
-       else
+       else if (ret) {
+               dev_err(&hdev->pdev->dev, "config pfc failed! ret = %d\n",
+                       ret);
                return ret;
+       }
 
        return hclge_tm_bp_setup(hdev);
 }