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.

Reviewed-by: Vladimir Oltean <olte...@gmail.com>
Tested-by: Vladimir Oltean <olte...@gmail.com>
Signed-off-by: Andrew Lunn <and...@lunn.ch>
---
 net/dsa/dsa2.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

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);
-- 
2.28.0

Reply via email to