[PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}

Path	csiph.com!eternal-september.org!feeder.eternal-september.org!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	Brian Masney <masneyb@onstation.org>
Newsgroups	linux.kernel
Subject	[PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}
Date	Fri, 28 Oct 2016 12:10:02 +0200
Message-ID	<sxcgO-6SN-37@gated-at.bofh.it> (permalink)
References	<sxcgN-6SN-3@gated-at.bofh.it>
X-Original-To	jic23@kernel.org, linux-iio@vger.kernel.org
Dkim-Signature	v=1; a=rsa-sha256; c=simple/simple; d=onstation.org; s=default; t=1477648827; bh=K6DK7CtH+ghw/3z4z0KQfLnl/QrVEGjKD6qdvYG1HFY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=SA4tk5HifaV1NzbD6lOpr08E8Ujd4ooB+7YDaKu/jP7muY89DwHW3oUgb2OYzdtWf AqCoPBe3AsmVuuuuwJXD0p5YG1KqQ0QfTv0L70YPFYj+899i1RmsVRLAwnaObj/WMk tyRPDSqZrL9ENNSA7Fmc8P3Zpt66U5tWMjBhgEWw=
X-Mailer	git-send-email 2.7.4
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	73
Organization	linux.* mail to news gateway
X-Original-Cc	devel@driverdev.osuosl.org, gregkh@linuxfoundation.org, lars@metafoo.de, pmeerw@pmeerw.net, knaack.h@gmx.de, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, robh+dt@kernel.org, Mark.Rutland@arm.com
X-Original-Date	Fri, 28 Oct 2016 06:00:16 -0400
X-Original-Message-ID	<1477648821-3786-6-git-send-email-masneyb@onstation.org>
X-Original-References	<1477648821-3786-1-git-send-email-masneyb@onstation.org>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1511030

Show key headers only | View raw

The return value from taos_chip_on() and taos_chip_off() was not
checked in taos_luxtable_store() and taos_probe(). This patch adds
proper error checking to these function calls.

Signed-off-by: Brian Masney <masneyb@onstation.org>
---
 drivers/staging/iio/light/tsl2583.c | 26 ++++++++++++++++++--------
 1 file changed, 18 insertions(+), 8 deletions(-)

diff --git a/drivers/staging/iio/light/tsl2583.c b/drivers/staging/iio/light/tsl2583.c
index f8ccb4d..e975bba 100644
--- a/drivers/staging/iio/light/tsl2583.c
+++ b/drivers/staging/iio/light/tsl2583.c
@@ -727,7 +727,7 @@ static ssize_t taos_luxtable_store(struct device *dev,
 	struct iio_dev *indio_dev = dev_to_iio_dev(dev);
 	struct tsl2583_chip *chip = iio_priv(indio_dev);
 	int value[ARRAY_SIZE(taos_device_lux) * 3 + 1];
-	int n;
+	int n, ret = -EINVAL;
 
 	get_options(buf, ARRAY_SIZE(value), value);
 
@@ -739,23 +739,31 @@ static ssize_t taos_luxtable_store(struct device *dev,
 	n = value[0];
 	if ((n % 3) || n < 6 || n > ((ARRAY_SIZE(taos_device_lux) - 1) * 3)) {
 		dev_info(dev, "LUX TABLE INPUT ERROR 1 Value[0]=%d\n", n);
-		return -EINVAL;
+		goto done;
 	}
 	if ((value[(n - 2)] | value[(n - 1)] | value[n]) != 0) {
 		dev_info(dev, "LUX TABLE INPUT ERROR 2 Value[0]=%d\n", n);
-		return -EINVAL;
+		goto done;
 	}
 
-	if (chip->taos_chip_status == TSL258X_CHIP_WORKING)
-		taos_chip_off(indio_dev);
+	if (chip->taos_chip_status == TSL258X_CHIP_WORKING) {
+		ret = taos_chip_off(indio_dev);
+		if (ret < 0)
+			goto done;
+	}
 
 	/* Zero out the table */
 	memset(taos_device_lux, 0, sizeof(taos_device_lux));
 	memcpy(taos_device_lux, &value[1], (value[0] * 4));
 
-	taos_chip_on(indio_dev);
+	ret = taos_chip_on(indio_dev);
+	if (ret < 0)
+		goto done;
 
-	return len;
+	ret = len;
+
+done:
+	return ret;
 }
 
 static DEVICE_ATTR(illuminance0_calibscale, S_IRUGO | S_IWUSR,
@@ -883,7 +891,9 @@ static int taos_probe(struct i2c_client *clientp,
 	taos_defaults(chip);
 
 	/* Make sure the chip is on */
-	taos_chip_on(indio_dev);
+	ret = taos_chip_on(indio_dev);
+	if (ret < 0)
+		return ret;
 
 	dev_info(&clientp->dev, "Light sensor found.\n");
 	return 0;
-- 
2.7.4

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off} Brian Masney <masneyb@onstation.org> - 2016-10-28 12:10 +0200
  Re: [PATCH 05/10] staging: iio: tsl2583: check return values from  taos_chip_{on,off} Jonathan Cameron <jic23@kernel.org> - 2016-10-30 19:00 +0100

csiph-web