diff mbox series

[net-next,v4,7/8] net: dsa: add MRP support

Message ID 20210216214205.32385-8-horatiu.vultur@microchip.com
State New
Headers show
Series bridge: mrp: Extend br_mrp_switchdev_* | expand

Commit Message

Horatiu Vultur Feb. 16, 2021, 9:42 p.m. UTC 
Add support for offloading MRP in HW. Currently implement the switchdev
calls 'SWITCHDEV_OBJ_ID_MRP', 'SWITCHDEV_OBJ_ID_RING_ROLE_MRP',
to allow to create MRP instances and to set the role of these instances.

Add DSA_NOTIFIER_MRP_ADD/DEL and DSA_NOTIFIER_MRP_ADD/DEL_RING_ROLE
which calls to .port_mrp_add/del and .port_mrp_add/del_ring_role in the
DSA driver for the switch.

Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
---
 include/net/dsa.h  |  12 ++++++
 net/dsa/dsa_priv.h |  26 +++++++++++
 net/dsa/port.c     |  48 +++++++++++++++++++++
 net/dsa/slave.c    |  22 ++++++++++
 net/dsa/switch.c   | 105 +++++++++++++++++++++++++++++++++++++++++++++
 5 files changed, 213 insertions(+)

Comments

Vladimir Oltean Feb. 17, 2021, 11:23 a.m. UTC | #1 
On Tue, Feb 16, 2021 at 10:42:04PM +0100, Horatiu Vultur wrote:
> Add support for offloading MRP in HW. Currently implement the switchdev

> calls 'SWITCHDEV_OBJ_ID_MRP', 'SWITCHDEV_OBJ_ID_RING_ROLE_MRP',

> to allow to create MRP instances and to set the role of these instances.

> 

> Add DSA_NOTIFIER_MRP_ADD/DEL and DSA_NOTIFIER_MRP_ADD/DEL_RING_ROLE

> which calls to .port_mrp_add/del and .port_mrp_add/del_ring_role in the

> DSA driver for the switch.

> 

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

> ---

>  include/net/dsa.h  |  12 ++++++

>  net/dsa/dsa_priv.h |  26 +++++++++++

>  net/dsa/port.c     |  48 +++++++++++++++++++++

>  net/dsa/slave.c    |  22 ++++++++++

>  net/dsa/switch.c   | 105 +++++++++++++++++++++++++++++++++++++++++++++

>  5 files changed, 213 insertions(+)

> 

> diff --git a/include/net/dsa.h b/include/net/dsa.h

> index 68f8159564a3..83a933e563fe 100644

> --- a/include/net/dsa.h

> +++ b/include/net/dsa.h

> @@ -792,6 +792,18 @@ struct dsa_switch_ops {

>  				 struct net_device *hsr);

>  	int	(*port_hsr_leave)(struct dsa_switch *ds, int port,

>  				  struct net_device *hsr);

> +

> +	/*

> +	 * MRP integration

> +	 */

> +	int	(*port_mrp_add)(struct dsa_switch *ds, int port,

> +				const struct switchdev_obj_mrp *mrp);

> +	int	(*port_mrp_del)(struct dsa_switch *ds, int port,

> +				const struct switchdev_obj_mrp *mrp);

> +	int	(*port_mrp_add_ring_role)(struct dsa_switch *ds, int port,

> +					  const struct switchdev_obj_ring_role_mrp *mrp);

> +	int	(*port_mrp_del_ring_role)(struct dsa_switch *ds, int port,

> +					  const struct switchdev_obj_ring_role_mrp *mrp);

>  };

>  

>  #define DSA_DEVLINK_PARAM_DRIVER(_id, _name, _type, _cmodes)		\

> diff --git a/net/dsa/dsa_priv.h b/net/dsa/dsa_priv.h

> index e9d1e76c42ba..2eeaa42f2e08 100644

> --- a/net/dsa/dsa_priv.h

> +++ b/net/dsa/dsa_priv.h

> @@ -31,6 +31,10 @@ enum {

>  	DSA_NOTIFIER_VLAN_DEL,

>  	DSA_NOTIFIER_MTU,

>  	DSA_NOTIFIER_TAG_PROTO,

> +	DSA_NOTIFIER_MRP_ADD,

> +	DSA_NOTIFIER_MRP_DEL,

> +	DSA_NOTIFIER_MRP_ADD_RING_ROLE,

> +	DSA_NOTIFIER_MRP_DEL_RING_ROLE,

>  };

>  

