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

Jonathan Cameron
2015-06-21T13:20:57Z

On 21/06/15 14:19, Jonathan Cameron wrote:
> 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.
Or I could just check beyond patch 1 and see that he's already
reviewed them in V1. oops.
>
> Path 1 is clearly a bug fix, but the others can go the slow
> route.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in