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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240517142807.00005b90@Huawei.com>
Date: Fri, 17 May 2024 14:28:07 +0100
From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
To: Dimitri Fedrau <dima.fedrau@...il.com>
CC: Jonathan Cameron <jic23@...nel.org>, Lars-Peter Clausen <lars@...afoo.de>,
	Andrew Hepp <andrew.hepp@...pp.dev>, Marcelo Schmitt
	<marcelo.schmitt1@...il.com>, <linux-iio@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/2] Add threshold events support

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