diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2026-08-21 07:49:16 -0700 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2026-08-23 07:39:34 -0700 |
| commit | 83a16909eb64360fd0659f7970665272cdbe65a1 (patch) | |
| tree | 6f2d9c1a7d5e505a5d168898ae2650c2e60e8048 /scripts/objdiff | |
| parent | 2c2739b719d32b99f72f60d26addd55258011bb2 (diff) | |
| download | linux-next-83a16909eb64360fd0659f7970665272cdbe65a1.tar.gz linux-next-83a16909eb64360fd0659f7970665272cdbe65a1.zip | |
hwmon: (sht4x) Fix return value from heater_enable_store()
Sashiko reports:
The return value in heater_enable_store() causes an unexpected write
failure in user-space.
When the heater is successfully enabled, the function returns 0
instead of count:
drivers/hwmon/sht4x.c:heater_enable_store() {
...
data->heating_complete = jiffies + msecs_to_jiffies(heating_time_bound);
data->data_pending = true;
return 0;
}
Returning 0 signals to VFS that no bytes were processed. Standard
user-space tools will retry the write with the remaining bytes. On the
retry, time_before(jiffies, data->heating_complete) evaluates to true,
and the function immediately fails with -EBUSY.
Return count as expected to fix the problem.
Fixes: 0eed6fc3d2b9e ("hwmon: (sht4x): add heater support")
Cc: Antoni Pokusinski <apokusinski01@gmail.com>
Cc: Alessandro Zini <alessandro.zini@siemens.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Link: https://patch.msgid.link/20260821144916.2889031-2-linux@roeck-us.net
Diffstat (limited to 'scripts/objdiff')
0 files changed, 0 insertions, 0 deletions
