diff options
| author | Kazuhiro Abe <fj1078ii@aa.jp.fujitsu.com> | 2025-01-15 08:35:32 +0100 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2025-01-15 15:21:19 +0100 |
| commit | 8d6bf2e1055fa2cca4bf233f46d4d1e2086cc5ff (patch) | |
| tree | 8592740d28192c90d1acb4a371beaa7749c99d44 /drivers | |
| parent | hwmon: (tmp513) Fix division of negative numbers (diff) | |
| download | linux-8d6bf2e1055fa2cca4bf233f46d4d1e2086cc5ff.tar.xz linux-8d6bf2e1055fa2cca4bf233f46d4d1e2086cc5ff.zip | |
hwmon: (acpi_power_meter) Fix a check for the return value of read_domain_devices().
After commit fabb1f813ec0 ("hwmon: (acpi_power_meter) Fix fail to load
module on platform without _PMD method"),
the acpi_power_meter driver fails to load if the platform has _PMD method.
To address this, add a check for successful read_domain_devices().
Tested on Nvidia Grace machine.
Fixes: fabb1f813ec0 ("hwmon: (acpi_power_meter) Fix fail to load module on platform without _PMD method")
Signed-off-by: Kazuhiro Abe <fj1078ii@aa.jp.fujitsu.com>
Link: https://lore.kernel.org/r/20250115073532.3211000-1-fj1078ii@aa.jp.fujitsu.com
[groeck: Dropped unnecessary () from expression]
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/hwmon/acpi_power_meter.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/acpi_power_meter.c b/drivers/hwmon/acpi_power_meter.c index 2f1c9d97ad21..3db1b9d09c32 100644 --- a/drivers/hwmon/acpi_power_meter.c +++ b/drivers/hwmon/acpi_power_meter.c @@ -682,7 +682,7 @@ static int setup_attrs(struct acpi_power_meter_resource *resource) /* _PMD method is optional. */ res = read_domain_devices(resource); - if (res != -ENODEV) + if (res && res != -ENODEV) return res; if (resource->caps.flags & POWER_METER_CAN_MEASURE) { |