Re: [PATCH v2 0/2] Add threshold events support

[Jonathan Cameron]

On Fri, 17 May 2024 10:10:48 +0200
Dimitri Fedrau <dima.fedrau@xxxxxxxxx> wrote:

> Add patch for providing index for both channels and add patch for threshold
> events support for the MCP9600 device.

Always mention subsystem and device in the cover letter title!

iio: temperature mcp9600: Add threshold events support.

Makes it easier for people to judge if they want to read a thread or not.

No need to resend for this though.
> 
> Changes in V2:
>   - Remove pretty printing patches from series
>   - Add patch for providing index for both channels(ABI change)!
>     Suggested by Jonathan, hope this okay.
>   - Remove formatting in a precursor patch
>   - Add lock documentation
>   - Add define MCP9600_TEMP_SCALE_NUM and use it instead of MICRO. MICRO is
>     type unsigned long which we have to cast to int when using
>     multiplication or division, because we are handling negative values.
>   - Use switch statement in mcp9600_write_thresh
>   - Replaced generic interrupt handler with four separate interrupt handler
>   - Use one lock instead of four
>   - Added error check for mcp9600_probe_alerts
>      
> 
> Dimitri Fedrau (2):
>   iio: temperature: mcp9600: Provide index for both channels
>   iio: temperature: mcp9600: add threshold events support
> 
>  drivers/iio/temperature/mcp9600.c | 398 +++++++++++++++++++++++++++++-
>  1 file changed, 396 insertions(+), 2 deletions(-)
>