Re: [RFC PATCH 0/2] Support for Avago APDS9306 Ambient Light Sensor

Subhajit Ghosh
2023-04-12T04:47:07Z

Hi Jonathan,
Thank you for getting back.

> Hi Subhajit,
>
> No need to sign off a cover letter. The content isn't captured in the
> git tree anyway.
>
> For an RFC, I'd expect to see a clear statement in the cover letter of
> why it is an RFC rather than a formal patch submission. What specifically
> are you looking for comments on?
>
> Point us in the right direction and we might answer the questions quicker.
>
> Thanks,
>
> Jonathan
Thank you for clearing it up.
Next version of RFC I will put specific reasons.
Before submitting a formal patch I wanted to check if my implementation of
single reads of ALS data raw values from userspace when interrupts are
enabled is the right thing to do or not. Also wanted to check if my event
related userspace ABI implementation is in line with IIO subsystem.
I will put it into better words in the next cover letter.

Can you also help me out with the git tree I should use to format the
patches? As per my understanding it is the subsystem maintainer tree
and the main branch but the macros and functions which you have suggested
in other reviews are available in Linux mainline.

Regards,
Subhajit Ghosh