Re: [PATCH] iio: adc: tsc2046: fix a warning message in tsc2046_adc_update_scan_mode()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 15 May 2021 12:51:52 +0300
Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:

> These variables are unsigned so the condition can't be less than zero
> and the warning message will never be printed.
> 
> Fixes: 5fec3541aa88 ("iio: adc: add ADC driver for the TI TSC2046 controller")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Applied.  Thanks.

Chances are I'm going to rebase togreg shortly to unwind a merge conflict
with fixes-togreg so hopefully I'll manage to remember to update that fixes
tag.

Thanks,

Jonathan


> ---
>  drivers/iio/adc/ti-tsc2046.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/adc/ti-tsc2046.c b/drivers/iio/adc/ti-tsc2046.c
> index 89a818b653b4..cf5373d5cdd7 100644
> --- a/drivers/iio/adc/ti-tsc2046.c
> +++ b/drivers/iio/adc/ti-tsc2046.c
> @@ -398,7 +398,7 @@ static int tsc2046_adc_update_scan_mode(struct iio_dev *indio_dev,
>  	priv->xfer.len = size;
>  	priv->time_per_scan_us = size * 8 * priv->time_per_bit_ns / NSEC_PER_USEC;
>  
> -	if ((priv->scan_interval_us - priv->time_per_scan_us) < 0)
> +	if (priv->scan_interval_us > priv->time_per_scan_us)
>  		dev_warn(&priv->spi->dev, "The scan interval (%d) is less then calculated scan time (%d)\n",
>  			 priv->scan_interval_us, priv->time_per_scan_us);
>  




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux