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

Jonathan Cameron
2024-05-17T13:28:12Z

On Fri, 17 May 2024 10:10:48 +0200
Dimitri Fedrau <dima.fedrau@gmail.com> 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(-)
>