Re: [PATCH v2 4/5] iio: light: ROHM BU27008 color sensor

[Andy Shevchenko]

On Mon, Apr 24, 2023 at 04:10:09PM +0300, Matti Vaittinen wrote:
> The ROHM BU27008 is a sensor with 5 photodiodes (red, green, blue, clear
> and IR) with four configurable channels. Red and green being always
> available and two out of the rest three (blue, clear, IR) can be
> selected to be simultaneously measured. Typical application is adjusting
> LCD backlight of TVs, mobile phones and tablet PCs.
> 
> Add initial support for the ROHM BU27008 color sensor.
>  - raw_read() of RGB and clear channels
>  - triggered buffer w/ DRDY interrtupt

...

This...

> +#include <linux/bits.h>

...is guaranteed to be included by this.

> +#include <linux/bitops.h>

...

> +static const struct of_device_id bu27008_of_match[] = {
> +	{ .compatible = "rohm,bu27008", },

Inner comma is not necessary to have.

> +	{ }
> +};


-- 
With Best Regards,
Andy Shevchenko