mbox series

[v3,0/6] iio: pressure: hsc030pa: cleanup and triggered buffer

Message ID 20240211075645.28777-1-petre.rodan@subdimension.ro (mailing list archive)
Headers show
Series iio: pressure: hsc030pa: cleanup and triggered buffer | expand

Message

Petre Rodan Feb. 11, 2024, 7:56 a.m. UTC
This set of patches covers the following:

 - small cleanup
 - mandatory 2ms delay between readings
 - support for triggered buffer readings

The support for devices that have "sleep mode" factory option that was
present in v1 of this patchset was removed, for a few reasons:

 - a Honeywell employee told me that this chip variant is extremely
    unlikely to be found in the wild, which also makes testing the
    driver functionality impossible
 - I found no reliable way of generating i2c/spi bus traffic with no
    payload (toggle CS for SPI case, send i2c packet containing only
    the address byte) that are required for the wakeup sequence.

Petre Rodan (6):
  dt-bindings: iio: pressure: honeywell,hsc030pa.yaml add spi props
  iio: pressure: hsc030pa: use signed type to hold div_64() result
  iio: pressure: hsc030pa: include cleanup
  iio: pressure: hsc030pa: update datasheet URLs
  iio: pressure: hsc030pa add mandatory delay
  iio: pressure: hsc030pa add triggered buffer

 .../iio/pressure/honeywell,hsc030pa.yaml      |  3 ++
 drivers/iio/pressure/Kconfig                  |  2 +
 drivers/iio/pressure/hsc030pa.c               | 49 ++++++++++++++++++-
 drivers/iio/pressure/hsc030pa.h               |  7 +++
 drivers/iio/pressure/hsc030pa_i2c.c           |  9 +++-
 drivers/iio/pressure/hsc030pa_spi.c           |  7 ++-
 6 files changed, 73 insertions(+), 4 deletions(-)

Comments

Jonathan Cameron Feb. 16, 2024, 3 p.m. UTC | #1
On Sun, 11 Feb 2024 09:56:31 +0200
Petre Rodan <petre.rodan@subdimension.ro> wrote:

> This set of patches covers the following:
> 
>  - small cleanup
>  - mandatory 2ms delay between readings
>  - support for triggered buffer readings

Hi Petre

Applied to the togreg branch of iio.git but that will initially be pushed
out as testing for 0-day to take a look at it.

Thanks,

Jonathan

> 
> The support for devices that have "sleep mode" factory option that was
> present in v1 of this patchset was removed, for a few reasons:
> 
>  - a Honeywell employee told me that this chip variant is extremely
>     unlikely to be found in the wild, which also makes testing the
>     driver functionality impossible
>  - I found no reliable way of generating i2c/spi bus traffic with no
>     payload (toggle CS for SPI case, send i2c packet containing only
>     the address byte) that are required for the wakeup sequence.

> 
> Petre Rodan (6):
>   dt-bindings: iio: pressure: honeywell,hsc030pa.yaml add spi props
>   iio: pressure: hsc030pa: use signed type to hold div_64() result
>   iio: pressure: hsc030pa: include cleanup
>   iio: pressure: hsc030pa: update datasheet URLs
>   iio: pressure: hsc030pa add mandatory delay
>   iio: pressure: hsc030pa add triggered buffer
> 
>  .../iio/pressure/honeywell,hsc030pa.yaml      |  3 ++
>  drivers/iio/pressure/Kconfig                  |  2 +
>  drivers/iio/pressure/hsc030pa.c               | 49 ++++++++++++++++++-
>  drivers/iio/pressure/hsc030pa.h               |  7 +++
>  drivers/iio/pressure/hsc030pa_i2c.c           |  9 +++-
>  drivers/iio/pressure/hsc030pa_spi.c           |  7 ++-
>  6 files changed, 73 insertions(+), 4 deletions(-)
>