platform/mellanox: mlxreg-hotplug: Add hotplug hwmon uevent notification

Notify user when hotplug device signal is received in order to allow user
to handle such case, if it wishes to take some action on this matter.

Signed-off-by: Vadim Pasternak <vadimp@mellanox.com>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>

diff --git a/drivers/platform/mellanox/mlxreg-hotplug.c b/drivers/platform/mellanox/mlxreg-hotplug.c
index f363e0786ef4076c4271a80f012ed2171f4a6a71..b6d44550d98cf3fbe95a021b79b23dd1ad54bea2 100644 (file)
--- a/drivers/platform/mellanox/mlxreg-hotplug.c
+++ b/drivers/platform/mellanox/mlxreg-hotplug.c
@@ -102,6 +102,9 @@ static int mlxreg_hotplug_device_create(struct mlxreg_hotplug_priv_data *priv,
 {
        struct mlxreg_core_hotplug_platform_data *pdata;
 
+       /* Notify user by sending hwmon uevent. */
+       kobject_uevent(&priv->hwmon->kobj, KOBJ_CHANGE);
+
        /*
         * Return if adapter number is negative. It could be in case hotplug
         * event is not associated with hotplug device.
@@ -133,8 +136,13 @@ static int mlxreg_hotplug_device_create(struct mlxreg_hotplug_priv_data *priv,
        return 0;
 }
 
-static void mlxreg_hotplug_device_destroy(struct mlxreg_core_data *data)
+static void
+mlxreg_hotplug_device_destroy(struct mlxreg_hotplug_priv_data *priv,
+                             struct mlxreg_core_data *data)
 {
+       /* Notify user by sending hwmon uevent. */
+       kobject_uevent(&priv->hwmon->kobj, KOBJ_CHANGE);
+
        if (data->hpdev.client) {
                i2c_unregister_device(data->hpdev.client);
                data->hpdev.client = NULL;
@@ -277,14 +285,14 @@ mlxreg_hotplug_work_helper(struct mlxreg_hotplug_priv_data *priv,
                data = item->data + bit;
                if (regval & BIT(bit)) {
                        if (item->inversed)
-                               mlxreg_hotplug_device_destroy(data);
+                               mlxreg_hotplug_device_destroy(priv, data);
                        else
                                mlxreg_hotplug_device_create(priv, data);
                } else {
                        if (item->inversed)
                                mlxreg_hotplug_device_create(priv, data);
                        else
-                               mlxreg_hotplug_device_destroy(data);
+                               mlxreg_hotplug_device_destroy(priv, data);
                }
        }
 
@@ -351,7 +359,7 @@ mlxreg_hotplug_health_work_helper(struct mlxreg_hotplug_priv_data *priv,
                                 * in steady state. Disconnect associated
                                 * device, if it has been connected.
                                 */
-                               mlxreg_hotplug_device_destroy(data);
+                               mlxreg_hotplug_device_destroy(priv, data);
                                data->attached = false;
                                data->health_cntr = 0;
                        }
@@ -569,7 +577,7 @@ static void mlxreg_hotplug_unset_irq(struct mlxreg_hotplug_priv_data *priv)
                /* Remove all the attached devices in group. */
                count = item->count;
                for (j = 0; j < count; j++, data++)
-                       mlxreg_hotplug_device_destroy(data);
+                       mlxreg_hotplug_device_destroy(priv, data);
        }
 }
 
@@ -634,10 +642,6 @@ static int mlxreg_hotplug_probe(struct platform_device *pdev)
        disable_irq(priv->irq);
        spin_lock_init(&priv->lock);
        INIT_DELAYED_WORK(&priv->dwork_irq, mlxreg_hotplug_work_handler);
-       /* Perform initial interrupts setup. */
-       mlxreg_hotplug_set_irq(priv);
-
-       priv->after_probe = true;
        dev_set_drvdata(&pdev->dev, priv);
 
        err = mlxreg_hotplug_attr_init(priv);
@@ -655,6 +659,10 @@ static int mlxreg_hotplug_probe(struct platform_device *pdev)
                return PTR_ERR(priv->hwmon);
        }
 
+       /* Perform initial interrupts setup. */
+       mlxreg_hotplug_set_irq(priv);
+       priv->after_probe = true;
+
        return 0;
 }