diff mbox series

[net-next,v2,2/7] net: dsa: Make use of devlink port flavour unused

Message ID 20200926210632.3888886-3-andrew@lunn.ch
State Superseded
Headers show
Series None | expand

Commit Message

Andrew Lunn Sept. 26, 2020, 9:06 p.m. UTC 
If a port is unused, still create a devlink port for it, but set the
flavour to unused. This allows us to attach devlink regions to the
port, etc.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
 net/dsa/dsa2.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Vladimir Oltean Sept. 26, 2020, 10:51 p.m. UTC | #1 
On Sat, Sep 26, 2020 at 11:06:27PM +0200, Andrew Lunn wrote:
> If a port is unused, still create a devlink port for it, but set the

> flavour to unused. This allows us to attach devlink regions to the

> port, etc.

>

> Signed-off-by: Andrew Lunn <andrew@lunn.ch>

> ---


Reviewed-by: Vladimir Oltean <olteanv@gmail.com>

Tested-by: Vladimir Oltean <olteanv@gmail.com>
diff mbox series

Patch

diff --git a/net/dsa/dsa2.c b/net/dsa/dsa2.c
index 3cf67f5fe54a..2c149fb36928 100644
--- a/net/dsa/dsa2.c
+++ b/net/dsa/dsa2.c
@@ -272,6 +272,15 @@  static int dsa_port_setup(struct dsa_port *dp)
 
 	switch (dp->type) {
 	case DSA_PORT_TYPE_UNUSED:
+		memset(dlp, 0, sizeof(*dlp));
+		attrs.flavour = DEVLINK_PORT_FLAVOUR_UNUSED;
+		devlink_port_attrs_set(dlp, &attrs);
+		err = devlink_port_register(dl, dlp, dp->index);
+		if (err)
+			break;
+
+		devlink_port_registered = true;
+
 		dsa_port_disable(dp);
 		break;
 	case DSA_PORT_TYPE_CPU:
@@ -355,6 +364,7 @@  static void dsa_port_teardown(struct dsa_port *dp)
 
 	switch (dp->type) {
 	case DSA_PORT_TYPE_UNUSED:
+		devlink_port_unregister(dlp);
 		break;
 	case DSA_PORT_TYPE_CPU:
 		dsa_port_disable(dp);