>  /* DSA_NOTIFIER_AGEING_TIME */

> @@ -91,6 +95,20 @@ struct dsa_notifier_tag_proto_info {

>  	const struct dsa_device_ops *tag_ops;

>  };

>  

> +/* DSA_NOTIFIER_MRP_* */

> +struct dsa_notifier_mrp_info {

> +	const struct switchdev_obj_mrp *mrp;

> +	int sw_index;

> +	int port;

> +};

> +

> +/* DSA_NOTIFIER_MRP_* */

> +struct dsa_notifier_mrp_ring_role_info {

> +	const struct switchdev_obj_ring_role_mrp *mrp;

> +	int sw_index;

> +	int port;

> +};

> +

>  struct dsa_switchdev_event_work {

>  	struct dsa_switch *ds;

>  	int port;

> @@ -198,6 +216,14 @@ int dsa_port_vlan_add(struct dsa_port *dp,

>  		      struct netlink_ext_ack *extack);

>  int dsa_port_vlan_del(struct dsa_port *dp,

>  		      const struct switchdev_obj_port_vlan *vlan);

> +int dsa_port_mrp_add(const struct dsa_port *dp,

> +		     const struct switchdev_obj_mrp *mrp);

> +int dsa_port_mrp_del(const struct dsa_port *dp,

> +		     const struct switchdev_obj_mrp *mrp);

> +int dsa_port_mrp_add_ring_role(const struct dsa_port *dp,

> +			       const struct switchdev_obj_ring_role_mrp *mrp);

> +int dsa_port_mrp_del_ring_role(const struct dsa_port *dp,

> +			       const struct switchdev_obj_ring_role_mrp *mrp);

>  int dsa_port_link_register_of(struct dsa_port *dp);

>  void dsa_port_link_unregister_of(struct dsa_port *dp);

>  int dsa_port_hsr_join(struct dsa_port *dp, struct net_device *hsr);

> diff --git a/net/dsa/port.c b/net/dsa/port.c

> index 14a1d0d77657..c9c6d7ab3f47 100644

> --- a/net/dsa/port.c

> +++ b/net/dsa/port.c

> @@ -564,6 +564,54 @@ int dsa_port_vlan_del(struct dsa_port *dp,

>  	return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_DEL, &info);

>  }

>  

> +int dsa_port_mrp_add(const struct dsa_port *dp,

> +		     const struct switchdev_obj_mrp *mrp)

> +{

> +	struct dsa_notifier_mrp_info info = {

> +		.sw_index = dp->ds->index,

> +		.port = dp->index,

> +		.mrp = mrp,

> +	};

> +

> +	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_ADD, &info);

> +}

> +

> +int dsa_port_mrp_del(const struct dsa_port *dp,

> +		     const struct switchdev_obj_mrp *mrp)

> +{

> +	struct dsa_notifier_mrp_info info = {

> +		.sw_index = dp->ds->index,

> +		.port = dp->index,

> +		.mrp = mrp,

> +	};

> +

> +	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_DEL, &info);

> +}

> +

> +int dsa_port_mrp_add_ring_role(const struct dsa_port *dp,

> +			       const struct switchdev_obj_ring_role_mrp *mrp)

> +{

> +	struct dsa_notifier_mrp_ring_role_info info = {

> +		.sw_index = dp->ds->index,

> +		.port = dp->index,

> +		.mrp = mrp,

> +	};

> +

> +	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_ADD_RING_ROLE, &info);

> +}

> +

> +int dsa_port_mrp_del_ring_role(const struct dsa_port *dp,

> +			       const struct switchdev_obj_ring_role_mrp *mrp)

> +{

> +	struct dsa_notifier_mrp_ring_role_info info = {

> +		.sw_index = dp->ds->index,

> +		.port = dp->index,

> +		.mrp = mrp,

> +	};

> +

> +	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_DEL_RING_ROLE, &info);

> +}

> +

>  void dsa_port_set_tag_protocol(struct dsa_port *cpu_dp,

>  			       const struct dsa_device_ops *tag_ops)

>  {

> diff --git a/net/dsa/slave.c b/net/dsa/slave.c

> index 5ecb43a1b6e0..491e3761b5f4 100644

> --- a/net/dsa/slave.c

> +++ b/net/dsa/slave.c

> @@ -404,6 +404,17 @@ static int dsa_slave_port_obj_add(struct net_device *dev,

>  	case SWITCHDEV_OBJ_ID_PORT_VLAN:

>  		err = dsa_slave_vlan_add(dev, obj, extack);

>  		break;

> +	case SWITCHDEV_OBJ_ID_MRP:

> +		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))

