Re: [PATCH] iio: light: Add driver for PixArt PAJ7620

[David Lechner]

On 4/12/26 7:09 PM, Harpreet Saini wrote:
> Signed-off-by: Harpreet Saini <sainiharpreet29@xxxxxxxxx>
> ---
> This is part 2 of a 2-patch series proposing a driver for the PixArt
> PAJ7620 gesture sensor. This patch adds the actual C driver, Kconfig 
> hooks, and Makefile integrations. The device tree bindings were submitted
> in the preceding patch.

Patches should be combined into a series with a cover letter.

The cover letter should explain clearly why this should be considered
for the IIO subsystem instead of the input subsystem. It doesn't seem
like there is a valid justification though.

> 
>  drivers/iio/light/Kconfig   |   7 +
>  drivers/iio/light/Makefile  |   1 +
>  drivers/iio/light/paj7620.c | 433 ++++++++++++++++++++++++++++++++++++
>  3 files changed, 441 insertions(+)
>  create mode 100644 drivers/iio/light/paj7620.c

There is no IIO_LIGHT channel, so this does not look at all like
a light sensor.