mbox series

[v2,0/4] iio: accel: bmc150: Add support for INT2 and BMC156

Message ID 20210802155657.102766-1-stephan@gerhold.net (mailing list archive)
Headers show
Series iio: accel: bmc150: Add support for INT2 and BMC156 | expand

Message

Stephan Gerhold Aug. 2, 2021, 3:56 p.m. UTC
This series makes it possible to set up interrupts with the BMC150 driver
on boards where only the INT2 pin is connected (and not INT1). This is
particularly always the case for BMC156 since for some reason it only
has the INT2 pin and not the INT1 pin.

These changes were already partially discussed here:
https://lore.kernel.org/linux-iio/YMOphuXSoODIVX06@gerhold.net/

Changes in v2:
  - PATCH 1/4: Clarify order of "interrupts" with "interrupt-names"
  - PATCH 4/4: Wrap a long line, clarify BOSCH_UNKNOWN with a comment

v1: https://lore.kernel.org/linux-iio/20210719112156.27087-1-stephan@gerhold.net/

Stephan Gerhold (4):
  dt-bindings: iio: accel: bma255: Add interrupt-names
  dt-bindings: iio: accel: bma255: Add bosch,bmc156_accel
  iio: accel: bmc150: Make it possible to configure INT2 instead of INT1
  iio: accel: bmc150: Add support for BMC156

 .../bindings/iio/accel/bosch,bma255.yaml      | 34 +++++++-
 drivers/iio/accel/Kconfig                     |  5 +-
 drivers/iio/accel/bmc150-accel-core.c         | 78 +++++++++++++++----
 drivers/iio/accel/bmc150-accel-i2c.c          | 10 ++-
 drivers/iio/accel/bmc150-accel-spi.c          | 10 ++-
 drivers/iio/accel/bmc150-accel.h              | 20 ++++-
 6 files changed, 134 insertions(+), 23 deletions(-)

Comments

Jonathan Cameron Aug. 8, 2021, 3:51 p.m. UTC | #1
On Mon,  2 Aug 2021 17:56:53 +0200
Stephan Gerhold <stephan@gerhold.net> wrote:

> This series makes it possible to set up interrupts with the BMC150 driver
> on boards where only the INT2 pin is connected (and not INT1). This is
> particularly always the case for BMC156 since for some reason it only
> has the INT2 pin and not the INT1 pin.
> 
> These changes were already partially discussed here:
> https://lore.kernel.org/linux-iio/YMOphuXSoODIVX06@gerhold.net/

Hopefully one of us or someone else will come back to this and
figure out a clean solution to generic fw support for getting named IRQs.
In the meantime this will be fine for this particular driver.

Some fun to look forwards to ;)

Applied to the togreg branch of iio.git and pushed out as testing for 0-day
to poke at it and see what we missed.

Thanks,

Jonathan

> 
> Changes in v2:
>   - PATCH 1/4: Clarify order of "interrupts" with "interrupt-names"
>   - PATCH 4/4: Wrap a long line, clarify BOSCH_UNKNOWN with a comment
> 
> v1: https://lore.kernel.org/linux-iio/20210719112156.27087-1-stephan@gerhold.net/
> 
> Stephan Gerhold (4):
>   dt-bindings: iio: accel: bma255: Add interrupt-names
>   dt-bindings: iio: accel: bma255: Add bosch,bmc156_accel
>   iio: accel: bmc150: Make it possible to configure INT2 instead of INT1
>   iio: accel: bmc150: Add support for BMC156
> 
>  .../bindings/iio/accel/bosch,bma255.yaml      | 34 +++++++-
>  drivers/iio/accel/Kconfig                     |  5 +-
>  drivers/iio/accel/bmc150-accel-core.c         | 78 +++++++++++++++----
>  drivers/iio/accel/bmc150-accel-i2c.c          | 10 ++-
>  drivers/iio/accel/bmc150-accel-spi.c          | 10 ++-
>  drivers/iio/accel/bmc150-accel.h              | 20 ++++-
>  6 files changed, 134 insertions(+), 23 deletions(-)
>