> +			return -EOPNOTSUPP;

> +		err = dsa_port_mrp_add(dp, SWITCHDEV_OBJ_MRP(obj));

> +		break;

> +	case SWITCHDEV_OBJ_ID_RING_ROLE_MRP:

> +		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))

> +			return -EOPNOTSUPP;

> +		err = dsa_port_mrp_add_ring_role(dp,

> +						 SWITCHDEV_OBJ_RING_ROLE_MRP(obj));

> +		break;

>  	default:

>  		err = -EOPNOTSUPP;

>  		break;

> @@ -461,6 +472,17 @@ static int dsa_slave_port_obj_del(struct net_device *dev,

>  	case SWITCHDEV_OBJ_ID_PORT_VLAN:

>  		err = dsa_slave_vlan_del(dev, obj);

>  		break;

> +	case SWITCHDEV_OBJ_ID_MRP:

> +		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))

> +			return -EOPNOTSUPP;

> +		err = dsa_port_mrp_del(dp, SWITCHDEV_OBJ_MRP(obj));

> +		break;

> +	case SWITCHDEV_OBJ_ID_RING_ROLE_MRP:

> +		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))

> +			return -EOPNOTSUPP;

> +		err = dsa_port_mrp_del_ring_role(dp,

> +						 SWITCHDEV_OBJ_RING_ROLE_MRP(obj));

> +		break;

>  	default:

>  		err = -EOPNOTSUPP;

>  		break;

> diff --git a/net/dsa/switch.c b/net/dsa/switch.c

> index db2a9b221988..4b5da89dc27a 100644

> --- a/net/dsa/switch.c

> +++ b/net/dsa/switch.c

> @@ -372,6 +372,99 @@ static int dsa_switch_change_tag_proto(struct dsa_switch *ds,

>  	return 0;

>  }

>  

> +static bool dsa_switch_mrp_match(struct dsa_switch *ds, int port,

> +				 struct dsa_notifier_mrp_info *info)

> +{

> +	if (ds->index == info->sw_index && port == info->port)

> +		return true;

> +

> +	if (dsa_is_dsa_port(ds, port))

> +		return true;

> +

> +	return false;

> +}

> +

> +static int dsa_switch_mrp_add(struct dsa_switch *ds,

> +			      struct dsa_notifier_mrp_info *info)

> +{

> +	int err = 0;

> +	int port;

> +

> +	if (!ds->ops->port_mrp_add)

> +		return -EOPNOTSUPP;

> +

> +	for (port = 0; port < ds->num_ports; port++) {

> +		if (dsa_switch_mrp_match(ds, port, info)) {

> +			err = ds->ops->port_mrp_add(ds, port, info->mrp);

> +			if (err)

> +				break;

> +		}

> +	}

> +

> +	return err;

> +}

> +

> +static int dsa_switch_mrp_del(struct dsa_switch *ds,

> +			      struct dsa_notifier_mrp_info *info)

> +{

> +	if (!ds->ops->port_mrp_del)

> +		return -EOPNOTSUPP;

> +

> +	if (ds->index == info->sw_index)

> +		return ds->ops->port_mrp_del(ds, info->port, info->mrp);

> +

> +	return 0;

> +}

> +


Why not use dsa_switch_mrp_match here too? (question valid for the ring
role below too)

> +static bool

> +dsa_switch_mrp_ring_role_match(struct dsa_switch *ds, int port,

> +			       struct dsa_notifier_mrp_ring_role_info *info)

> +{

> +	if (ds->index == info->sw_index && port == info->port)

> +		return true;

> +

> +	if (dsa_is_dsa_port(ds, port))

> +		return true;

> +

> +	return false;

> +}

> +

> +static int

> +dsa_switch_mrp_add_ring_role(struct dsa_switch *ds,

> +			     struct dsa_notifier_mrp_ring_role_info *info)

> +{

> +	int err = 0;

> +	int port;

> +

> +	if (!ds->ops->port_mrp_add)

> +		return -EOPNOTSUPP;

> +

> +	for (port = 0; port < ds->num_ports; port++) {

> +		if (dsa_switch_mrp_ring_role_match(ds, port, info)) {

> +			err = ds->ops->port_mrp_add_ring_role(ds, port,

> +							      info->mrp);

> +			if (err)

> +				break;

> +		}

> +	}

> +

> +	return err;

> +}

