Re: [PATCH net-next 01/12] net: hns3: log detail error info of ROCEE ECC and AXI errors

From: David Miller
Date: Thu Jun 06 2019 - 13:40:32 EST


From: Huazhong Tan <tanhuazhong@xxxxxxxxxx>
Date: Thu, 6 Jun 2019 16:20:56 +0800

> +static int hclge_log_rocee_axi_error(struct hclge_dev *hdev)
> +{
...
> + ret = hclge_cmd_send(&hdev->hw, &desc[0], 3);
> + if (ret) {
> + dev_err(dev, "failed(%d) to query ROCEE AXI error sts\n", ret);
> + return ret;
> + }
...
> + ret = hclge_log_rocee_axi_error(hdev);
> + if (ret) {
> + dev_err(dev, "failed(%d) to process axi error\n", ret);

You log the error twice which is unnecessary.