mbox series

[v2,0/5] iio: accel: mma9551/mma9553 Cleanup and update

Message ID 20210603185207.3646368-1-jic23@kernel.org (mailing list archive)
Headers show
Series iio: accel: mma9551/mma9553 Cleanup and update | expand

Message

Jonathan Cameron June 3, 2021, 6:52 p.m. UTC
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

v2: mma9551: Drop the gpio based irq support as not known to be used
    and adds complexity which is nice to get rid of.

This series came about because I was looking to write a dt-binding for these
two (currently missing entirely) and I discovered the mma9551 driver in
particular was doing some unusual things.

Note however, I've only tested the fwnode_irq_get() patch using a hacked
up version of QEMU and stubbing out some error paths because I'm too
lazy to emulate it properly ;)

The ACPI entries seem unlikely, but please shout if anyone knows of
them being used in the wild.

It would be particularly helpful if anyone who has either of these
parts could both give this a spin and let me know so I can ask
for testing in future.

Thanks,

Jonathan

Jonathan Cameron (5):
  iio: accel: mma9551/mma9553: Drop explicit ACPI match support
  iio: accel: mma9551/mma9553: Simplify pm logic
  iio: accel: mma9551: Add support to get irqs directly from fwnode
  iio: accel: mma9551: Use devm managed functions to tidy up probe()
  iio: accel: mma9553: Use devm managed functions to tidy up probe()

 drivers/iio/accel/mma9551.c | 151 ++++++++++++------------------------
 drivers/iio/accel/mma9553.c | 121 +++++++++--------------------
 2 files changed, 85 insertions(+), 187 deletions(-)

Comments

Jonathan Cameron July 18, 2021, 3:14 p.m. UTC | #1
On Thu,  3 Jun 2021 19:52:02 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Hi All,

This is a series I'd appreciate some more eyes on if anyone has the time
to spare!

Thanks,

Jonathan

> 
> v2: mma9551: Drop the gpio based irq support as not known to be used
>     and adds complexity which is nice to get rid of.
> 
> This series came about because I was looking to write a dt-binding for these
> two (currently missing entirely) and I discovered the mma9551 driver in
> particular was doing some unusual things.
> 
> Note however, I've only tested the fwnode_irq_get() patch using a hacked
> up version of QEMU and stubbing out some error paths because I'm too
> lazy to emulate it properly ;)
> 
> The ACPI entries seem unlikely, but please shout if anyone knows of
> them being used in the wild.
> 
> It would be particularly helpful if anyone who has either of these
> parts could both give this a spin and let me know so I can ask
> for testing in future.
> 
> Thanks,
> 
> Jonathan
> 
> Jonathan Cameron (5):
>   iio: accel: mma9551/mma9553: Drop explicit ACPI match support
>   iio: accel: mma9551/mma9553: Simplify pm logic
>   iio: accel: mma9551: Add support to get irqs directly from fwnode
>   iio: accel: mma9551: Use devm managed functions to tidy up probe()
>   iio: accel: mma9553: Use devm managed functions to tidy up probe()
> 
>  drivers/iio/accel/mma9551.c | 151 ++++++++++++------------------------
>  drivers/iio/accel/mma9553.c | 121 +++++++++--------------------
>  2 files changed, 85 insertions(+), 187 deletions(-)
>