hwmon: (pmbus/core) Declare regulator notification function as void
authorGuenter Roeck <linux@roeck-us.net>
Sun, 26 Jan 2025 04:40:46 +0000 (20:40 -0800)
committerGuenter Roeck <linux@roeck-us.net>
Sun, 2 Mar 2025 16:48:59 +0000 (08:48 -0800)
The regulator notification function never returns an error.
Declare it as void.

While at it, fix its indentation.

No functional change.

Reviewed-by: Tzung-Bi Shih <tzungbi@kernel.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/pmbus/pmbus_core.c

index a19b32eb9a047012c1d5cece56b45a57aa200cfb..d9d5efc447154a592db8271b9e4c88264a7ee7aa 100644 (file)
@@ -3312,17 +3312,16 @@ static int pmbus_regulator_register(struct pmbus_data *data)
        return 0;
 }
 
-static int pmbus_regulator_notify(struct pmbus_data *data, int page, int event)
+static void pmbus_regulator_notify(struct pmbus_data *data, int page, int event)
 {
-               int j;
+       int j;
 
-               for (j = 0; j < data->info->num_regulators; j++) {
-                       if (page == rdev_get_id(data->rdevs[j])) {
-                               regulator_notifier_call_chain(data->rdevs[j], event, NULL);
-                               break;
-                       }
+       for (j = 0; j < data->info->num_regulators; j++) {
+               if (page == rdev_get_id(data->rdevs[j])) {
+                       regulator_notifier_call_chain(data->rdevs[j], event, NULL);
+                       break;
                }
-               return 0;
+       }
 }
 #else
 static int pmbus_regulator_register(struct pmbus_data *data)
@@ -3330,9 +3329,8 @@ static int pmbus_regulator_register(struct pmbus_data *data)
        return 0;
 }
 
-static int pmbus_regulator_notify(struct pmbus_data *data, int page, int event)
+static void pmbus_regulator_notify(struct pmbus_data *data, int page, int event)
 {
-               return 0;
 }
 #endif