diff mbox series

[net,2/2] switchdev: mrp: Remove SWITCHDEV_ATTR_ID_MRP_PORT_STAT

Message ID 20210206214734.1577849-3-horatiu.vultur@microchip.com
State New
Headers show
Series bridge: mrp: Fix br_mrp_port_switchdev_set_state | expand

Commit Message

Horatiu Vultur Feb. 6, 2021, 9:47 p.m. UTC
Now that MRP started to use also SWITCHDEV_ATTR_ID_PORT_STP_STATE to
notify HW, then SWITCHDEV_ATTR_ID_MRP_PORT_STAT is not used anywhere
else, therefore we can remove it.

Fixes: c284b545900830 ("switchdev: mrp: Extend switchdev API to offload MRP")
Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
---
 include/net/switchdev.h | 2 --
 1 file changed, 2 deletions(-)

Comments

Rasmus Villemoes Feb. 9, 2021, 7:38 a.m. UTC | #1
On 06/02/2021 22.47, Horatiu Vultur wrote:
> Now that MRP started to use also SWITCHDEV_ATTR_ID_PORT_STP_STATE to

> notify HW, then SWITCHDEV_ATTR_ID_MRP_PORT_STAT is not used anywhere

> else, therefore we can remove it.

> 

> Fixes: c284b545900830 ("switchdev: mrp: Extend switchdev API to offload MRP")

> Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>


Acked-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
diff mbox series

Patch

diff --git a/include/net/switchdev.h b/include/net/switchdev.h
index 99cd538d6519..afdf8bd1b4fe 100644
--- a/include/net/switchdev.h
+++ b/include/net/switchdev.h
@@ -42,7 +42,6 @@  enum switchdev_attr_id {
 	SWITCHDEV_ATTR_ID_BRIDGE_MC_DISABLED,
 	SWITCHDEV_ATTR_ID_BRIDGE_MROUTER,
 #if IS_ENABLED(CONFIG_BRIDGE_MRP)
-	SWITCHDEV_ATTR_ID_MRP_PORT_STATE,
 	SWITCHDEV_ATTR_ID_MRP_PORT_ROLE,
 #endif
 };
@@ -62,7 +61,6 @@  struct switchdev_attr {
 		u16 vlan_protocol;			/* BRIDGE_VLAN_PROTOCOL */
 		bool mc_disabled;			/* MC_DISABLED */
 #if IS_ENABLED(CONFIG_BRIDGE_MRP)
-		u8 mrp_port_state;			/* MRP_PORT_STATE */
 		u8 mrp_port_role;			/* MRP_PORT_ROLE */
 #endif
 	} u;