[PATCH net-next 0/3] net: hns3: misc updates for -next

[PATCH net-next 0/3] net: hns3: misc updates for -next

[Huazhong Tan]

This series includes some misc updates for the HNS3 ethernet driver.

Huazhong Tan (3):
  net: hns3: remove a duplicate pf reset counting
  net: hns3: cleanup inappropriate spaces in struct hlcgevf_tqp_stats
  net: hns3: change the value of the SEPARATOR_VALUE macro in
    hclgevf_main.c

 drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c   | 2 --
 drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c | 2 +-
 drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h | 4 ++--
 3 files changed, 3 insertions(+), 5 deletions(-)

-- 
2.7.4

[PATCH net-next 1/3] net: hns3: remove a duplicate pf reset counting

[Huazhong Tan]

When enter suspend mode the counter of pf reset will be increased
twice, since both hclge_prepare_general() and hclge_prepare_wait()
increase this counter. So remove the duplicate counting in
hclge_prepare_general().

Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
 drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index dc06986..c296ab6 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -11126,8 +11126,6 @@ static void hclge_reset_prepare_general(struct hnae3_ae_dev *ae_dev,
 
 	if (hdev->reset_type == HNAE3_FLR_RESET)
 		hdev->rst_stats.flr_rst_cnt++;
-	else if (hdev->reset_type == HNAE3_FUNC_RESET)
-		hdev->rst_stats.pf_rst_cnt++;
 }
 
 static void hclge_reset_done(struct hnae3_ae_dev *ae_dev)
-- 
2.7.4

[PATCH net-next 2/3] net: hns3: cleanup inappropriate spaces in struct hlcgevf_tqp_stats

[Huazhong Tan]

Modify some inappropriate spaces in comments of struct
hlcgevf_tqp_stats.

Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
 drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
index 956095b..265c9b0 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
@@ -177,9 +177,9 @@ struct hclgevf_hw {
 
 /* TQP stats */
 struct hlcgevf_tqp_stats {
-	/* query_tqp_tx_queue_statistics ,opcode id:  0x0B03 */
+	/* query_tqp_tx_queue_statistics, opcode id: 0x0B03 */
 	u64 rcb_tx_ring_pktnum_rcd; /* 32bit */
-	/* query_tqp_rx_queue_statistics ,opcode id:  0x0B13 */
+	/* query_tqp_rx_queue_statistics, opcode id: 0x0B13 */
 	u64 rcb_rx_ring_pktnum_rcd; /* 32bit */
 };
 
-- 
2.7.4

[PATCH net-next 3/3] net: hns3: change the value of the SEPARATOR_VALUE macro in hclgevf_main.c

[Huazhong Tan]

The SEPARATOR_VALUE macro is used as separator when getting
the register value, but the value of this macro is different
between pf and vf, it is a bit confusing for the user, so
synchronize the value of vf with pf.

Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
 drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
index 07066c4..0db51ef1 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
@@ -3646,7 +3646,7 @@ static void hclgevf_get_link_mode(struct hnae3_handle *handle,
 }
 
 #define MAX_SEPARATE_NUM	4
-#define SEPARATOR_VALUE		0xFFFFFFFF
+#define SEPARATOR_VALUE		0xFDFCFBFA
 #define REG_NUM_PER_LINE	4
 #define REG_LEN_PER_LINE	(REG_NUM_PER_LINE * sizeof(u32))
 
-- 
2.7.4

Re: [PATCH net-next 0/3] net: hns3: misc updates for -next

[patchwork-bot+netdevbpf]

Hello:

This series was applied to netdev/net-next.git (refs/heads/master):

On Sat, 17 Apr 2021 15:09:21 +0800 you wrote:
> This series includes some misc updates for the HNS3 ethernet driver.
> 
> Huazhong Tan (3):
>   net: hns3: remove a duplicate pf reset counting
>   net: hns3: cleanup inappropriate spaces in struct hlcgevf_tqp_stats
>   net: hns3: change the value of the SEPARATOR_VALUE macro in
>     hclgevf_main.c
> 
> [...]

Here is the summary with links:
  - [net-next,1/3] net: hns3: remove a duplicate pf reset counting
    https://git.kernel.org/netdev/net-next/c/1c5a2ba67989
  - [net-next,2/3] net: hns3: cleanup inappropriate spaces in struct hlcgevf_tqp_stats
    https://git.kernel.org/netdev/net-next/c/8ed64dbe0bdf
  - [net-next,3/3] net: hns3: change the value of the SEPARATOR_VALUE macro in hclgevf_main.c
    https://git.kernel.org/netdev/net-next/c/e407efdd94cd

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html