[PATCH] Staging: iio/adc: small data type glitch for kstrtoul

Andreas Ruprecht rupran at einserver.de
Wed Nov 23 20:11:23 UTC 2011


When doing changes from strict_strtoul to kstrtoul I didn't check if the target variable was of the right data type, and missed a long used as the parameter for kstrtoul.
Thanks to Dan for pointing this out.

Signed-off-by: Andreas Ruprecht <rupran at einserver.de>
---
 drivers/staging/iio/adc/ad7280a.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/iio/adc/ad7280a.c b/drivers/staging/iio/adc/ad7280a.c
index e754465..98d6cfc 100644
--- a/drivers/staging/iio/adc/ad7280a.c
+++ b/drivers/staging/iio/adc/ad7280a.c
@@ -455,7 +455,7 @@ static ssize_t ad7280_store_balance_timer(struct device *dev,
 	struct iio_dev *indio_dev = dev_get_drvdata(dev);
 	struct ad7280_state *st = iio_priv(indio_dev);
 	struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
-	long val;
+	unsigned long val;
 	int ret;
 
 	ret = kstrtoul(buf, 10, &val);
-- 
1.7.5.4




More information about the devel mailing list