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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux