[PATCH v2 0/2] iio: pressure: dps310: Reset chip if MEAS_CFG is corrupt

Eddie James posted 2 patches 3 years, 11 months ago
There is a newer version of this series
drivers/iio/pressure/dps310.c | 281 +++++++++++++++++++++-------------
1 file changed, 174 insertions(+), 107 deletions(-)
[PATCH v2 0/2] iio: pressure: dps310: Reset chip if MEAS_CFG is corrupt
Posted by Eddie James 3 years, 11 months ago
Corruption of the MEAS_CFG register has been observed soon after
system boot. In order to recover this scenario, check MEAS_CFG if
measurement isn't ready, and if it's incorrect, reset the DPS310
and execute the startup procedure. Include a patch to move the
startup procedure into a function.

Changes since v1:
 - Separate into two patches
 - Rename 'dps310_verify_meas_cfg' to 'dps310_check_reset_meas_cfg'

Eddie James (2):
  iio: pressure: dps310: Refactor startup procedure
  iio: pressure: dps310: Reset chip if MEAS_CFG is corrupt

 drivers/iio/pressure/dps310.c | 281 +++++++++++++++++++++-------------
 1 file changed, 174 insertions(+), 107 deletions(-)

-- 
2.27.0
Re: [PATCH v2 0/2] iio: pressure: dps310: Reset chip if MEAS_CFG is corrupt
Posted by Jonathan Cameron 3 years, 11 months ago
On Wed, 18 May 2022 09:48:16 -0500
Eddie James <eajames@linux.ibm.com> wrote:

> Corruption of the MEAS_CFG register has been observed soon after
> system boot. In order to recover this scenario, check MEAS_CFG if
> measurement isn't ready, and if it's incorrect, reset the DPS310
> and execute the startup procedure. Include a patch to move the
> startup procedure into a function.
> 
> Changes since v1:
>  - Separate into two patches
>  - Rename 'dps310_verify_meas_cfg' to 'dps310_check_reset_meas_cfg'

Looks good to me. I'll leave this a little longer on list to give Joel
and others the opportunity to take a look.

If I seem to have lost it in a few weeks time, feel free to ping me
(it's been known to happen though I think my tracking is much less
error prone now I have moved to patchwork + my local system).

Jonathan

> 
> Eddie James (2):
>   iio: pressure: dps310: Refactor startup procedure
>   iio: pressure: dps310: Reset chip if MEAS_CFG is corrupt
> 
>  drivers/iio/pressure/dps310.c | 281 +++++++++++++++++++++-------------
>  1 file changed, 174 insertions(+), 107 deletions(-)
>