Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit da29461c authored by Cristina Opriceana's avatar Cristina Opriceana Committed by Greg Kroah-Hartman
Browse files

Staging: iio: Simplify return logic 



This patch removes the conditional return of the ade77*_probe functions
based on the return values of iio_device_register as the latter
returns 0 or ret, the same as the checked values.

Warning found by coccinelle.

Signed-off-by: default avatarCristina Opriceana <cristina.opriceana@gmail.com>
Reviewed-by: default avatarDaniel Baluta <daniel.baluta@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5db4851d

drivers/staging/iio/meter/ade7753.c

+1 −5
Original line number Diff line number Diff line
@@ -517,11 +517,7 @@ static int ade7753_probe(struct spi_device *spi) 
	if (ret)

		return ret;



	ret = iio_device_register(indio_dev);

	if (ret)

		return ret;



	return 0;

	return iio_device_register(indio_dev);

}



/* fixme, confirm ordering in this function */

drivers/staging/iio/meter/ade7754.c

+1 −5
Original line number Diff line number Diff line
@@ -540,11 +540,7 @@ static int ade7754_probe(struct spi_device *spi) 
	ret = ade7754_initial_setup(indio_dev);

	if (ret)

		return ret;

	ret = iio_device_register(indio_dev);

	if (ret)

		return ret;



	return 0;

	return iio_device_register(indio_dev);

}



/* fixme, confirm ordering in this function */

drivers/staging/iio/meter/ade7759.c

+1 −5
Original line number Diff line number Diff line
@@ -462,11 +462,7 @@ static int ade7759_probe(struct spi_device *spi) 
	if (ret)

		return ret;



	ret = iio_device_register(indio_dev);

	if (ret)

		return ret;



	return 0;

	return iio_device_register(indio_dev);

}



/* fixme, confirm ordering in this function */