Re: [PATCH 1/9] iio: light: isl29028: return zero in write_raw() on success

From: Brian Masney

Date: Tue Jul 28 2026 - 16:02:31 EST


On Tue, Jul 28, 2026 at 03:47:38AM +0900, Sang-Heon Jeon wrote:
> 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@xxxxxxxxx>

Reviewed-by: Brian Masney <bmasney@xxxxxxxxxx>