| Message ID | 1520236170-14668-3-git-send-email-himanshujha199640@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, 5 Mar 2018 13:19:21 +0530 Himanshu Jha <himanshujha199640@gmail.com> wrote: > Adding a blank space before/after some returns improves readability. > > Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com> Applied - patch title adjusted. Thanks, Jonathan > --- > drivers/staging/iio/accel/adis16201.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/staging/iio/accel/adis16201.c b/drivers/staging/iio/accel/adis16201.c > index 0f6a204..0fae8aa 100644 > --- a/drivers/staging/iio/accel/adis16201.c > +++ b/drivers/staging/iio/accel/adis16201.c > @@ -232,6 +232,7 @@ static int adis16201_read_raw(struct iio_dev *indio_dev, > *val = val16; > return IIO_VAL_INT; > } > + > return -EINVAL; > } > > @@ -262,6 +263,7 @@ static int adis16201_write_raw(struct iio_dev *indio_dev, > addr = adis16201_addresses[chan->scan_index]; > return adis_write_reg_16(st, addr, val16); > } > + > return -EINVAL; > } > > @@ -336,6 +338,7 @@ static int adis16201_probe(struct spi_device *spi) > ret = adis_init(st, indio_dev, spi, &adis16201_data); > if (ret) > return ret; > + > ret = adis_setup_buffer_and_trigger(st, indio_dev, NULL); > if (ret) > return ret; > @@ -348,6 +351,7 @@ static int adis16201_probe(struct spi_device *spi) > ret = iio_device_register(indio_dev); > if (ret < 0) > goto error_cleanup_buffer_trigger; > + > return 0; > > error_cleanup_buffer_trigger: -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/staging/iio/accel/adis16201.c b/drivers/staging/iio/accel/adis16201.c index 0f6a204..0fae8aa 100644 --- a/drivers/staging/iio/accel/adis16201.c +++ b/drivers/staging/iio/accel/adis16201.c @@ -232,6 +232,7 @@ static int adis16201_read_raw(struct iio_dev *indio_dev, *val = val16; return IIO_VAL_INT; } + return -EINVAL; } @@ -262,6 +263,7 @@ static int adis16201_write_raw(struct iio_dev *indio_dev, addr = adis16201_addresses[chan->scan_index]; return adis_write_reg_16(st, addr, val16); } + return -EINVAL; } @@ -336,6 +338,7 @@ static int adis16201_probe(struct spi_device *spi) ret = adis_init(st, indio_dev, spi, &adis16201_data); if (ret) return ret; + ret = adis_setup_buffer_and_trigger(st, indio_dev, NULL); if (ret) return ret; @@ -348,6 +351,7 @@ static int adis16201_probe(struct spi_device *spi) ret = iio_device_register(indio_dev); if (ret < 0) goto error_cleanup_buffer_trigger; + return 0; error_cleanup_buffer_trigger:
Adding a blank space before/after some returns improves readability. Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com> --- drivers/staging/iio/accel/adis16201.c | 4 ++++ 1 file changed, 4 insertions(+)