summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngus Gardner <angusg778@gmail.com>2026-05-04 19:20:57 +1000
committerJonathan Cameron <jic23@kernel.org>2026-05-31 10:59:33 +0100
commit19f2c27251d86cf1272d475c52a190d01e9f996b (patch)
treefc85f1edf7f3fa72fd869e0d2ac3cf859fb6685c
parentb4e9ede52a6094b4a7fcf584f7348542d0619632 (diff)
downloadlinux-next-19f2c27251d86cf1272d475c52a190d01e9f996b.tar.gz
linux-next-19f2c27251d86cf1272d475c52a190d01e9f996b.zip
staging: iio: ad9834: simplify -ENOMEM return in probe
devm_iio_device_alloc() failure returns -ENOMEM via a local variable unnecessarily. Return -ENOMEM directly instead. Signed-off-by: Angus Gardner <angusg778@gmail.com> Reviewed-by: Maxwell Doose <m32285159@gmail.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
-rw-r--r--drivers/staging/iio/frequency/ad9834.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/iio/frequency/ad9834.c b/drivers/staging/iio/frequency/ad9834.c
index 330db78fe766..e7aee3307061 100644
--- a/drivers/staging/iio/frequency/ad9834.c
+++ b/drivers/staging/iio/frequency/ad9834.c
@@ -392,10 +392,8 @@ static int ad9834_probe(struct spi_device *spi)
return dev_err_probe(&spi->dev, ret, "Failed to enable specified AVDD supply\n");
indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st));
- if (!indio_dev) {
- ret = -ENOMEM;
- return ret;
- }
+ if (!indio_dev)
+ return -ENOMEM;
st = iio_priv(indio_dev);
mutex_init(&st->lock);
st->mclk = devm_clk_get_enabled(&spi->dev, NULL);