Merge branch 'task/name_param_cb' into 'master'

esp_param_name: Give an option to handle name param in application callback

See merge request app-frameworks/esp-rainmaker!286

Author: shahpiyushv

Diff for: components/esp_rainmaker/Kconfig.projbuild

@@ -79,6 +79,14 @@ menu "ESP RainMaker Config"
             can take appropriate action w.r.t user permissions. It is recommended to enable this option
             for security reasons.
 
+    config RMAKER_NAME_PARAM_CB
+        bool "Call device callback for Name param"
+        default n
+        help
+            By default, the "Name" parameter (esp.param.name) changes are handled internally. If Applications
+            want to handle this themselves, this config option can be enabled. Please ensure that you update
+            and report the name parameter in your callback so that it reflects correctly everywhere.
+            If no device callback is registered, the name paramater will be handled internally.
 
     config ESP_RMAKER_LOCAL_CTRL_ENABLE
         bool "ESP RainMaker Local Control"

Diff for: components/esp_rainmaker/src/core/esp_rmaker_param.c

@@ -357,7 +357,19 @@ static esp_err_t esp_rmaker_device_set_params(_esp_rmaker_device_t *device, jpar
              * of calling the registered callback.
              */
             if (param->type && (strcmp(param->type, ESP_RMAKER_PARAM_NAME) == 0)) {
+#ifdef CONFIG_RMAKER_NAME_PARAM_CB
+                if (device->write_cb) {
+                    esp_rmaker_write_ctx_t ctx = {
+                        .src = src,
+                    };
+                    device->write_cb((esp_rmaker_device_t *)device, (esp_rmaker_param_t *)param,
+                                new_val, device->priv_data, &ctx);
+                } else {
+                    esp_rmaker_param_update_and_report((esp_rmaker_param_t *)param, new_val);
+                }
+#else
                 esp_rmaker_param_update_and_report((esp_rmaker_param_t *)param, new_val);
+#endif
             } else if (device->write_cb) {
                 esp_rmaker_write_ctx_t ctx = {
                     .src = src,