]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
iio: admv1013: replace redundant ternary operator with just len
authorColin Ian King <colin.i.king@gmail.com>
Wed, 7 May 2025 13:45:02 +0000 (14:45 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Wed, 21 May 2025 13:21:34 +0000 (14:21 +0100)
The variable ret is being assigned a return value and non-zero error
return paths are taken at all stages. At the end of the function ret
is always zero, so the ternary operator checking for zero ret is
redundant and can be replaced with just len instead.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: David Lechner <dlechner@baylibre.com>
Link: https://patch.msgid.link/20250507134502.254736-1-colin.i.king@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/frequency/admv1013.c

index 6f50884d7130637a9a74af7807c6d328def58589..d8e8d541990f874785d7d8ae2d555908d9c2c573 100644 (file)
@@ -319,7 +319,7 @@ static ssize_t admv1013_write(struct iio_dev *indio_dev,
                return -EINVAL;
        }
 
-       return ret ? ret : len;
+       return len;
 }
 
 static int admv1013_update_quad_filters(struct admv1013_state *st)