From: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Date: Sat, 18 Jan 2014 19:05:00 +0000 (+0000)
Subject: iio: mxs-lradc: remove useless check
X-Git-Tag: v3.15-rc1~139^2~1171^2~6
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=c842f241956f880fbba9bc041b196a249f80d131;p=~emulex%2Finfiniband.git

iio: mxs-lradc: remove useless check

Checking the channel number is useless since mxs_lradc_read_raw() is called from
a controlled environment and the driver is responsible for filing the struct
iio_chan_spec.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Acked-by: Marek Vasut <marex@denx.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
---

diff --git a/drivers/staging/iio/adc/mxs-lradc.c b/drivers/staging/iio/adc/mxs-lradc.c
index df71669bb60..7b9224198bb 100644
--- a/drivers/staging/iio/adc/mxs-lradc.c
+++ b/drivers/staging/iio/adc/mxs-lradc.c
@@ -897,10 +897,6 @@ static int mxs_lradc_read_raw(struct iio_dev *iio_dev,
 {
 	struct mxs_lradc *lradc = iio_priv(iio_dev);
 
-	/* Check for invalid channel */
-	if (chan->channel > LRADC_MAX_TOTAL_CHANS)
-		return -EINVAL;
-
 	switch (m) {
 	case IIO_CHAN_INFO_RAW:
 		if (chan->type == IIO_TEMP)