On 28/09/16 14:41, Ico Doornekamp wrote:
> reported by 'make W=2' in ad7606_oversampling_get_index()
>
> Signed-off-by: Ico Doornekamp <ico@xxxxxxxx>
hmm. It's pretty much a non problem in the first place as we know
it can't cause issues (can see it is 7). Also this really just
moves to typecasting it later when we return i.
So unless I'm missing something I'm not going to apply this one.
If Greg picks it up I don't mind though as doesn't do any harm
really.
Jonathan
> ---
> drivers/staging/iio/adc/ad7606_core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/adc/ad7606_core.c b/drivers/staging/iio/adc/ad7606_core.c
> index f79ee61..681389a 100644
> --- a/drivers/staging/iio/adc/ad7606_core.c
> +++ b/drivers/staging/iio/adc/ad7606_core.c
> @@ -158,7 +158,7 @@ static ssize_t ad7606_show_oversampling_ratio(struct device *dev,
> static int ad7606_oversampling_get_index(unsigned int val)
> {
> unsigned char supported[] = {0, 2, 4, 8, 16, 32, 64};
> - int i;
> + unsigned int i;
>
> for (i = 0; i < ARRAY_SIZE(supported); i++)
> if (val == supported[i])
>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
