Re: [PATCH v2 6/7] iio:light:cm3323: make use of GENMASK

Jonathan Cameron
2015-06-21T13:28:37Z

On 17/06/15 23:32, Hartmut Knaack wrote:
> Use GENMASK to define the integration time bitmask.
>
> Signed-off-by: Hartmut Knaack <knaack.h@gmx.de>
Applied
> ---
> drivers/iio/light/cm3323.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/light/cm3323.c b/drivers/iio/light/cm3323.c
> index 2c49ec2..1fc9f78 100644
> --- a/drivers/iio/light/cm3323.c
> +++ b/drivers/iio/light/cm3323.c
> @@ -29,7 +29,7 @@
>
> #define CM3323_CONF_SD_BIT BIT(0) /* sensor disable */
> #define CM3323_CONF_AF_BIT BIT(1) /* auto/manual force mode */
> -#define CM3323_CONF_IT_MASK (BIT(4) | BIT(5) | BIT(6))
> +#define CM3323_CONF_IT_MASK GENMASK(6, 4)
> #define CM3323_CONF_IT_SHIFT 4
>
> #define CM3323_INT_TIME_AVAILABLE "0.04 0.08 0.16 0.32 0.64 1.28"