Commit dd112358 authored by Yonglong Liu's avatar Yonglong Liu Committed by Hao Chen
Browse files

net: hns3: delete redundant enabling actions for Layer 2 fowarding 

driver inclusion
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IAN3KC


CVE: NA

----------------------------------------------------------------------

Due to the datasheet, the HCLGE_SWITCH_ALW_DST_OVRD_B bit is not for Layer
2 fowarding function, so delete the redundant codes.

Fixes: 18c7dc4f ("net: hns3: add support to get/set 1d torus param")
Signed-off-by: default avatarYonglong Liu <liuyonglong@huawei.com>
parent 2e07192a

drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h

+0 −2
Original line number Diff line number Diff line
@@ -514,12 +514,10 @@ struct hclge_port_vlan_filter_bypass_cmd { 
#define HCLGE_SWITCH_ANTI_SPOOF_B	0U

#define HCLGE_SWITCH_ALW_LPBK_B		1U

#define HCLGE_SWITCH_ALW_LCL_LPBK_B	2U

#define HCLGE_SWITCH_ALW_DST_OVRD_B	3U

#define HCLGE_SWITCH_NO_MASK		0x0

#define HCLGE_SWITCH_ANTI_SPOOF_MASK	0xFE

#define HCLGE_SWITCH_ALW_LPBK_MASK	0xFD

#define HCLGE_SWITCH_ALW_LCL_LPBK_MASK	0xFB

#define HCLGE_SWITCH_LW_DST_OVRD_MASK	0xF7



struct hclge_mac_vlan_switch_cmd {

	u8 roce_sel;

drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_ext.c

+0 −3
Original line number Diff line number Diff line
@@ -180,9 +180,6 @@ static int hclge_torus_cfg_switch(struct hclge_dev *hdev, bool is_rocee, 
	hnae3_set_bit(req->switch_param, HCLGE_SWITCH_ALW_LPBK_B, 1);

	hnae3_set_bit(req->switch_param, HCLGE_SWITCH_ALW_LCL_LPBK_B, 0);

	hnae3_set_bit(req->switch_param, HCLGE_SWITCH_ANTI_SPOOF_B, enabled);

	if (!is_rocee)

		hnae3_set_bit(req->switch_param, HCLGE_SWITCH_ALW_DST_OVRD_B,

			      enabled);



	hclge_comm_cmd_reuse_desc(&desc, false);

	ret = hclge_cmd_send(&hdev->hw, &desc, 1);