summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorNikhil Gautam <nikhilgtr@gmail.com>2026-04-14 22:33:07 +0530
committerJonathan Cameron <jic23@kernel.org>2026-04-27 09:58:20 +0100
commit090839ecff2b30fa87b73f5001cdb150673a4d4c (patch)
treed97c67ef387ba1eb91eb7abcc82a9b172118f0e5 /drivers
parentf4d618c6185101efa072fd5845000fee2074b8b8 (diff)
downloadlinux-090839ecff2b30fa87b73f5001cdb150673a4d4c.tar.gz
linux-090839ecff2b30fa87b73f5001cdb150673a4d4c.zip
iio: dac: mcp4821: move state initialization outside switch
Move the iio_priv() call outside the switch statement in mcp4821_read_raw() to avoid repeating it in multiple cases. No functional change. Signed-off-by: Nikhil Gautam <nikhilgtr@gmail.com> Reviewed-by: David Lechner <dlechner@baylibre.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/iio/dac/mcp4821.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iio/dac/mcp4821.c b/drivers/iio/dac/mcp4821.c
index 29187f2a9d3c..102ae3718514 100644
--- a/drivers/iio/dac/mcp4821.c
+++ b/drivers/iio/dac/mcp4821.c
@@ -115,11 +115,10 @@ static int mcp4821_read_raw(struct iio_dev *indio_dev,
struct iio_chan_spec const *chan, int *val,
int *val2, long mask)
{
- struct mcp4821_state *state;
+ struct mcp4821_state *state = iio_priv(indio_dev);
switch (mask) {
case IIO_CHAN_INFO_RAW:
- state = iio_priv(indio_dev);
*val = state->dac_value[chan->channel];
return IIO_VAL_INT;
case IIO_CHAN_INFO_SCALE: