linux-kernel - Re: [PATCH v1 1/1] iio: pressure: dps310: Adjust Timeout Settings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20230903125309.12c59189@jic23-huawei>
Date:   Sun, 3 Sep 2023 12:53:09 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Lakshmi Yadlapati <lakshmiy@...ibm.com>
Cc:     eajames@...ux.ibm.com, lars@...afoo.de, joel@....id.au,
        andrew@...id.au, linux-iio@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 1/1] iio: pressure: dps310: Adjust Timeout Settings

On Tue, 29 Aug 2023 13:02:22 -0500
Lakshmi Yadlapati <lakshmiy@...ibm.com> wrote:

> The DPS310 sensor chip has been encountering intermittent errors while
> reading the sensor device across various system designs. This issue causes
> the chip to become "stuck," preventing the indication of "ready" status
> for pressure and temperature measurements in the MEAS_CFG register.
> 
> To address this issue, this commit fixes the timeout settings to improve
> sensor stability:
> - After sending a reset command to the chip, the timeout has been extended
>   from 2.5 ms to 15 ms, aligning with the DPS310 specification.
> - The read timeout value of the MEAS_CFG register has been adjusted from
>   20ms to 30ms to match the specification.
> 
> Signed-off-by: Lakshmi Yadlapati <lakshmiy@...ibm.com>
Hi Lakshmi,

This seems reasonable to me, but as we probably want to backport it to
stable kernels, please could you figure out an appropriate Fixes tag.

Fine to just reply to this email with the tag rather than resend.

Thanks

Jonathan
 
> ---
>  drivers/iio/pressure/dps310.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/iio/pressure/dps310.c b/drivers/iio/pressure/dps310.c
> index b10dbf5cf494..1ff091b2f764 100644
> --- a/drivers/iio/pressure/dps310.c
> +++ b/drivers/iio/pressure/dps310.c
> @@ -57,8 +57,8 @@
>  #define  DPS310_RESET_MAGIC	0x09
>  #define DPS310_COEF_BASE	0x10
>  
> -/* Make sure sleep time is <= 20ms for usleep_range */
> -#define DPS310_POLL_SLEEP_US(t)		min(20000, (t) / 8)
> +/* Make sure sleep time is <= 30ms for usleep_range */
> +#define DPS310_POLL_SLEEP_US(t)		min(30000, (t) / 8)
>  /* Silently handle error in rate value here */
>  #define DPS310_POLL_TIMEOUT_US(rc)	((rc) <= 0 ? 1000000 : 1000000 / (rc))
>  
> @@ -402,8 +402,8 @@ static int dps310_reset_wait(struct dps310_data *data)
>  	if (rc)
>  		return rc;
>  
> -	/* Wait for device chip access: 2.5ms in specification */
> -	usleep_range(2500, 12000);
> +	/* Wait for device chip access: 15ms in specification */
> +	usleep_range(15000, 55000);
>  	return 0;
>  }
>