RDMA/hns: Remove unused member and variable of CMDQ
authorLang Cheng <chenglang@huawei.com>
Sun, 7 Feb 2021 08:55:39 +0000 (16:55 +0800)
committerJason Gunthorpe <jgg@nvidia.com>
Tue, 16 Feb 2021 18:42:58 +0000 (14:42 -0400)
last_status of structure hns_roce_v2_cmq has never been used, and the
variable named 'complete' in __hns_roce_cmq_send() is meaningless.

Fixes: a04ff739f2a9 ("RDMA/hns: Add command queue support for hip08 RoCE driver")
Link: https://lore.kernel.org/r/1612688143-28226-2-git-send-email-liweihang@huawei.com
Signed-off-by: Lang Cheng <chenglang@huawei.com>
Signed-off-by: Weihang Li <liweihang@huawei.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/hns/hns_roce_hw_v2.c
drivers/infiniband/hw/hns/hns_roce_hw_v2.h

index d9f94b4..4bbbc24 100644 (file)
@@ -1314,7 +1314,6 @@ static int __hns_roce_cmq_send(struct hns_roce_dev *hr_dev,
        struct hns_roce_v2_priv *priv = hr_dev->priv;
        struct hns_roce_v2_cmq_ring *csq = &priv->cmq.csq;
        struct hns_roce_cmq_desc *desc_to_use;
-       bool complete = false;
        u32 timeout = 0;
        int handle = 0;
        u16 desc_ret;
@@ -1361,7 +1360,6 @@ static int __hns_roce_cmq_send(struct hns_roce_dev *hr_dev,
        }
 
        if (hns_roce_cmq_csq_done(hr_dev)) {
-               complete = true;
                handle = 0;
                while (handle < num) {
                        /* get the result of hardware write back */
@@ -1373,16 +1371,15 @@ static int __hns_roce_cmq_send(struct hns_roce_dev *hr_dev,
                                ret = 0;
                        else
                                ret = -EIO;
-                       priv->cmq.last_status = desc_ret;
+
                        ntc++;
                        handle++;
                        if (ntc == csq->desc_num)
                                ntc = 0;
                }
-       }
-
-       if (!complete)
+       } else {
                ret = -EAGAIN;
+       }
 
        /* clean the command send queue */
        handle = hns_roce_cmq_csq_clean(hr_dev);
index f29438c..631f679 100644 (file)
@@ -1890,7 +1890,6 @@ struct hns_roce_v2_cmq {
        struct hns_roce_v2_cmq_ring csq;
        struct hns_roce_v2_cmq_ring crq;
        u16 tx_timeout;
-       u16 last_status;
 };
 
 enum hns_roce_link_table_type {