The patch adds devlink interfaces for drivers to publish/unpublish the
devlink port parameters.

Signed-off-by: Sudarsana Reddy Kalluru <skall...@marvell.com>
Signed-off-by: Ariel Elior <ael...@marvell.com>
---
 include/net/devlink.h |  2 ++
 net/core/devlink.c    | 42 ++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 44 insertions(+)

diff --git a/include/net/devlink.h b/include/net/devlink.h
index 6c51e86..2e2d7fc 100644
--- a/include/net/devlink.h
+++ b/include/net/devlink.h
@@ -651,6 +651,8 @@ int devlink_port_params_register(struct devlink_port 
*devlink_port,
 void devlink_port_params_unregister(struct devlink_port *devlink_port,
                                    const struct devlink_param *params,
                                    size_t params_count);
+void devlink_port_params_publish(struct devlink_port *devlink_port);
+void devlink_port_params_unpublish(struct devlink_port *ddevlink_port);
 int devlink_param_driverinit_value_get(struct devlink *devlink, u32 param_id,
                                       union devlink_param_value *init_val);
 int devlink_param_driverinit_value_set(struct devlink *devlink, u32 param_id,
diff --git a/net/core/devlink.c b/net/core/devlink.c
index 4baf716..c06c23f 100644
--- a/net/core/devlink.c
+++ b/net/core/devlink.c
@@ -6378,6 +6378,48 @@ void devlink_port_params_unregister(struct devlink_port 
*devlink_port,
 }
 EXPORT_SYMBOL_GPL(devlink_port_params_unregister);
 
+/**
+ *     devlink_port_params_publish - publish port configuration parameters
+ *
+ *     @devlink_port: devlink port
+ *
+ *     Publish previously registered port configuration parameters.
+ */
+void devlink_port_params_publish(struct devlink_port *devlink_port)
+{
+       struct devlink_param_item *param_item;
+
+       list_for_each_entry(param_item, &devlink_port->param_list, list) {
+               if (param_item->published)
+                       continue;
+               param_item->published = true;
+               devlink_param_notify(devlink_port->devlink, devlink_port->index,
+                                    param_item, DEVLINK_CMD_PORT_PARAM_NEW);
+       }
+}
+EXPORT_SYMBOL_GPL(devlink_port_params_publish);
+
+/**
+ *     devlink_port_params_unpublish - unpublish port configuration parameters
+ *
+ *     @devlink_port: devlink port
+ *
+ *     Unpublish previously registered port configuration parameters.
+ */
+void devlink_port_params_unpublish(struct devlink_port *devlink_port)
+{
+       struct devlink_param_item *param_item;
+
+       list_for_each_entry(param_item, &devlink_port->param_list, list) {
+               if (!param_item->published)
+                       continue;
+               param_item->published = false;
+               devlink_param_notify(devlink_port->devlink, devlink_port->index,
+                                    param_item, DEVLINK_CMD_PORT_PARAM_DEL);
+       }
+}
+EXPORT_SYMBOL_GPL(devlink_port_params_unpublish);
+
 static int
 __devlink_param_driverinit_value_get(struct list_head *param_list, u32 
param_id,
                                     union devlink_param_value *init_val)
-- 
1.8.3.1

Reply via email to