diff --git a/drivers/net/ethernet/huawei/hinic/hinic_cfg.c b/drivers/net/ethernet/huawei/hinic/hinic_cfg.c
index 3684f9ab99e013acad635d570232cacf86851a83..fbacba16f35964f2791c443c1e8f6cd9b649da5c 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_cfg.c
+++ b/drivers/net/ethernet/huawei/hinic/hinic_cfg.c
@@ -711,9 +711,7 @@ static void rdma_param_fix(struct hinic_hwdev *dev)
 	rdma_cap->max_gid_per_port = MAX_GID_PER_PORT;
 	rdma_cap->gid_entry_sz = GID_ENTRY_SZ;
 	rdma_cap->reserved_lkey = RSVD_LKEY;
-	/* start */
 	rdma_cap->num_comp_vectors = (u32)dev->cfg_mgmt->eq_info.num_ceq;
-	/* end */
 	rdma_cap->page_size_cap = PAGE_SZ_CAP;
 	rdma_cap->flags = (RDMA_BMME_FLAG_LOCAL_INV |
 			   RDMA_BMME_FLAG_REMOTE_INV |
diff --git a/drivers/net/ethernet/huawei/hinic/hinic_cfg.h b/drivers/net/ethernet/huawei/hinic/hinic_cfg.h
index b31c624ef61db50f5a7bde344ebe90b2a3026cb7..55d038335f3c4d0f7d09a7b68a504c2046488326 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_cfg.h
+++ b/drivers/net/ethernet/huawei/hinic/hinic_cfg.h
@@ -122,7 +122,7 @@ enum {
 
 #define LOCAL_ACK_DELAY     15
 #define RDMA_NUM_PORTS      1
-#define ROCE_MAX_MSG_SZ     (2 * G_UNIT) /*  */
+#define ROCE_MAX_MSG_SZ     (2 * G_UNIT)
 #define IWARP_MAX_MSG_SZ    (1 * G_UNIT)
 
 #define DB_PAGE_SZ          (4 * K_UNIT)
diff --git a/drivers/net/ethernet/huawei/hinic/hinic_hwdev.c b/drivers/net/ethernet/huawei/hinic/hinic_hwdev.c
index 74582e62bf69657928bf310e263ff5fb6591c6b5..86e6da167bb2c907de03800136ec62aa02d785fb 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_hwdev.c
+++ b/drivers/net/ethernet/huawei/hinic/hinic_hwdev.c
@@ -1434,7 +1434,6 @@ static int hinic_vf_rx_tx_flush_in_pf(struct hinic_hwdev *hwdev, u16 vf_id)
 			ret = -EFAULT;
 	}
 
-	/* wait ucode stop I/O */
 	msleep(100);
 
 	/* notice up begine vf flush */
@@ -1467,7 +1466,7 @@ static int hinic_pf_rx_tx_flush(struct hinic_hwdev *hwdev)
 	int err;
 	int ret = 0;
 
-	/*  wait ucode stop I/O */
+	/* wait ucode stop I/O */
 	msleep(100);
 
 	err = wait_cmdq_stop(hwdev);
@@ -4399,7 +4398,7 @@ static u8 hinic_get_heartbeat_status(struct hinic_hwdev *hwdev)
 		sdk_err(hwdev->dev_hdl, "Detect pcie is link down\n");
 		hinic_set_chip_absent(hwdev);
 		hinic_force_complete_all(hwdev);
-	/* :should notify chiperr to pangea
+	/* should notify chiperr to pangea
 	 * when detecting pcie link down
 	 */
 		return 1;
diff --git a/drivers/net/ethernet/huawei/hinic/hinic_lld.c b/drivers/net/ethernet/huawei/hinic/hinic_lld.c
index 86e499d26bc6604451b944f905f0d5b82bb59d00..4260ed404c6576404d698a423bcd8a1b08585f71 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_lld.c
+++ b/drivers/net/ethernet/huawei/hinic/hinic_lld.c
@@ -1923,7 +1923,7 @@ static int mapping_bar(struct pci_dev *pdev, struct hinic_pcidev *pci_adapter)
 
 	dwqe_addr = pci_adapter->db_base_phy + HINIC_DB_DWQE_SIZE;
 
-	/* arm do not support call ioremap_wc(), refer to  */
+	/* arm do not support call ioremap_wc() */
 	pci_adapter->dwqe_mapping = __ioremap(dwqe_addr, HINIC_DB_DWQE_SIZE,
 					      __pgprot(PROT_DEVICE_nGnRnE));
 	if (!pci_adapter->dwqe_mapping) {
diff --git a/drivers/net/ethernet/huawei/hinic/hinic_tx.c b/drivers/net/ethernet/huawei/hinic/hinic_tx.c
index 276e4492617a1e20ef0410e7dbb3179b624baabd..0088acc9836b427e7929a56597ea3b20214a0363 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_tx.c
+++ b/drivers/net/ethernet/huawei/hinic/hinic_tx.c
@@ -804,7 +804,7 @@ static netdev_tx_t hinic_send_one_skb(struct sk_buff *skb,
 
 	num_sge = skb_nr_frags + 1;
 
-	/* :if skb->len is more than 65536B but num_sge is 1,
+	/* if skb->len is more than 65536B but num_sge is 1,
 	 * driver will drop it
 	 */
 	if (unlikely(skb->len > HINIC_GSO_MAX_SIZE && num_sge == 1)) {