Message ID | 20240922160304.520613-2-vassilisamir@gmail.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | iio: Remove assignment to pf->timestamp by using elvis | expand |
diff --git a/drivers/iio/light/acpi-als.c b/drivers/iio/light/acpi-als.c index 2d91caf24dd0..cea3499c3af3 100644 --- a/drivers/iio/light/acpi-als.c +++ b/drivers/iio/light/acpi-als.c @@ -171,10 +171,8 @@ static irqreturn_t acpi_als_trigger_handler(int irq, void *p) * * If the timestamp was actually 0, the timestamp is set one more time. */ - if (!pf->timestamp) - pf->timestamp = iio_get_time_ns(indio_dev); - - iio_push_to_buffers_with_timestamp(indio_dev, buffer, pf->timestamp); + iio_push_to_buffers_with_timestamp(indio_dev, buffer, + pf->timestamp ?: iio_get_time_ns(indio_dev)); out: mutex_unlock(&als->lock); iio_trigger_notify_done(indio_dev->trig);
By using the elvis operator here, we can get rid of the if statement and simplify the code. Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com> --- drivers/iio/light/acpi-als.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)