> +

> +static int

> +dsa_switch_mrp_del_ring_role(struct dsa_switch *ds,

> +			     struct dsa_notifier_mrp_ring_role_info *info)

> +{

> +	if (!ds->ops->port_mrp_del)

> +		return -EOPNOTSUPP;

> +

> +	if (ds->index == info->sw_index)

> +		return ds->ops->port_mrp_del_ring_role(ds, info->port,

> +						       info->mrp);

> +

> +	return 0;

> +}

> +

>  static int dsa_switch_event(struct notifier_block *nb,

>  			    unsigned long event, void *info)

>  {

> @@ -427,6 +520,18 @@ static int dsa_switch_event(struct notifier_block *nb,

>  	case DSA_NOTIFIER_TAG_PROTO:

>  		err = dsa_switch_change_tag_proto(ds, info);

>  		break;

> +	case DSA_NOTIFIER_MRP_ADD:

> +		err = dsa_switch_mrp_add(ds, info);

> +		break;

> +	case DSA_NOTIFIER_MRP_DEL:

> +		err = dsa_switch_mrp_del(ds, info);

> +		break;

> +	case DSA_NOTIFIER_MRP_ADD_RING_ROLE:

> +		err = dsa_switch_mrp_add_ring_role(ds, info);

> +		break;

> +	case DSA_NOTIFIER_MRP_DEL_RING_ROLE:

> +		err = dsa_switch_mrp_del_ring_role(ds, info);

> +		break;

>  	default:

>  		err = -EOPNOTSUPP;

>  		break;

> -- 

> 2.27.0

>
diff mbox series

Patch

diff --git a/include/net/dsa.h b/include/net/dsa.h
index 68f8159564a3..83a933e563fe 100644
--- a/include/net/dsa.h
+++ b/include/net/dsa.h
@@ -792,6 +792,18 @@  struct dsa_switch_ops {
 				 struct net_device *hsr);
 	int	(*port_hsr_leave)(struct dsa_switch *ds, int port,
 				  struct net_device *hsr);
+
+	/*
+	 * MRP integration
+	 */
+	int	(*port_mrp_add)(struct dsa_switch *ds, int port,
+				const struct switchdev_obj_mrp *mrp);
+	int	(*port_mrp_del)(struct dsa_switch *ds, int port,
+				const struct switchdev_obj_mrp *mrp);
+	int	(*port_mrp_add_ring_role)(struct dsa_switch *ds, int port,
+					  const struct switchdev_obj_ring_role_mrp *mrp);
+	int	(*port_mrp_del_ring_role)(struct dsa_switch *ds, int port,
+					  const struct switchdev_obj_ring_role_mrp *mrp);
 };
 
 #define DSA_DEVLINK_PARAM_DRIVER(_id, _name, _type, _cmodes)		\
diff --git a/net/dsa/dsa_priv.h b/net/dsa/dsa_priv.h
index e9d1e76c42ba..2eeaa42f2e08 100644
--- a/net/dsa/dsa_priv.h
+++ b/net/dsa/dsa_priv.h
@@ -31,6 +31,10 @@  enum {
 	DSA_NOTIFIER_VLAN_DEL,
 	DSA_NOTIFIER_MTU,
 	DSA_NOTIFIER_TAG_PROTO,
+	DSA_NOTIFIER_MRP_ADD,
+	DSA_NOTIFIER_MRP_DEL,
+	DSA_NOTIFIER_MRP_ADD_RING_ROLE,
+	DSA_NOTIFIER_MRP_DEL_RING_ROLE,
 };
 
 /* DSA_NOTIFIER_AGEING_TIME */
@@ -91,6 +95,20 @@  struct dsa_notifier_tag_proto_info {
 	const struct dsa_device_ops *tag_ops;
 };
 
+/* DSA_NOTIFIER_MRP_* */
+struct dsa_notifier_mrp_info {
+	const struct switchdev_obj_mrp *mrp;
+	int sw_index;
+	int port;
+};
+
+/* DSA_NOTIFIER_MRP_* */
+struct dsa_notifier_mrp_ring_role_info {
+	const struct switchdev_obj_ring_role_mrp *mrp;
+	int sw_index;
+	int port;
+};
+
 struct dsa_switchdev_event_work {
 	struct dsa_switch *ds;
 	int port;
@@ -198,6 +216,14 @@  int dsa_port_vlan_add(struct dsa_port *dp,
 		      struct netlink_ext_ack *extack);
 int dsa_port_vlan_del(struct dsa_port *dp,
 		      const struct switchdev_obj_port_vlan *vlan);
+int dsa_port_mrp_add(const struct dsa_port *dp,
+		     const struct switchdev_obj_mrp *mrp);
+int dsa_port_mrp_del(const struct dsa_port *dp,
+		     const struct switchdev_obj_mrp *mrp);
+int dsa_port_mrp_add_ring_role(const struct dsa_port *dp,
+			       const struct switchdev_obj_ring_role_mrp *mrp);
+int dsa_port_mrp_del_ring_role(const struct dsa_port *dp,
+			       const struct switchdev_obj_ring_role_mrp *mrp);
 int dsa_port_link_register_of(struct dsa_port *dp);
 void dsa_port_link_unregister_of(struct dsa_port *dp);
 int dsa_port_hsr_join(struct dsa_port *dp, struct net_device *hsr);
diff --git a/net/dsa/port.c b/net/dsa/port.c
index 14a1d0d77657..c9c6d7ab3f47 100644
--- a/net/dsa/port.c
+++ b/net/dsa/port.c
@@ -564,6 +564,54 @@  int dsa_port_vlan_del(struct dsa_port *dp,
 	return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_DEL, &info);
 }
 
+int dsa_port_mrp_add(const struct dsa_port *dp,
+		     const struct switchdev_obj_mrp *mrp)
+{
+	struct dsa_notifier_mrp_info info = {
+		.sw_index = dp->ds->index,
+		.port = dp->index,
+		.mrp = mrp,
+	};
+
+	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_ADD, &info);
+}
+
+int dsa_port_mrp_del(const struct dsa_port *dp,
+		     const struct switchdev_obj_mrp *mrp)
+{
+	struct dsa_notifier_mrp_info info = {
+		.sw_index = dp->ds->index,
+		.port = dp->index,
+		.mrp = mrp,
+	};
+
+	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_DEL, &info);
+}
+
+int dsa_port_mrp_add_ring_role(const struct dsa_port *dp,
+			       const struct switchdev_obj_ring_role_mrp *mrp)
+{
+	struct dsa_notifier_mrp_ring_role_info info = {
+		.sw_index = dp->ds->index,
+		.port = dp->index,
+		.mrp = mrp,
+	};
+
+	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_ADD_RING_ROLE, &info);
+}
+
+int dsa_port_mrp_del_ring_role(const struct dsa_port *dp,
+			       const struct switchdev_obj_ring_role_mrp *mrp)
+{
+	struct dsa_notifier_mrp_ring_role_info info = {
+		.sw_index = dp->ds->index,
+		.port = dp->index,
+		.mrp = mrp,
+	};
+
+	return dsa_port_notify(dp, DSA_NOTIFIER_MRP_DEL_RING_ROLE, &info);
+}
+
 void dsa_port_set_tag_protocol(struct dsa_port *cpu_dp,
 			       const struct dsa_device_ops *tag_ops)
 {
diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index 5ecb43a1b6e0..491e3761b5f4 100644
--- a/net/dsa/slave.c
+++ b/net/dsa/slave.c
@@ -404,6 +404,17 @@  static int dsa_slave_port_obj_add(struct net_device *dev,
 	case SWITCHDEV_OBJ_ID_PORT_VLAN:
 		err = dsa_slave_vlan_add(dev, obj, extack);
 		break;
+	case SWITCHDEV_OBJ_ID_MRP:
+		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))
+			return -EOPNOTSUPP;
+		err = dsa_port_mrp_add(dp, SWITCHDEV_OBJ_MRP(obj));
+		break;
+	case SWITCHDEV_OBJ_ID_RING_ROLE_MRP:
+		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))
+			return -EOPNOTSUPP;
+		err = dsa_port_mrp_add_ring_role(dp,
+						 SWITCHDEV_OBJ_RING_ROLE_MRP(obj));
+		break;
 	default:
 		err = -EOPNOTSUPP;
 		break;
