Re: [PATCH v5 0/9] Improve MCP3911 driver

Jonathan Cameron
2022-08-14T14:24:30Z

On Tue, 9 Aug 2022 09:36:39 +0200
Marcus Folkesson <marcus.folkesson@gmail.com> wrote:

> Hi,
>
> This patch series intend to fix bugs and improve functionality of the MCP3911 driver.
> The main features added are
> - Support for buffers
> - Interrupt driven readings
> - Support for oversampling ratio
> - Support for set scale values (Gain)
>
> Among the bug fixes, there are changes in the formula for calculate raw value and a fix for mismatch in the devicetree property.
>
> Another general improvement for the driver is to use managed resources for all allocated resources.
>
Given requested changes from Andy and I were all small, I would normally
have just made them whilst applying. However, patches 1-3 still need
to work their way through as fixes to be upstream so we can't merge the
rest yet anyway.

Looks good to me in general, just those little things to tidy up in a v6.

Thanks,

Jonathan

> General changes for the series:
>
> v3:
> - Drop Phase patch
> - Add Fixes tags for those patches that are fixes
> - Move Fixes patches to the beginning of the patchset
>
> v4:
> - Split up devm-cleanup functions
> - Cosmetic cleanups
> - Add
> select IIO_BUFFER
> select IIO_TRIGGERED_BUFFER
> To Kconfig
> - Add .endianness = IIO_BE
>
> v5:
> - Drop remove function
> - Split tx&rx transfers in mcp3911_trigger_handler()
> - Moved Kconfig changes to right patch
>
>
> Best regards,
> Marcus Folkesson
>
>