mbox series

[v3,0/3] Add Reset and Wakeup support for CCS811

Message ID 20200414184147.4857-1-mani@kernel.org (mailing list archive)
Headers show
Series Add Reset and Wakeup support for CCS811 | expand

Message

Manivannan Sadhasivam April 14, 2020, 6:41 p.m. UTC
From: Manivannan Sadhasivam <mani@kernel.org>

Hello,

This patchset adds external reset and wakeup support for CCS811 VOC
gas sensor. The nRESET and nWAKE pins available on the device are
utilised to provide these functionalities.

The patchset has been validated using CCS811 connected to STM32MP1 based
board over I2C.

While at it, the devicetree yaml binding and OF match table for this sensor
are also added.

Thanks,
Mani

Changes in v3:

* Added ccs811_set_wake(false) to all error paths before calling it actually
* Added Andy's reviewed-by tag
* Added comment for reset procedure and dropped error print for gpio request

Changes in v2:

* Fixed DT binding schema and switched to dual license (GPL/BSD)
* Returned actual error code from devm_gpiod_get_optional()
* Dropped of.h include and of_match_ptr()

Manivannan Sadhasivam (3):
  iio: chemical: Add support for external Reset and Wakeup in CCS811
  iio: chemical: Add OF match table for CCS811 VOC sensor
  dt-bindings: serial: Add binding for software flow control in STM32
    UART

 .../bindings/serial/st,stm32-uart.yaml        |  15 ++-
 drivers/iio/chemical/ccs811.c                 | 112 ++++++++++++++++--
 2 files changed, 115 insertions(+), 12 deletions(-)

Comments

Manivannan Sadhasivam April 14, 2020, 6:44 p.m. UTC | #1
On Wed, Apr 15, 2020 at 12:11:44AM +0530, mani@kernel.org wrote:
> From: Manivannan Sadhasivam <mani@kernel.org>
> 
> Hello,
> 
> This patchset adds external reset and wakeup support for CCS811 VOC
> gas sensor. The nRESET and nWAKE pins available on the device are
> utilised to provide these functionalities.
> 
> The patchset has been validated using CCS811 connected to STM32MP1 based
> board over I2C.
> 
> While at it, the devicetree yaml binding and OF match table for this sensor
> are also added.
> 
> Thanks,
> Mani
> 
> Changes in v3:
> 
> * Added ccs811_set_wake(false) to all error paths before calling it actually
> * Added Andy's reviewed-by tag
> * Added comment for reset procedure and dropped error print for gpio request
> 
> Changes in v2:
> 
> * Fixed DT binding schema and switched to dual license (GPL/BSD)
> * Returned actual error code from devm_gpiod_get_optional()
> * Dropped of.h include and of_match_ptr()
> 
> Manivannan Sadhasivam (3):
>   iio: chemical: Add support for external Reset and Wakeup in CCS811
>   iio: chemical: Add OF match table for CCS811 VOC sensor
>   dt-bindings: serial: Add binding for software flow control in STM32

Err... please ignore this series. I'll resend it.

Thanks,
Mani

>     UART
> 
>  .../bindings/serial/st,stm32-uart.yaml        |  15 ++-
>  drivers/iio/chemical/ccs811.c                 | 112 ++++++++++++++++--
>  2 files changed, 115 insertions(+), 12 deletions(-)
> 
> -- 
> 2.17.1
>