提交 d57eff1f 编写于 作者: L liweihang 提交者: Xie XiuQi

net: hns3: simplify hclge_cmd_query_error()

driver inclusion
category: bugfix
bugzilla: NA
CVE: NA

The 4th and 5th parameter of hclge_cmd_query_error is useless, so this
patch remove them.

Feature or Bugfix: Bugfix
Signed-off-by: Nliweihang <liweihang@huawei.com>
Reviewed-by: Nlipeng <lipeng321@huawei.com>
Reviewed-by: NYang Yingliang <yangyingliang@huawei.com>
Signed-off-by: NYang Yingliang <yangyingliang@huawei.com>
上级 d671b258
...@@ -680,16 +680,11 @@ struct hclge_desc *hclge_query_bd_num(struct hclge_dev *hdev, ...@@ -680,16 +680,11 @@ struct hclge_desc *hclge_query_bd_num(struct hclge_dev *hdev,
* @desc: descriptor for describing the command * @desc: descriptor for describing the command
* @cmd: command opcode * @cmd: command opcode
* @flag: flag for extended command structure * @flag: flag for extended command structure
* @w_num: offset for setting the read interrupt type.
* @int_type: select which type of the interrupt for which the error
* info will be read(RAS-CE/RAS-NFE/RAS-FE etc).
* *
* This function query the error info from hw register/s using command * This function query the error info from hw register/s using command
*/ */
static int hclge_cmd_query_error(struct hclge_dev *hdev, static int hclge_cmd_query_error(struct hclge_dev *hdev,
struct hclge_desc *desc, u32 cmd, struct hclge_desc *desc, u32 cmd, u16 flag)
u16 flag, u8 w_num,
enum hclge_err_int_type int_type)
{ {
struct device *dev = &hdev->pdev->dev; struct device *dev = &hdev->pdev->dev;
int desc_num = 1; int desc_num = 1;
...@@ -701,8 +696,6 @@ static int hclge_cmd_query_error(struct hclge_dev *hdev, ...@@ -701,8 +696,6 @@ static int hclge_cmd_query_error(struct hclge_dev *hdev,
hclge_cmd_setup_basic_desc(&desc[1], cmd, true); hclge_cmd_setup_basic_desc(&desc[1], cmd, true);
desc_num = 2; desc_num = 2;
} }
if (w_num)
desc[0].data[w_num] = cpu_to_le32(int_type);
ret = hclge_cmd_send(&hdev->hw, &desc[0], desc_num); ret = hclge_cmd_send(&hdev->hw, &desc[0], desc_num);
if (ret) if (ret)
...@@ -900,8 +893,7 @@ static int hclge_config_tm_hw_err_int(struct hclge_dev *hdev, bool en) ...@@ -900,8 +893,7 @@ static int hclge_config_tm_hw_err_int(struct hclge_dev *hdev, bool en)
} }
/* configure TM QCN hw errors */ /* configure TM QCN hw errors */
ret = hclge_cmd_query_error(hdev, &desc, HCLGE_TM_QCN_MEM_INT_CFG, ret = hclge_cmd_query_error(hdev, &desc, HCLGE_TM_QCN_MEM_INT_CFG, 0);
0, 0, 0);
if (ret) { if (ret) {
dev_err(dev, "fail(%d) to read TM QCN CFG status\n", ret); dev_err(dev, "fail(%d) to read TM QCN CFG status\n", ret);
return ret; return ret;
...@@ -1388,7 +1380,7 @@ static int hclge_log_rocee_ecc_error(struct hclge_dev *hdev) ...@@ -1388,7 +1380,7 @@ static int hclge_log_rocee_ecc_error(struct hclge_dev *hdev)
ret = hclge_cmd_query_error(hdev, &desc[0], ret = hclge_cmd_query_error(hdev, &desc[0],
HCLGE_QUERY_ROCEE_ECC_RAS_INFO_CMD, HCLGE_QUERY_ROCEE_ECC_RAS_INFO_CMD,
HCLGE_CMD_FLAG_NEXT, 0, 0); HCLGE_CMD_FLAG_NEXT);
if (ret) { if (ret) {
dev_err(dev, "failed(%d) to query ROCEE ECC error sts\n", ret); dev_err(dev, "failed(%d) to query ROCEE ECC error sts\n", ret);
return ret; return ret;
...@@ -1412,7 +1404,7 @@ static int hclge_log_rocee_ovf_error(struct hclge_dev *hdev) ...@@ -1412,7 +1404,7 @@ static int hclge_log_rocee_ovf_error(struct hclge_dev *hdev)
/* read overflow error status */ /* read overflow error status */
ret = hclge_cmd_query_error(hdev, &desc[0], HCLGE_ROCEE_PF_RAS_INT_CMD, ret = hclge_cmd_query_error(hdev, &desc[0], HCLGE_ROCEE_PF_RAS_INT_CMD,
0, 0, 0); 0);
if (ret) { if (ret) {
dev_err(dev, "failed(%d) to query ROCEE OVF error sts\n", ret); dev_err(dev, "failed(%d) to query ROCEE OVF error sts\n", ret);
return ret; return ret;
...@@ -1461,8 +1453,7 @@ hclge_log_and_clear_rocee_ras_error(struct hclge_dev *hdev) ...@@ -1461,8 +1453,7 @@ hclge_log_and_clear_rocee_ras_error(struct hclge_dev *hdev)
/* read RAS error interrupt status */ /* read RAS error interrupt status */
ret = hclge_cmd_query_error(hdev, &desc[0], ret = hclge_cmd_query_error(hdev, &desc[0],
HCLGE_QUERY_CLEAR_ROCEE_RAS_INT, HCLGE_QUERY_CLEAR_ROCEE_RAS_INT, 0);
0, 0, 0);
if (ret) { if (ret) {
dev_err(dev, "failed(%d) to query ROCEE RAS INT SRC\n", ret); dev_err(dev, "failed(%d) to query ROCEE RAS INT SRC\n", ret);
/* reset everything for now */ /* reset everything for now */
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册