diff options
author | Alexandru Ardelean <alexandru.ardelean@analog.com> | 2018-05-21 09:40:05 +0300 |
---|---|---|
committer | Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2018-05-22 18:13:45 +0100 |
commit | 2636d005758928c2884ac610589f383c73919bcf (patch) | |
tree | 7326f2112232daeb0235f1453b0c4247c6af32b8 /drivers/iio/amplifiers | |
parent | 55dda0abcf9d36bf5c3e21c8423b7c00a1125e27 (diff) | |
download | linux-2636d005758928c2884ac610589f383c73919bcf.tar.gz linux-2636d005758928c2884ac610589f383c73919bcf.tar.bz2 linux-2636d005758928c2884ac610589f383c73919bcf.zip |
iio: amplifiers: ad8366: move channel init before iio_device_register()
Otherwise a race condition can occur, where userspace can start operations
before the channels have been properly initialized.
Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/iio/amplifiers')
-rw-r--r-- | drivers/iio/amplifiers/ad8366.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/iio/amplifiers/ad8366.c b/drivers/iio/amplifiers/ad8366.c index 43667866321e..0138337aedd1 100644 --- a/drivers/iio/amplifiers/ad8366.c +++ b/drivers/iio/amplifiers/ad8366.c @@ -161,12 +161,14 @@ static int ad8366_probe(struct spi_device *spi) indio_dev->channels = ad8366_channels; indio_dev->num_channels = ARRAY_SIZE(ad8366_channels); + ret = ad8366_write(indio_dev, 0 , 0); + if (ret < 0) + goto error_disable_reg; + ret = iio_device_register(indio_dev); if (ret) goto error_disable_reg; - ad8366_write(indio_dev, 0, 0); - return 0; error_disable_reg: |