@@ -461,6 +472,17 @@  static int dsa_slave_port_obj_del(struct net_device *dev,
 	case SWITCHDEV_OBJ_ID_PORT_VLAN:
 		err = dsa_slave_vlan_del(dev, obj);
 		break;
+	case SWITCHDEV_OBJ_ID_MRP:
+		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))
+			return -EOPNOTSUPP;
+		err = dsa_port_mrp_del(dp, SWITCHDEV_OBJ_MRP(obj));
+		break;
+	case SWITCHDEV_OBJ_ID_RING_ROLE_MRP:
+		if (!dsa_port_offloads_netdev(dp, obj->orig_dev))
+			return -EOPNOTSUPP;
+		err = dsa_port_mrp_del_ring_role(dp,
+						 SWITCHDEV_OBJ_RING_ROLE_MRP(obj));
+		break;
 	default:
 		err = -EOPNOTSUPP;
 		break;
diff --git a/net/dsa/switch.c b/net/dsa/switch.c
index db2a9b221988..4b5da89dc27a 100644
--- a/net/dsa/switch.c
+++ b/net/dsa/switch.c
@@ -372,6 +372,99 @@  static int dsa_switch_change_tag_proto(struct dsa_switch *ds,
 	return 0;
 }
 
+static bool dsa_switch_mrp_match(struct dsa_switch *ds, int port,
+				 struct dsa_notifier_mrp_info *info)
+{
+	if (ds->index == info->sw_index && port == info->port)
+		return true;
+
+	if (dsa_is_dsa_port(ds, port))
+		return true;
+
+	return false;
+}
+
+static int dsa_switch_mrp_add(struct dsa_switch *ds,
+			      struct dsa_notifier_mrp_info *info)
+{
+	int err = 0;
+	int port;
+
+	if (!ds->ops->port_mrp_add)
+		return -EOPNOTSUPP;
+
+	for (port = 0; port < ds->num_ports; port++) {
+		if (dsa_switch_mrp_match(ds, port, info)) {
+			err = ds->ops->port_mrp_add(ds, port, info->mrp);
+			if (err)
+				break;
+		}
+	}
+
+	return err;
+}
+
+static int dsa_switch_mrp_del(struct dsa_switch *ds,
+			      struct dsa_notifier_mrp_info *info)
+{
+	if (!ds->ops->port_mrp_del)
+		return -EOPNOTSUPP;
+
+	if (ds->index == info->sw_index)
+		return ds->ops->port_mrp_del(ds, info->port, info->mrp);
+
+	return 0;
+}
+
+static bool
+dsa_switch_mrp_ring_role_match(struct dsa_switch *ds, int port,
+			       struct dsa_notifier_mrp_ring_role_info *info)
+{
+	if (ds->index == info->sw_index && port == info->port)
+		return true;
+
+	if (dsa_is_dsa_port(ds, port))
+		return true;
+
+	return false;
+}
+
+static int
+dsa_switch_mrp_add_ring_role(struct dsa_switch *ds,
+			     struct dsa_notifier_mrp_ring_role_info *info)
+{
+	int err = 0;
+	int port;
+
+	if (!ds->ops->port_mrp_add)
+		return -EOPNOTSUPP;
+
+	for (port = 0; port < ds->num_ports; port++) {
+		if (dsa_switch_mrp_ring_role_match(ds, port, info)) {
+			err = ds->ops->port_mrp_add_ring_role(ds, port,
+							      info->mrp);
+			if (err)
+				break;
+		}
+	}
+
+	return err;
+}
+
+static int
+dsa_switch_mrp_del_ring_role(struct dsa_switch *ds,
+			     struct dsa_notifier_mrp_ring_role_info *info)
+{
+	if (!ds->ops->port_mrp_del)
+		return -EOPNOTSUPP;
+
+	if (ds->index == info->sw_index)
+		return ds->ops->port_mrp_del_ring_role(ds, info->port,
+						       info->mrp);
+
+	return 0;
+}
+
 static int dsa_switch_event(struct notifier_block *nb,
 			    unsigned long event, void *info)
 {
@@ -427,6 +520,18 @@  static int dsa_switch_event(struct notifier_block *nb,
 	case DSA_NOTIFIER_TAG_PROTO:
 		err = dsa_switch_change_tag_proto(ds, info);
 		break;
+	case DSA_NOTIFIER_MRP_ADD:
+		err = dsa_switch_mrp_add(ds, info);
+		break;
+	case DSA_NOTIFIER_MRP_DEL:
+		err = dsa_switch_mrp_del(ds, info);
+		break;
+	case DSA_NOTIFIER_MRP_ADD_RING_ROLE:
+		err = dsa_switch_mrp_add_ring_role(ds, info);
+		break;
+	case DSA_NOTIFIER_MRP_DEL_RING_ROLE:
+		err = dsa_switch_mrp_del_ring_role(ds, info);
+		break;
 	default:
 		err = -EOPNOTSUPP;
 		break;