diff options
| author | Sang-Heon Jeon <ekffu200098@gmail.com> | 2026-07-28 03:47:38 +0900 |
|---|---|---|
| committer | Jonathan Cameron <jonathan.cameron@oss.qualcomm.com> | 2026-08-03 22:32:32 +0100 |
| commit | 55b75622829779223b9e32aa9600a8651d3e2df4 (patch) | |
| tree | 22a2702a5f3e9dad9a8cdc4fe763b8127f7b9495 | |
| parent | 512d321b9397092d779120a590682c8f632a9e3a (diff) | |
| download | linux-55b75622829779223b9e32aa9600a8651d3e2df4.tar.gz linux-55b75622829779223b9e32aa9600a8651d3e2df4.zip | |
iio: light: isl29028: return zero in write_raw() on success
isl29028_write_raw() returns the value of pm_runtime_put_autosuspend(),
which is 1 if the device is already runtime suspended.
In that case write() on the sysfs attribute returns 1 instead of the
number of bytes written. Make isl29028_write_raw() always return zero
on success.
Fixes: 2db5054ac28d ("staging: iio: isl29028: add runtime power management support")
Signed-off-by: Sang-Heon Jeon <ekffu200098@gmail.com>
Reviewed-by: Brian Masney <bmasney@redhat.com>
Signed-off-by: Jonathan Cameron <jonathan.cameron@oss.qualcomm.com>
| -rw-r--r-- | drivers/iio/light/isl29028.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/light/isl29028.c b/drivers/iio/light/isl29028.c index b88e7c4eae3e..6d5dd17e7a7d 100644 --- a/drivers/iio/light/isl29028.c +++ b/drivers/iio/light/isl29028.c @@ -409,7 +409,7 @@ static int isl29028_write_raw(struct iio_dev *indio_dev, if (ret < 0) return ret; - return ret; + return 0; } static int isl29028_read_raw(struct iio_dev *indio_dev, |