Re: [External] Re: [PATCH v2] hwmon:Add MEC172x Micro Chip driver for Lenovo motherboards

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/31/23 08:14, David Ober wrote:
That was the changes from the original commit, my bad.

Do I submit the entire patch again and will it replace this, not sure how to fix this


Please

- do not top-post
- never drop the mailing list from your replies

"git rebase -i <parent>", where <parent> is the baseline SHA,
lets you edit the list of patches. Select "squash" to squash
a patch into a previous one.

Guenter


David

-----Original Message-----
From: Guenter Roeck <groeck7@xxxxxxxxx> On Behalf Of Guenter Roeck
Sent: Tuesday, October 31, 2023 10:52 AM
To: David Ober <dober6023@xxxxxxxxx>; linux-hwmon@xxxxxxxxxxxxxxx
Cc: linux-doc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; jdelvare@xxxxxxxx; corbet@xxxxxxx; David Ober <dober@xxxxxxxxxx>; Mark Pearson <mpearson@xxxxxxxxxx>
Subject: [External] Re: [PATCH v2] hwmon:Add MEC172x Micro Chip driver for Lenovo motherboards

On 10/31/23 05:09, David Ober wrote:
This addition adds in the ability for the system to scan the MEC172x
EC chip in Lenovo ThinkStation systems to get the current fan RPM
speeds and the Maximum speed value for each fan also provides the
current CPU and DIMM thermal status

Signed-off-by: David Ober <dober6023@xxxxxxxxx>

v2 fixed mixcased naming
v2 add mutex protection
v2 removed references to ACPI as it is not used
v2 added comment to explain why returning a -1 is needed
---
   drivers/hwmon/lenovo-ec-sensors.c | 81
++++++++++++++++++-------------

What is this patch based on ? I don't see that driver in drivers/hwmon/, and based on the patch I would be surprised if I had accepted it.

Guenter





[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux