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>

diff --git a/drivers/iio/dac/mcp4821.c b/drivers/iio/dac/mcp4821.c
index 29187f2a9d3c48375ad17f00164a597c29ae527e..102ae37185148b3c1e4643c23bffb1980afff84a 100644 (file)
--- 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: