2020-05-20 11:13:17

by Horatiu Vultur

Subject: [PATCH 2/3] switchdev: mrp: Remove the variable mrp_ring_state

Remove the variable mrp_ring_state from switchdev_attr because is not
used anywhere.
The ring state is set using SWITCHDEV_OBJ_ID_RING_STATE_MRP.

Fixes: c284b5459008 ("switchdev: mrp: Extend switchdev API to offload MRP")
Signed-off-by: Horatiu Vultur <[email protected]>
---
include/net/switchdev.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/include/net/switchdev.h b/include/net/switchdev.h
index ae7aeb0d1f9ca..db519957e134b 100644
--- a/include/net/switchdev.h
+++ b/include/net/switchdev.h
@@ -62,7 +62,6 @@ struct switchdev_attr {
#if IS_ENABLED(CONFIG_BRIDGE_MRP)
u8 mrp_port_state; /* MRP_PORT_STATE */
u8 mrp_port_role; /* MRP_PORT_ROLE */
- u8 mrp_ring_state; /* MRP_RING_STATE */
#endif
} u;
};
--
2.26.2

2020-05-20 16:01:18

by Ivan Vecera

Subject: Re: [PATCH 2/3] switchdev: mrp: Remove the variable mrp_ring_state

On Wed, 20 May 2020 13:09:22 +0000
Horatiu Vultur <[email protected]> wrote:

> Remove the variable mrp_ring_state from switchdev_attr because is not
> used anywhere.
> The ring state is set using SWITCHDEV_OBJ_ID_RING_STATE_MRP.
>
> Fixes: c284b5459008 ("switchdev: mrp: Extend switchdev API to offload MRP")
> Signed-off-by: Horatiu Vultur <[email protected]>
> ---
> include/net/switchdev.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/include/net/switchdev.h b/include/net/switchdev.h
> index ae7aeb0d1f9ca..db519957e134b 100644
> --- a/include/net/switchdev.h
> +++ b/include/net/switchdev.h
> @@ -62,7 +62,6 @@ struct switchdev_attr {
> #if IS_ENABLED(CONFIG_BRIDGE_MRP)
> u8 mrp_port_state; /* MRP_PORT_STATE */
> u8 mrp_port_role; /* MRP_PORT_ROLE */
> - u8 mrp_ring_state; /* MRP_RING_STATE */
> #endif
> } u;
> };

Acked-by: Ivan Vecera <[email protected]>