* [PATCH net-next 00/11] net: hns3: misc updates for -next
@ 2020-05-28 12:45 Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 01/11] net: hns3: add a missing mutex destroy in hclge_init_ad_dev() Huazhong Tan
` (11 more replies)
0 siblings, 12 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
This patchset includes some updates for the HNS3 ethernet driver.
#1 adds a missing mutex destroy.
#2&3 refactor two function, make them more readable and maintainable.
#4&5 fix unsuitable type of gro enable field both for PF & VF.
#6-#10 removes some unused fields, macro and redundant definitions.
#11 adds more debug info for parsing speed fails.
Huazhong Tan (11):
net: hns3: add a missing mutex destroy in hclge_init_ad_dev()
net: hns3: refactor hclge_config_tso()
net: hns3: refactor hclge_query_bd_num_cmd_send()
net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd
net: hns3: modify an incorrect type in struct
hclgevf_cfg_gro_status_cmd
net: hns3: remove some unused fields in struct hns3_nic_priv
net: hns3; remove unused HNAE3_RESTORE_CLIENT in enum
hnae3_reset_notify_type
net: hns3: remove unused struct hnae3_unic_private_info
net: hns3: remove two duplicated register macros in hclgevf_main.h
net: hns3: remove some unused fields in struct hclge_dev
net: hns3: print out speed info when parsing speed fails
drivers/net/ethernet/hisilicon/hns3/hnae3.h | 12 ------
drivers/net/ethernet/hisilicon/hns3/hns3_enet.h | 22 -----------
.../net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h | 4 +-
.../ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 44 ++++++++++------------
.../ethernet/hisilicon/hns3/hns3pf/hclge_main.h | 6 ---
.../ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h | 4 +-
.../ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c | 8 ++--
.../ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h | 4 +-
8 files changed, 29 insertions(+), 75 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH net-next 01/11] net: hns3: add a missing mutex destroy in hclge_init_ad_dev()
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 02/11] net: hns3: refactor hclge_config_tso() Huazhong Tan
` (10 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Add a mutex destroy call in hclge_init_ae_dev() when fails.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index 7c9f2ba..ff8ad4b 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -10110,6 +10110,7 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
pci_release_regions(pdev);
pci_disable_device(pdev);
out:
+ mutex_destroy(&hdev->vport_lock);
return ret;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 02/11] net: hns3: refactor hclge_config_tso()
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 01/11] net: hns3: add a missing mutex destroy in hclge_init_ad_dev() Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 03/11] net: hns3: refactor hclge_query_bd_num_cmd_send() Huazhong Tan
` (9 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Since parameters 'tso_mss_min' and 'tso_mss_max' only indicate
the minimum and maximum MSS, the hnae3_set_field() calls are
meaningless, remove them and change the type of these two
parameters to u16.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 17 ++++-------------
1 file changed, 4 insertions(+), 13 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index ff8ad4b..21ad23e 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -1429,26 +1429,17 @@ static int hclge_configure(struct hclge_dev *hdev)
return ret;
}
-static int hclge_config_tso(struct hclge_dev *hdev, unsigned int tso_mss_min,
- unsigned int tso_mss_max)
+static int hclge_config_tso(struct hclge_dev *hdev, u16 tso_mss_min,
+ u16 tso_mss_max)
{
struct hclge_cfg_tso_status_cmd *req;
struct hclge_desc desc;
- u16 tso_mss;
hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_TSO_GENERIC_CONFIG, false);
req = (struct hclge_cfg_tso_status_cmd *)desc.data;
-
- tso_mss = 0;
- hnae3_set_field(tso_mss, HCLGE_TSO_MSS_MIN_M,
- HCLGE_TSO_MSS_MIN_S, tso_mss_min);
- req->tso_mss_min = cpu_to_le16(tso_mss);
-
- tso_mss = 0;
- hnae3_set_field(tso_mss, HCLGE_TSO_MSS_MIN_M,
- HCLGE_TSO_MSS_MIN_S, tso_mss_max);
- req->tso_mss_max = cpu_to_le16(tso_mss);
+ req->tso_mss_min = cpu_to_le16(tso_mss_min);
+ req->tso_mss_max = cpu_to_le16(tso_mss_max);
return hclge_cmd_send(&hdev->hw, &desc, 1);
}
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 03/11] net: hns3: refactor hclge_query_bd_num_cmd_send()
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 01/11] net: hns3: add a missing mutex destroy in hclge_init_ad_dev() Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 02/11] net: hns3: refactor hclge_config_tso() Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 04/11] net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd Huazhong Tan
` (8 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
In order to improve code maintainability and readability, rewrite
the process of BDs' initialization in hclge_query_bd_num_cmd_send().
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
.../net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 21 ++++++++++++---------
1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index 21ad23e..6c84413 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -10725,16 +10725,19 @@ static int hclge_get_64_bit_regs(struct hclge_dev *hdev, u32 regs_num,
int hclge_query_bd_num_cmd_send(struct hclge_dev *hdev, struct hclge_desc *desc)
{
- /*prepare 4 commands to query DFX BD number*/
- hclge_cmd_setup_basic_desc(&desc[0], HCLGE_OPC_DFX_BD_NUM, true);
- desc[0].flag |= cpu_to_le16(HCLGE_CMD_FLAG_NEXT);
- hclge_cmd_setup_basic_desc(&desc[1], HCLGE_OPC_DFX_BD_NUM, true);
- desc[1].flag |= cpu_to_le16(HCLGE_CMD_FLAG_NEXT);
- hclge_cmd_setup_basic_desc(&desc[2], HCLGE_OPC_DFX_BD_NUM, true);
- desc[2].flag |= cpu_to_le16(HCLGE_CMD_FLAG_NEXT);
- hclge_cmd_setup_basic_desc(&desc[3], HCLGE_OPC_DFX_BD_NUM, true);
+ int i;
+
+ /* initialize command BD except the last one */
+ for (i = 0; i < HCLGE_GET_DFX_REG_TYPE_CNT - 1; i++) {
+ hclge_cmd_setup_basic_desc(&desc[i], HCLGE_OPC_DFX_BD_NUM,
+ true);
+ desc[i].flag |= cpu_to_le16(HCLGE_CMD_FLAG_NEXT);
+ }
+
+ /* initialize the last command BD */
+ hclge_cmd_setup_basic_desc(&desc[i], HCLGE_OPC_DFX_BD_NUM, true);
- return hclge_cmd_send(&hdev->hw, desc, 4);
+ return hclge_cmd_send(&hdev->hw, desc, HCLGE_GET_DFX_REG_TYPE_CNT);
}
static int hclge_get_dfx_reg_bd_num(struct hclge_dev *hdev,
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 04/11] net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (2 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 03/11] net: hns3: refactor hclge_query_bd_num_cmd_send() Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 05/11] net: hns3: modify an incorrect type in struct hclgevf_cfg_gro_status_cmd Huazhong Tan
` (7 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Modify field .gro_en in struct hclge_cfg_gro_status_cmd to u8
according to the UM, otherwise, it will overwrite the reserved
byte which may be used for other purpose.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h | 4 ++--
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
index e3bab8f..463f291 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
@@ -884,8 +884,8 @@ struct hclge_cfg_tso_status_cmd {
#define HCLGE_GRO_EN_B 0
struct hclge_cfg_gro_status_cmd {
- __le16 gro_en;
- u8 rsv[22];
+ u8 gro_en;
+ u8 rsv[23];
};
#define HCLGE_TSO_MSS_MIN 256
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index 6c84413..87473f7 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -1456,7 +1456,7 @@ static int hclge_config_gro(struct hclge_dev *hdev, bool en)
hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_GRO_GENERIC_CONFIG, false);
req = (struct hclge_cfg_gro_status_cmd *)desc.data;
- req->gro_en = cpu_to_le16(en ? 1 : 0);
+ req->gro_en = en ? 1 : 0;
ret = hclge_cmd_send(&hdev->hw, &desc, 1);
if (ret)
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 05/11] net: hns3: modify an incorrect type in struct hclgevf_cfg_gro_status_cmd
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (3 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 04/11] net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 06/11] net: hns3: remove some unused fields in struct hns3_nic_priv Huazhong Tan
` (6 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Modify field .gro_en in struct hclgevf_cfg_gro_status_cmd to u8
according to the UM, otherwise, it will overwrite the reserved
byte which may be used for other purpose.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h | 4 ++--
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h
index f830eef..40d6e602 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h
@@ -161,8 +161,8 @@ struct hclgevf_query_res_cmd {
#define HCLGEVF_GRO_EN_B 0
struct hclgevf_cfg_gro_status_cmd {
- __le16 gro_en;
- u8 rsv[22];
+ u8 gro_en;
+ u8 rsv[23];
};
#define HCLGEVF_RSS_DEFAULT_OUTPORT_B 4
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
index 59fcb80..be5789b 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
@@ -2403,7 +2403,7 @@ static int hclgevf_config_gro(struct hclgevf_dev *hdev, bool en)
false);
req = (struct hclgevf_cfg_gro_status_cmd *)desc.data;
- req->gro_en = cpu_to_le16(en ? 1 : 0);
+ req->gro_en = en ? 1 : 0;
ret = hclgevf_cmd_send(&hdev->hw, &desc, 1);
if (ret)
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 06/11] net: hns3: remove some unused fields in struct hns3_nic_priv
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (4 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 05/11] net: hns3: modify an incorrect type in struct hclgevf_cfg_gro_status_cmd Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 07/11] net: hns3; remove unused HNAE3_RESTORE_CLIENT in enum hnae3_reset_notify_type Huazhong Tan
` (5 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Remove some fileds which defined in struct hns3_nic_priv,
but not used, and remove the related definition of struct
hns3_udp_tunnel and enum hns3_udp_tnl_type.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3_enet.h | 22 ----------------------
1 file changed, 22 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3_enet.h b/drivers/net/ethernet/hisilicon/hns3/hns3_enet.h
index 60f82ad..66cd439 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3_enet.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3_enet.h
@@ -469,21 +469,8 @@ struct hns3_enet_tqp_vector {
unsigned long last_jiffies;
} ____cacheline_internodealigned_in_smp;
-enum hns3_udp_tnl_type {
- HNS3_UDP_TNL_VXLAN,
- HNS3_UDP_TNL_GENEVE,
- HNS3_UDP_TNL_MAX,
-};
-
-struct hns3_udp_tunnel {
- u16 dst_port;
- int used;
-};
-
struct hns3_nic_priv {
struct hnae3_handle *ae_handle;
- u32 enet_ver;
- u32 port_id;
struct net_device *netdev;
struct device *dev;
@@ -495,19 +482,10 @@ struct hns3_nic_priv {
struct hns3_enet_tqp_vector *tqp_vector;
u16 vector_num;
- /* The most recently read link state */
- int link;
u64 tx_timeout_count;
unsigned long state;
- struct timer_list service_timer;
-
- struct work_struct service_task;
-
- struct notifier_block notifier_block;
- /* Vxlan/Geneve information */
- struct hns3_udp_tunnel udp_tnl[HNS3_UDP_TNL_MAX];
struct hns3_enet_coalesce tx_coal;
struct hns3_enet_coalesce rx_coal;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 07/11] net: hns3; remove unused HNAE3_RESTORE_CLIENT in enum hnae3_reset_notify_type
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (5 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 06/11] net: hns3: remove some unused fields in struct hns3_nic_priv Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 08/11] net: hns3: remove unused struct hnae3_unic_private_info Huazhong Tan
` (4 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Remove HNAE3_RESTORE_CLIENT which is not needed now.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hnae3.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hnae3.h b/drivers/net/ethernet/hisilicon/hns3/hnae3.h
index 7506cab..0a4aac4 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hnae3.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hnae3.h
@@ -145,7 +145,6 @@ enum hnae3_reset_notify_type {
HNAE3_DOWN_CLIENT,
HNAE3_INIT_CLIENT,
HNAE3_UNINIT_CLIENT,
- HNAE3_RESTORE_CLIENT,
};
enum hnae3_hw_error_type {
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 08/11] net: hns3: remove unused struct hnae3_unic_private_info
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (6 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 07/11] net: hns3; remove unused HNAE3_RESTORE_CLIENT in enum hnae3_reset_notify_type Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 09/11] net: hns3: remove two duplicated register macros in hclgevf_main.h Huazhong Tan
` (3 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Since field .uinfo in struct hnae3_handle never be used,
so remove it and its structure definition.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hnae3.h | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hnae3.h b/drivers/net/ethernet/hisilicon/hns3/hnae3.h
index 0a4aac4..d041cac 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hnae3.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hnae3.h
@@ -621,16 +621,6 @@ struct hnae3_roce_private_info {
unsigned long state;
};
-struct hnae3_unic_private_info {
- struct net_device *netdev;
- u16 rx_buf_len;
- u16 num_tx_desc;
- u16 num_rx_desc;
-
- u16 num_tqps; /* total number of tqps in this handle */
- struct hnae3_queue **tqp; /* array base of all TQPs of this instance */
-};
-
#define HNAE3_SUPPORT_APP_LOOPBACK BIT(0)
#define HNAE3_SUPPORT_PHY_LOOPBACK BIT(1)
#define HNAE3_SUPPORT_SERDES_SERIAL_LOOPBACK BIT(2)
@@ -656,7 +646,6 @@ struct hnae3_handle {
union {
struct net_device *netdev; /* first member */
struct hnae3_knic_private_info kinfo;
- struct hnae3_unic_private_info uinfo;
struct hnae3_roce_private_info rinfo;
};
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 09/11] net: hns3: remove two duplicated register macros in hclgevf_main.h
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (7 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 08/11] net: hns3: remove unused struct hnae3_unic_private_info Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 10/11] net: hns3: remove some unused fields in struct hclge_dev Huazhong Tan
` (2 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
HCLGEVF_CMDQ_INTR_SRC_REG and HCLGEVF_CMDQ_INTR_STS_REG are same
as HCLGEVF_VECTOR0_CMDQ_SRC_REG and HCLGEVF_VECTOR0_CMDQ_STAT_REG,
replace the former with the latter, and rename macro
HCLGEVF_VECTOR0_CMDQ_STAT_REG since 'stat' is not abbreviation of
'state'.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c | 6 +++---
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h | 4 +---
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
index be5789b..a8c0e79 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c
@@ -46,7 +46,7 @@ static const u32 cmdq_reg_addr_list[] = {HCLGEVF_CMDQ_TX_ADDR_L_REG,
HCLGEVF_CMDQ_RX_TAIL_REG,
HCLGEVF_CMDQ_RX_HEAD_REG,
HCLGEVF_VECTOR0_CMDQ_SRC_REG,
- HCLGEVF_CMDQ_INTR_STS_REG,
+ HCLGEVF_VECTOR0_CMDQ_STATE_REG,
HCLGEVF_CMDQ_INTR_EN_REG,
HCLGEVF_CMDQ_INTR_GEN_REG};
@@ -1826,7 +1826,7 @@ static void hclgevf_dump_rst_info(struct hclgevf_dev *hdev)
dev_info(&hdev->pdev->dev, "vector0 interrupt enable status: 0x%x\n",
hclgevf_read_dev(&hdev->hw, HCLGEVF_MISC_VECTOR_REG_BASE));
dev_info(&hdev->pdev->dev, "vector0 interrupt status: 0x%x\n",
- hclgevf_read_dev(&hdev->hw, HCLGEVF_VECTOR0_CMDQ_STAT_REG));
+ 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));
dev_info(&hdev->pdev->dev, "function reset status: 0x%x\n",
@@ -2250,7 +2250,7 @@ static enum hclgevf_evt_cause hclgevf_check_evt_cause(struct hclgevf_dev *hdev,
/* fetch the events from their corresponding regs */
cmdq_stat_reg = hclgevf_read_dev(&hdev->hw,
- HCLGEVF_VECTOR0_CMDQ_STAT_REG);
+ HCLGEVF_VECTOR0_CMDQ_STATE_REG);
if (BIT(HCLGEVF_VECTOR0_RST_INT_B) & cmdq_stat_reg) {
rst_ing_reg = hclgevf_read_dev(&hdev->hw, HCLGEVF_RST_ING);
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
index f19583c..738de12 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
@@ -42,8 +42,6 @@
#define HCLGEVF_CMDQ_RX_DEPTH_REG 0x27020
#define HCLGEVF_CMDQ_RX_TAIL_REG 0x27024
#define HCLGEVF_CMDQ_RX_HEAD_REG 0x27028
-#define HCLGEVF_CMDQ_INTR_SRC_REG 0x27100
-#define HCLGEVF_CMDQ_INTR_STS_REG 0x27104
#define HCLGEVF_CMDQ_INTR_EN_REG 0x27108
#define HCLGEVF_CMDQ_INTR_GEN_REG 0x2710C
@@ -88,7 +86,7 @@
/* Vector0 interrupt CMDQ event source register(RW) */
#define HCLGEVF_VECTOR0_CMDQ_SRC_REG 0x27100
/* Vector0 interrupt CMDQ event status register(RO) */
-#define HCLGEVF_VECTOR0_CMDQ_STAT_REG 0x27104
+#define HCLGEVF_VECTOR0_CMDQ_STATE_REG 0x27104
/* CMDQ register bits for RX event(=MBX event) */
#define HCLGEVF_VECTOR0_RX_CMDQ_INT_B 1
/* RST register bits for RESET event */
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 10/11] net: hns3: remove some unused fields in struct hclge_dev
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (8 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 09/11] net: hns3: remove two duplicated register macros in hclgevf_main.h Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 11/11] net: hns3: print out speed info when parsing speed fails Huazhong Tan
2020-05-28 12:49 ` [PATCH net-next 00/11] net: hns3: misc updates for -next tanhuazhong
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
Remove some fields in struct hclge_dev which have not been used.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h
index 913c4f6..46e6e0f 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h
@@ -771,12 +771,6 @@ struct hclge_dev {
u16 num_roce_msi; /* Num of roce vectors for this PF */
int roce_base_vector;
- u16 pending_udp_bitmap;
-
- u16 rx_itr_default;
- u16 tx_itr_default;
-
- u16 adminq_work_limit; /* Num of admin receive queue desc to process */
unsigned long service_timer_period;
unsigned long service_timer_previous;
struct timer_list reset_timer;
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH net-next 11/11] net: hns3: print out speed info when parsing speed fails
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (9 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 10/11] net: hns3: remove some unused fields in struct hclge_dev Huazhong Tan
@ 2020-05-28 12:45 ` Huazhong Tan
2020-05-28 12:49 ` [PATCH net-next 00/11] net: hns3: misc updates for -next tanhuazhong
11 siblings, 0 replies; 13+ messages in thread
From: Huazhong Tan @ 2020-05-28 12:45 UTC (permalink / raw)
To: davem
Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba,
Huazhong Tan
When calling hclge_parse_speed() fails, printing out the speed is
helpful for debugging.
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index 87473f7..21a73b0 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -1387,7 +1387,8 @@ static int hclge_configure(struct hclge_dev *hdev)
ret = hclge_parse_speed(cfg.default_speed, &hdev->hw.mac.speed);
if (ret) {
- dev_err(&hdev->pdev->dev, "Get wrong speed ret=%d.\n", ret);
+ dev_err(&hdev->pdev->dev, "failed to parse speed %u, ret = %d\n",
+ cfg.default_speed, ret);
return ret;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH net-next 00/11] net: hns3: misc updates for -next
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
` (10 preceding siblings ...)
2020-05-28 12:45 ` [PATCH net-next 11/11] net: hns3: print out speed info when parsing speed fails Huazhong Tan
@ 2020-05-28 12:49 ` tanhuazhong
11 siblings, 0 replies; 13+ messages in thread
From: tanhuazhong @ 2020-05-28 12:49 UTC (permalink / raw)
To: davem; +Cc: netdev, linux-kernel, salil.mehta, yisen.zhuang, linuxarm, kuba
Sorry, please ignore this patchset, will resend it later.
On 2020/5/28 20:45, Huazhong Tan wrote:
> This patchset includes some updates for the HNS3 ethernet driver.
>
> #1 adds a missing mutex destroy.
> #2&3 refactor two function, make them more readable and maintainable.
> #4&5 fix unsuitable type of gro enable field both for PF & VF.
> #6-#10 removes some unused fields, macro and redundant definitions.
> #11 adds more debug info for parsing speed fails.
>
> Huazhong Tan (11):
> net: hns3: add a missing mutex destroy in hclge_init_ad_dev()
> net: hns3: refactor hclge_config_tso()
> net: hns3: refactor hclge_query_bd_num_cmd_send()
> net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd
> net: hns3: modify an incorrect type in struct
> hclgevf_cfg_gro_status_cmd
> net: hns3: remove some unused fields in struct hns3_nic_priv
> net: hns3; remove unused HNAE3_RESTORE_CLIENT in enum
> hnae3_reset_notify_type
> net: hns3: remove unused struct hnae3_unic_private_info
> net: hns3: remove two duplicated register macros in hclgevf_main.h
> net: hns3: remove some unused fields in struct hclge_dev
> net: hns3: print out speed info when parsing speed fails
>
> drivers/net/ethernet/hisilicon/hns3/hnae3.h | 12 ------
> drivers/net/ethernet/hisilicon/hns3/hns3_enet.h | 22 -----------
> .../net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h | 4 +-
> .../ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 44 ++++++++++------------
> .../ethernet/hisilicon/hns3/hns3pf/hclge_main.h | 6 ---
> .../ethernet/hisilicon/hns3/hns3vf/hclgevf_cmd.h | 4 +-
> .../ethernet/hisilicon/hns3/hns3vf/hclgevf_main.c | 8 ++--
> .../ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h | 4 +-
> 8 files changed, 29 insertions(+), 75 deletions(-)
>
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-05-28 12:49 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-28 12:45 [PATCH net-next 00/11] net: hns3: misc updates for -next Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 01/11] net: hns3: add a missing mutex destroy in hclge_init_ad_dev() Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 02/11] net: hns3: refactor hclge_config_tso() Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 03/11] net: hns3: refactor hclge_query_bd_num_cmd_send() Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 04/11] net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 05/11] net: hns3: modify an incorrect type in struct hclgevf_cfg_gro_status_cmd Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 06/11] net: hns3: remove some unused fields in struct hns3_nic_priv Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 07/11] net: hns3; remove unused HNAE3_RESTORE_CLIENT in enum hnae3_reset_notify_type Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 08/11] net: hns3: remove unused struct hnae3_unic_private_info Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 09/11] net: hns3: remove two duplicated register macros in hclgevf_main.h Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 10/11] net: hns3: remove some unused fields in struct hclge_dev Huazhong Tan
2020-05-28 12:45 ` [PATCH net-next 11/11] net: hns3: print out speed info when parsing speed fails Huazhong Tan
2020-05-28 12:49 ` [PATCH net-next 00/11] net: hns3: misc updates for -next tanhuazhong
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.