Some datatypes were larger than they actually need to be, so they were adjusted 
to benefit from the new strto* functions.
Not everything might be optimal now, this is a starting point.

Signed-off-by: Andreas Ruprecht <[email protected]>
---
 drivers/staging/iio/adc/ad7793.c      |    4 ++--
 drivers/staging/iio/adc/ad799x_core.c |    8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/iio/adc/ad7793.c b/drivers/staging/iio/adc/ad7793.c
index ac24c5b..0d9e5ea 100644
--- a/drivers/staging/iio/adc/ad7793.c
+++ b/drivers/staging/iio/adc/ad7793.c
@@ -567,7 +567,7 @@ static ssize_t ad7793_write_frequency(struct device *dev,
 {
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct ad7793_state *st = iio_priv(indio_dev);
-       long lval;
+       u16 lval;
        int i, ret;
 
        mutex_lock(&indio_dev->mlock);
@@ -577,7 +577,7 @@ static ssize_t ad7793_write_frequency(struct device *dev,
        }
        mutex_unlock(&indio_dev->mlock);
 
-       ret = kstrtol(buf, 10, &lval);
+       ret = kstrtou16(buf, 10, &lval);
        if (ret)
                return ret;
 
diff --git a/drivers/staging/iio/adc/ad799x_core.c 
b/drivers/staging/iio/adc/ad799x_core.c
index 32052a6..72f380d 100644
--- a/drivers/staging/iio/adc/ad799x_core.c
+++ b/drivers/staging/iio/adc/ad799x_core.c
@@ -204,11 +204,11 @@ static ssize_t ad799x_write_frequency(struct device *dev,
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct ad799x_state *st = iio_priv(indio_dev);
 
-       long val;
+       unsigned int val;
        int ret, i;
        u8 t;
 
-       ret = kstrtol(buf, 10, &val);
+       ret = kstrtouint(buf, 10, &val);
        if (ret)
                return ret;
 
@@ -316,10 +316,10 @@ static ssize_t ad799x_write_channel_config(struct device 
*dev,
        struct ad799x_state *st = iio_priv(indio_dev);
        struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
 
-       long val;
+       u16 val;
        int ret;
 
-       ret = kstrtol(buf, 10, &val);
+       ret = kstrtou16(buf, 10, &val);
        if (ret)
                return ret;
 
-- 
1.7.5.4

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to