[v3,0/3] iio: humidity: hdc3020: add power and reset management

Message

Javier Carrasco March 3, 2024, 9:54 p.m. UTC

This series adds power management for the hdc3020 humidity and
temperature sensor as well as control over the reset signal the device
provides.

The hdc3020 carries out measurements automatically, which is not
necessary in low-power modes. Furthermore, if the low-power
configuration turns off the device, proper initialization is required to
account for the setup times and initial status register value.

This device provides an active low reset signal that must be handled if
connected. This signal can be used by the driver to keep the device
under minimal power consumption during low-power modes if the power
supply stays active.

This series uses char-misc-next as basis to include the last additions
to the driver to handle events [1] as well as the fix to include the
entries in the Makefile and Kconfig files [2].

[1] https://lore.kernel.org/linux-iio/[e-mail]/
[2] https://lore.kernel.org/linux-iio/[e-mail]/

Signed-off-by: Javier Carrasco <[e-mail]>
---
Changes in v3:
- Drop unnecessary casting to void in dev_set_drvdata.
- Call devm_add_action_or_reset right after powering on.
- Link to v2: https://lore.kernel.org/r/[e-mail]

Changes in v2:
- Trigger power off sequence if the power on sequence fails.
- Check return value of hdc3020_power_on() in the probe.
- Remove type casting for void pointer.
- Link to v1: https://lore.kernel.org/r/[e-mail]

---
Javier Carrasco (3):
      iio: humidity: hdc3020: add power management
      dt-bindings: iio: humidity: hdc3020: add reset-gpios
      iio: humidity: hdc3020: add reset management

 .../bindings/iio/humidity/ti,hdc3020.yaml          |   5 +
 drivers/iio/humidity/hdc3020.c                     | 111 +++++++++++++++++----
 2 files changed, 97 insertions(+), 19 deletions(-)
---
base-commit: d4551c189d6e6a3fcf7f625bd4b273e770fad35a
change-id: 20240217-hdc3020-pm-177983de3cab

Best regards,

Comments

Jonathan Cameron March 10, 2024, 3:24 p.m. UTC | #1

On Sun, 03 Mar 2024 22:54:19 +0100
Javier Carrasco <[e-mail]> wrote:

> This series adds power management for the hdc3020 humidity and
> temperature sensor as well as control over the reset signal the device
> provides.
> 
> The hdc3020 carries out measurements automatically, which is not
> necessary in low-power modes. Furthermore, if the low-power
> configuration turns off the device, proper initialization is required to
> account for the setup times and initial status register value.
> 
> This device provides an active low reset signal that must be handled if
> connected. This signal can be used by the driver to keep the device
> under minimal power consumption during low-power modes if the power
> supply stays active.
> 
> This series uses char-misc-next as basis to include the last additions
> to the driver to handle events [1] as well as the fix to include the
> entries in the Makefile and Kconfig files [2].
> 
> [1] https://lore.kernel.org/linux-iio/[e-mail]/
> [2] https://lore.kernel.org/linux-iio/[e-mail]/
> 
> Signed-off-by: Javier Carrasco <[e-mail]>
Applied to the togreg-normal branch of iio.git and pushed out for 0-day to
see what it can find.  Note this is now 6.10 material and I'll be rebasing
on rc1 once available. It won't go into a tree next picks up until after that.

Thanks,

Jonathan

> ---
> Changes in v3:
> - Drop unnecessary casting to void in dev_set_drvdata.
> - Call devm_add_action_or_reset right after powering on.
> - Link to v2: https://lore.kernel.org/r/[e-mail]
> 
> Changes in v2:
> - Trigger power off sequence if the power on sequence fails.
> - Check return value of hdc3020_power_on() in the probe.
> - Remove type casting for void pointer.
> - Link to v1: https://lore.kernel.org/r/[e-mail]
> 
> ---
> Javier Carrasco (3):
>       iio: humidity: hdc3020: add power management
>       dt-bindings: iio: humidity: hdc3020: add reset-gpios
>       iio: humidity: hdc3020: add reset management
> 
>  .../bindings/iio/humidity/ti,hdc3020.yaml          |   5 +
>  drivers/iio/humidity/hdc3020.c                     | 111 +++++++++++++++++----
>  2 files changed, 97 insertions(+), 19 deletions(-)
> ---
> base-commit: d4551c189d6e6a3fcf7f625bd4b273e770fad35a
> change-id: 20240217-hdc3020-pm-177983de3cab
> 
> Best regards,