linux-kernel - RE: [PATCH v9 2/2] hwmon: max31827: add MAX31827 driver

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: <PH0PR03MB6771D471E1AF58B0B50FE4D489439@PH0PR03MB6771.namprd03.prod.outlook.com>
Date:   Mon, 22 May 2023 14:18:11 +0000
From:   "Matyas, Daniel" <Daniel.Matyas@...log.com>
To:     Guenter Roeck <linux@...ck-us.net>
CC:     kernel test robot <lkp@...el.com>,
        Jean Delvare <jdelvare@...e.com>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Jonathan Corbet <corbet@....net>,
        "linux-hwmon@...r.kernel.org" <linux-hwmon@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>
Subject: RE: [PATCH v9 2/2] hwmon: max31827: add MAX31827 driver



-----Original Message-----
From: Guenter Roeck <groeck7@...il.com> On Behalf Of Guenter Roeck
Sent: Monday, May 22, 2023 4:25 PM
To: Matyas, Daniel <Daniel.Matyas@...log.com>
Cc: kernel test robot <lkp@...el.com>; Jean Delvare <jdelvare@...e.com>; Rob Herring <robh+dt@...nel.org>; Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>; Conor Dooley <conor+dt@...nel.org>; Jonathan Corbet <corbet@....net>; linux-hwmon@...r.kernel.org; devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; linux-doc@...r.kernel.org
Subject: Re: [PATCH v9 2/2] hwmon: max31827: add MAX31827 driver

[External]

On Tue, May 16, 2023 at 01:46:07PM +0300, Daniel Matyas wrote:
> MAX31827 is a low-power temperature switch with I2C interface.
> 
> The device is a ±1°C accuracy from -40°C to +125°C
> (12 bits) local temperature switch and sensor with I2C/SM- Bus 
> interface. The combination of small 6-bump wafer-lev- el package (WLP) 
> and high accuracy makes this temper- ature sensor/switch ideal for a 
> wide range of applications.
> 
> Signed-off-by: Daniel Matyas <daniel.matyas@...log.com>
> Reported-by: kernel test robot <lkp@...el.com>
> Link: 
> https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/2023
> 05112351.DBkFfs76-lkp@...el.com/__;!!A3Ni8CS0y2Y!5SeqSdRyCHcgA6Se6U-EN
> I_MJfRPgB-FEVkuwWyb944DQypaZ5XQAS6N3aZiVDd1LZDGK45xp7KJHH8RSzw$
> ---

Change log goes here. Without it, I'll have to manually re-review and compare against previous versions and look up comments to ensure that all feedback has been addressed. That will take time, which is always
scarce.   don't have right now. My apologies, but that means that
review and acceptance of your driver will be delayed.

Guenter


Dear Guenter,

I didn't write a changelog, because I changed literally nothing in the code. I sent a new patch, because in the previous Krzysztof Kozlowski asked me to remove the '--base' when using 'git format-patch'. I wrote this to commit 1/2, but consider it necessary to write the same thing again in patch 2/2.

I have the Reported-by flag, because I forgot to add it in patch v8.

I am sorry for the confusion!

Best regards,
Daniel Matyas