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

[Matyas, Daniel]

-----Original Message-----
From: Guenter Roeck <groeck7@xxxxxxxxx> On Behalf Of Guenter Roeck
Sent: Monday, May 22, 2023 4:25 PM
To: Matyas, Daniel <Daniel.Matyas@xxxxxxxxxx>
Cc: kernel test robot <lkp@xxxxxxxxx>; Jean Delvare <jdelvare@xxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Jonathan Corbet <corbet@xxxxxxx>; linux-hwmon@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-doc@xxxxxxxxxxxxxxx
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@xxxxxxxxxx>
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Link: 
> https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/2023
> 05112351.DBkFfs76-lkp@xxxxxxxxx/__;!!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