iio: admv1013: replace redundant ternary operator with just len

ColinIanKing · jic23 · commit 6cdb4009c201 · 2025-05-21T14:21:34.000+01:00
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>
diff --git a/drivers/iio/frequency/admv1013.c b/drivers/iio/frequency/admv1013.c
@@ -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)

Original file line number	Diff line number	Diff line change
`@@ -319,7 +319,7 @@ static ssize_t admv1013_write(struct iio_dev *indio_dev,`
`319`	`319`	`return -EINVAL;`
`320`	`320`	`}`
`321`	`321`
`322`		`- return ret ? ret : len;`
	`322`	`+ return len;`
`323`	`323`	`}`
`324`	`324`
`325`	`325`	`static int admv1013_update_quad_filters(struct admv1013_state *st)`