[PATCH v4 10/26] staging: iio: tsl2583: fix comparison between signed and unsigned integers

Jonathan Cameron jic23 at kernel.org
Sun Nov 13 12:14:34 UTC 2016


On 12/11/16 18:19, Brian Masney wrote:
> Fixed warning found by make W=2:
> 
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
> 
> Signed-off-by: Brian Masney <masneyb at onstation.org>
Applied.
> ---
>  drivers/staging/iio/light/tsl2583.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/iio/light/tsl2583.c b/drivers/staging/iio/light/tsl2583.c
> index d482a84..be3cbae 100644
> --- a/drivers/staging/iio/light/tsl2583.c
> +++ b/drivers/staging/iio/light/tsl2583.c
> @@ -510,7 +510,7 @@ static ssize_t in_illuminance_lux_table_show(struct device *dev,
>  					     struct device_attribute *attr,
>  					     char *buf)
>  {
> -	int i;
> +	unsigned int i;
>  	int offset = 0;
>  
>  	for (i = 0; i < ARRAY_SIZE(tsl2583_device_lux); i++) {
> @@ -541,7 +541,8 @@ static ssize_t in_illuminance_lux_table_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(tsl2583_device_lux) * 3 + 1];
> -	int n, ret = -EINVAL;
> +	int ret = -EINVAL;
> +	unsigned int n;
>  
>  	mutex_lock(&chip->als_mutex);
>  
> @@ -719,7 +720,7 @@ static int tsl2583_write_raw(struct iio_dev *indio_dev,
>  		break;
>  	case IIO_CHAN_INFO_CALIBSCALE:
>  		if (chan->type == IIO_LIGHT) {
> -			int i;
> +			unsigned int i;
>  
>  			for (i = 0; i < ARRAY_SIZE(gainadj); i++) {
>  				if (gainadj[i].mean == val) {
> 



More information about the devel mailing list