OSDN Git Service

net: hns3: merge some repetitive macros
authorPeng Li <lipeng321@huawei.com>
Fri, 27 Aug 2021 09:28:23 +0000 (17:28 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 27 Aug 2021 10:20:21 +0000 (11:20 +0100)
There are some repetitive macros have same meaning and value, this patch
merges them to make code clean.

Signed-off-by: Peng Li <lipeng321@huawei.com>
Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h

index 2e49a52..afca9ee 100644 (file)
@@ -1017,16 +1017,6 @@ struct hclge_common_lb_cmd {
 
 #define HCLGE_TYPE_CRQ                 0
 #define HCLGE_TYPE_CSQ                 1
-#define HCLGE_NIC_CSQ_BASEADDR_L_REG   0x27000
-#define HCLGE_NIC_CSQ_BASEADDR_H_REG   0x27004
-#define HCLGE_NIC_CSQ_DEPTH_REG                0x27008
-#define HCLGE_NIC_CSQ_TAIL_REG         0x27010
-#define HCLGE_NIC_CSQ_HEAD_REG         0x27014
-#define HCLGE_NIC_CRQ_BASEADDR_L_REG   0x27018
-#define HCLGE_NIC_CRQ_BASEADDR_H_REG   0x2701c
-#define HCLGE_NIC_CRQ_DEPTH_REG                0x27020
-#define HCLGE_NIC_CRQ_TAIL_REG         0x27024
-#define HCLGE_NIC_CRQ_HEAD_REG         0x27028
 
 /* this bit indicates that the driver is ready for hardware reset */
 #define HCLGE_NIC_SW_RST_RDY_B         16
index cb756cf..750390c 100644 (file)
@@ -92,23 +92,23 @@ static const struct pci_device_id ae_algo_pci_tbl[] = {
 
 MODULE_DEVICE_TABLE(pci, ae_algo_pci_tbl);
 
-static const u32 cmdq_reg_addr_list[] = {HCLGE_CMDQ_TX_ADDR_L_REG,
-                                        HCLGE_CMDQ_TX_ADDR_H_REG,
-                                        HCLGE_CMDQ_TX_DEPTH_REG,
-                                        HCLGE_CMDQ_TX_TAIL_REG,
-                                        HCLGE_CMDQ_TX_HEAD_REG,
-                                        HCLGE_CMDQ_RX_ADDR_L_REG,
-                                        HCLGE_CMDQ_RX_ADDR_H_REG,
-                                        HCLGE_CMDQ_RX_DEPTH_REG,
-                                        HCLGE_CMDQ_RX_TAIL_REG,
-                                        HCLGE_CMDQ_RX_HEAD_REG,
+static const u32 cmdq_reg_addr_list[] = {HCLGE_NIC_CSQ_BASEADDR_L_REG,
+                                        HCLGE_NIC_CSQ_BASEADDR_H_REG,
+                                        HCLGE_NIC_CSQ_DEPTH_REG,
+                                        HCLGE_NIC_CSQ_TAIL_REG,
+                                        HCLGE_NIC_CSQ_HEAD_REG,
+                                        HCLGE_NIC_CRQ_BASEADDR_L_REG,
+                                        HCLGE_NIC_CRQ_BASEADDR_H_REG,
+                                        HCLGE_NIC_CRQ_DEPTH_REG,
+                                        HCLGE_NIC_CRQ_TAIL_REG,
+                                        HCLGE_NIC_CRQ_HEAD_REG,
                                         HCLGE_VECTOR0_CMDQ_SRC_REG,
                                         HCLGE_CMDQ_INTR_STS_REG,
                                         HCLGE_CMDQ_INTR_EN_REG,
                                         HCLGE_CMDQ_INTR_GEN_REG};
 
 static const u32 common_reg_addr_list[] = {HCLGE_MISC_VECTOR_REG_BASE,
-                                          HCLGE_VECTOR0_OTER_EN_REG,
+                                          HCLGE_PF_OTHER_INT_REG,
                                           HCLGE_MISC_RESET_STS_REG,
                                           HCLGE_MISC_VECTOR_INT_STS,
                                           HCLGE_GLOBAL_RESET_REG,
index b6c1153..9ca7bb2 100644 (file)
 #define HCLGE_VECTOR_REG_OFFSET_H      0x1000
 #define HCLGE_VECTOR_VF_OFFSET         0x100000
 
-#define HCLGE_CMDQ_TX_ADDR_L_REG       0x27000
-#define HCLGE_CMDQ_TX_ADDR_H_REG       0x27004
-#define HCLGE_CMDQ_TX_DEPTH_REG                0x27008
-#define HCLGE_CMDQ_TX_TAIL_REG         0x27010
-#define HCLGE_CMDQ_TX_HEAD_REG         0x27014
-#define HCLGE_CMDQ_RX_ADDR_L_REG       0x27018
-#define HCLGE_CMDQ_RX_ADDR_H_REG       0x2701C
-#define HCLGE_CMDQ_RX_DEPTH_REG                0x27020
-#define HCLGE_CMDQ_RX_TAIL_REG         0x27024
-#define HCLGE_CMDQ_RX_HEAD_REG         0x27028
+#define HCLGE_NIC_CSQ_BASEADDR_L_REG   0x27000
+#define HCLGE_NIC_CSQ_BASEADDR_H_REG   0x27004
+#define HCLGE_NIC_CSQ_DEPTH_REG                0x27008
+#define HCLGE_NIC_CSQ_TAIL_REG         0x27010
+#define HCLGE_NIC_CSQ_HEAD_REG         0x27014
+#define HCLGE_NIC_CRQ_BASEADDR_L_REG   0x27018
+#define HCLGE_NIC_CRQ_BASEADDR_H_REG   0x2701C
+#define HCLGE_NIC_CRQ_DEPTH_REG                0x27020
+#define HCLGE_NIC_CRQ_TAIL_REG         0x27024
+#define HCLGE_NIC_CRQ_HEAD_REG         0x27028
+
 #define HCLGE_CMDQ_INTR_STS_REG                0x27104
 #define HCLGE_CMDQ_INTR_EN_REG         0x27108
 #define HCLGE_CMDQ_INTR_GEN_REG                0x2710C
 
 /* bar registers for common func */
-#define HCLGE_VECTOR0_OTER_EN_REG      0x20600
 #define HCLGE_GRO_EN_REG               0x28000
 #define HCLGE_RXD_ADV_LAYOUT_EN_REG    0x28008
 
index 5b82177..f6d6502 100644 (file)
@@ -266,16 +266,6 @@ struct hclgevf_cfg_tx_queue_pointer_cmd {
 
 #define HCLGEVF_TYPE_CRQ               0
 #define HCLGEVF_TYPE_CSQ               1
-#define HCLGEVF_NIC_CSQ_BASEADDR_L_REG 0x27000
-#define HCLGEVF_NIC_CSQ_BASEADDR_H_REG 0x27004
-#define HCLGEVF_NIC_CSQ_DEPTH_REG      0x27008
-#define HCLGEVF_NIC_CSQ_TAIL_REG       0x27010
-#define HCLGEVF_NIC_CSQ_HEAD_REG       0x27014
-#define HCLGEVF_NIC_CRQ_BASEADDR_L_REG 0x27018
-#define HCLGEVF_NIC_CRQ_BASEADDR_H_REG 0x2701c
-#define HCLGEVF_NIC_CRQ_DEPTH_REG      0x27020
-#define HCLGEVF_NIC_CRQ_TAIL_REG       0x27024
-#define HCLGEVF_NIC_CRQ_HEAD_REG       0x27028
 
 /* this bit indicates that the driver is ready for hardware reset */
 #define HCLGEVF_NIC_SW_RST_RDY_B       16
index 60588b1..82e7270 100644 (file)
@@ -40,16 +40,16 @@ static const u8 hclgevf_hash_key[] = {
 
 MODULE_DEVICE_TABLE(pci, ae_algovf_pci_tbl);
 
-static const u32 cmdq_reg_addr_list[] = {HCLGEVF_CMDQ_TX_ADDR_L_REG,
-                                        HCLGEVF_CMDQ_TX_ADDR_H_REG,
-                                        HCLGEVF_CMDQ_TX_DEPTH_REG,
-                                        HCLGEVF_CMDQ_TX_TAIL_REG,
-                                        HCLGEVF_CMDQ_TX_HEAD_REG,
-                                        HCLGEVF_CMDQ_RX_ADDR_L_REG,
-                                        HCLGEVF_CMDQ_RX_ADDR_H_REG,
-                                        HCLGEVF_CMDQ_RX_DEPTH_REG,
-                                        HCLGEVF_CMDQ_RX_TAIL_REG,
-                                        HCLGEVF_CMDQ_RX_HEAD_REG,
+static const u32 cmdq_reg_addr_list[] = {HCLGEVF_NIC_CSQ_BASEADDR_L_REG,
+                                        HCLGEVF_NIC_CSQ_BASEADDR_H_REG,
+                                        HCLGEVF_NIC_CSQ_DEPTH_REG,
+                                        HCLGEVF_NIC_CSQ_TAIL_REG,
+                                        HCLGEVF_NIC_CSQ_HEAD_REG,
+                                        HCLGEVF_NIC_CRQ_BASEADDR_L_REG,
+                                        HCLGEVF_NIC_CRQ_BASEADDR_H_REG,
+                                        HCLGEVF_NIC_CRQ_DEPTH_REG,
+                                        HCLGEVF_NIC_CRQ_TAIL_REG,
+                                        HCLGEVF_NIC_CRQ_HEAD_REG,
                                         HCLGEVF_VECTOR0_CMDQ_SRC_REG,
                                         HCLGEVF_VECTOR0_CMDQ_STATE_REG,
                                         HCLGEVF_CMDQ_INTR_EN_REG,
@@ -1963,7 +1963,7 @@ static void hclgevf_dump_rst_info(struct hclgevf_dev *hdev)
        dev_info(&hdev->pdev->dev, "vector0 interrupt status: 0x%x\n",
                 hclgevf_read_dev(&hdev->hw, HCLGEVF_VECTOR0_CMDQ_STATE_REG));
        dev_info(&hdev->pdev->dev, "handshake status: 0x%x\n",
-                hclgevf_read_dev(&hdev->hw, HCLGEVF_CMDQ_TX_DEPTH_REG));
+                hclgevf_read_dev(&hdev->hw, HCLGEVF_NIC_CSQ_DEPTH_REG));
        dev_info(&hdev->pdev->dev, "function reset status: 0x%x\n",
                 hclgevf_read_dev(&hdev->hw, HCLGEVF_RST_ING));
        dev_info(&hdev->pdev->dev, "hdev state: 0x%lx\n", hdev->state);
index 1de8e2d..883130a 100644 (file)
 #define HCLGEVF_VECTOR_VF_OFFSET               0x100000
 
 /* bar registers for cmdq */
-#define HCLGEVF_CMDQ_TX_ADDR_L_REG             0x27000
-#define HCLGEVF_CMDQ_TX_ADDR_H_REG             0x27004
-#define HCLGEVF_CMDQ_TX_DEPTH_REG              0x27008
-#define HCLGEVF_CMDQ_TX_TAIL_REG               0x27010
-#define HCLGEVF_CMDQ_TX_HEAD_REG               0x27014
-#define HCLGEVF_CMDQ_RX_ADDR_L_REG             0x27018
-#define HCLGEVF_CMDQ_RX_ADDR_H_REG             0x2701C
-#define HCLGEVF_CMDQ_RX_DEPTH_REG              0x27020
-#define HCLGEVF_CMDQ_RX_TAIL_REG               0x27024
-#define HCLGEVF_CMDQ_RX_HEAD_REG               0x27028
+#define HCLGEVF_NIC_CSQ_BASEADDR_L_REG         0x27000
+#define HCLGEVF_NIC_CSQ_BASEADDR_H_REG         0x27004
+#define HCLGEVF_NIC_CSQ_DEPTH_REG              0x27008
+#define HCLGEVF_NIC_CSQ_TAIL_REG               0x27010
+#define HCLGEVF_NIC_CSQ_HEAD_REG               0x27014
+#define HCLGEVF_NIC_CRQ_BASEADDR_L_REG         0x27018
+#define HCLGEVF_NIC_CRQ_BASEADDR_H_REG         0x2701C
+#define HCLGEVF_NIC_CRQ_DEPTH_REG              0x27020
+#define HCLGEVF_NIC_CRQ_TAIL_REG               0x27024
+#define HCLGEVF_NIC_CRQ_HEAD_REG               0x27028
+
 #define HCLGEVF_CMDQ_INTR_EN_REG               0x27108
 #define HCLGEVF_CMDQ_INTR_GEN_REG              0x2710C