By using the elvis operator here, we can get rid of the if statement and simplify the code. Signed-off-by: Vasileios Amoiridis <vassilisamir@xxxxxxxxx> --- drivers/iio/light/acpi-als.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) 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); -- 2.25.1