Re: [PATCH v2 0/7] cm3323 fixes and cleanups

Jonathan Cameron
2015-06-21T13:19:40Z

On 17/06/15 23:31, Hartmut Knaack wrote:
> Some small fixes and cleanups.
>
> Changes in V2:
> - apply the integration time bitmask right
> - fix order of GENMASK arguments as pointed out by Peter Meerwald
>
> Hartmut Knaack (7):
> iio:light:cm3323: clear bitmask before set
> iio:light:Kconfig: fix typo in description
> iio:light:cm3323: pass up error value
> iio:light:cm3323: drop barely used variable
> iio:light:cm3323: replace unneeded variable
> iio:light:cm3323: make use of GENMASK
> iio:light:cm3323: add empty lines for code structure
>
> drivers/iio/light/Kconfig | 2 +-
> drivers/iio/light/cm3323.c | 27 +++++++++++++++------------
> 2 files changed, 16 insertions(+), 13 deletions(-)
>
This whole series looks good to me, but I'll let them sit
for Daniel to have a look as it's his driver.

Path 1 is clearly a bug fix, but the others can go the slow
route.