On Wed, Nov 23, 2011 at 01:37:59PM +0100, Andreas Ruprecht wrote:
> diff --git a/drivers/staging/iio/accel/adis16220_core.c 
> b/drivers/staging/iio/accel/adis16220_core.c
> index 6d4503d..9966b64 100644
> --- a/drivers/staging/iio/accel/adis16220_core.c
> +++ b/drivers/staging/iio/accel/adis16220_core.c
> @@ -169,7 +169,7 @@ static ssize_t adis16220_write_16bit(struct device *dev,
>       int ret;
>       long val;
>  
> -     ret = strict_strtol(buf, 10, &val);
> +     ret = kstrtol(buf, 10, &val);
>       if (ret)
>               goto error_ret;
>       ret = adis16220_spi_write_reg_16(indio_dev, this_attr->address, val);

This one should be kstrtou16().

It's a behavior change, and a bugfix.  I haven't reviewed your whole
patch, only this first couple lines so there may be other places
where the original code could be improved.

regards,
dan carpenter

Attachment: signature.asc
Description: Digital signature

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

Reply via email to