[PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller

[PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller

[Balac, Arun Saravanan]

From: Arun Saravanan Balachandran <Arun_Saravanan_Balac@dell.com>

Add driver for MAX6620 Fan controller

Arun Saravanan Balachandran (2):
  Add attributes in MAX6620 driver to retrieve fan fault status
  Update MAX6620 driver as per linux code guidelines

Cumulus Networks (2):
  Driver for MAX6620 Fan sensor
  MAX6620 fix rpm calculation accuracy
 
Shuotian Cheng (1):
  Update MAX6620 driver to support newer kernel version

 drivers/hwmon/Kconfig   |  10 +
 drivers/hwmon/Makefile  |   1 +
 drivers/hwmon/max6620.c | 625 ++++++++++++++++++++++++++++++++++++++++
 3 files changed, 636 insertions(+)
 create mode 100644 drivers/hwmon/max6620.c


base-commit: 303392fd5c160822bf778270b28ec5ea50cab2b4
-- 
2.32.0

Re: [PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller

[Guenter Roeck]

On 7/6/21 6:01 AM, Balac, Arun Saravanan wrote:
> From: Arun Saravanan Balachandran <Arun_Saravanan_Balac@dell.com>
> 
> Add driver for MAX6620 Fan controller
> 
> Arun Saravanan Balachandran (2):
>    Add attributes in MAX6620 driver to retrieve fan fault status
>    Update MAX6620 driver as per linux code guidelines
> 
> Cumulus Networks (2):
>    Driver for MAX6620 Fan sensor
>    MAX6620 fix rpm calculation accuracy
>   
> Shuotian Cheng (1):
>    Update MAX6620 driver to support newer kernel version
> 
>   drivers/hwmon/Kconfig   |  10 +
>   drivers/hwmon/Makefile  |   1 +
>   drivers/hwmon/max6620.c | 625 ++++++++++++++++++++++++++++++++++++++++
>   3 files changed, 636 insertions(+)
>   create mode 100644 drivers/hwmon/max6620.c
> 
> 
> base-commit: 303392fd5c160822bf778270b28ec5ea50cab2b4
> 
Please squash all patches into one, and please use
devm_hwmon_device_register_with_info to register the hardware
monitoring device.

Also please follow the guidelines in
Documentation/hwmon/submitting-patches.rst.

Thanks,
Guenter

RE: [PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller

[Balac, Arun Saravanan]

Thank you for the review comments.
Am currently working on the suggested changes and will submit the same for your review.

Request your insights on the recommended method to include the author information of the earlier patches,
in the commit message while squashing them into one.

Regards,
Arun Saravanan

-----Original Message-----
From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck
Sent: Tuesday, July 6, 2021 6:42 PM
To: Balac, Arun Saravanan; jdelvare@suse.com
Cc: linux-hwmon@vger.kernel.org
Subject: Re: [PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller


On 7/6/21 6:01 AM, Balac, Arun Saravanan wrote:
> From: Arun Saravanan Balachandran <Arun_Saravanan_Balac@dell.com>
> 
> Add driver for MAX6620 Fan controller
> 
> Arun Saravanan Balachandran (2):
>    Add attributes in MAX6620 driver to retrieve fan fault status
>    Update MAX6620 driver as per linux code guidelines
> 
> Cumulus Networks (2):
>    Driver for MAX6620 Fan sensor
>    MAX6620 fix rpm calculation accuracy
>   
> Shuotian Cheng (1):
>    Update MAX6620 driver to support newer kernel version
> 
>   drivers/hwmon/Kconfig   |  10 +
>   drivers/hwmon/Makefile  |   1 +
>   drivers/hwmon/max6620.c | 625 ++++++++++++++++++++++++++++++++++++++++
>   3 files changed, 636 insertions(+)
>   create mode 100644 drivers/hwmon/max6620.c
> 
> 
> base-commit: 303392fd5c160822bf778270b28ec5ea50cab2b4
> 
Please squash all patches into one, and please use
devm_hwmon_device_register_with_info to register the hardware
monitoring device.

Also please follow the guidelines in
Documentation/hwmon/submitting-patches.rst.

Thanks,
Guenter

Re: [PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller

[Guenter Roeck]

On 7/26/21 12:47 AM, Balac, Arun Saravanan wrote:
> Thank you for the review comments.
> Am currently working on the suggested changes and will submit the same for your review.
> 
> Request your insights on the recommended method to include the author information of the earlier patches,
> in the commit message while squashing them into one.
> 

Either "based on ..." in the commit description or, more formally, "Originally-from:".

Guenter

> Regards,
> Arun Saravanan
> 
> -----Original Message-----
> From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck
> Sent: Tuesday, July 6, 2021 6:42 PM
> To: Balac, Arun Saravanan; jdelvare@suse.com
> Cc: linux-hwmon@vger.kernel.org
> Subject: Re: [PATCH 0/5] hwmon: Add driver for MAX6620 Fan controller
> 
> 
> On 7/6/21 6:01 AM, Balac, Arun Saravanan wrote:
>> From: Arun Saravanan Balachandran <Arun_Saravanan_Balac@dell.com>
>>
>> Add driver for MAX6620 Fan controller
>>
>> Arun Saravanan Balachandran (2):
>>     Add attributes in MAX6620 driver to retrieve fan fault status
>>     Update MAX6620 driver as per linux code guidelines
>>
>> Cumulus Networks (2):
>>     Driver for MAX6620 Fan sensor
>>     MAX6620 fix rpm calculation accuracy
>>    
>> Shuotian Cheng (1):
>>     Update MAX6620 driver to support newer kernel version
>>
>>    drivers/hwmon/Kconfig   |  10 +
>>    drivers/hwmon/Makefile  |   1 +
>>    drivers/hwmon/max6620.c | 625 ++++++++++++++++++++++++++++++++++++++++
>>    3 files changed, 636 insertions(+)
>>    create mode 100644 drivers/hwmon/max6620.c
>>
>>
>> base-commit: 303392fd5c160822bf778270b28ec5ea50cab2b4
>>
> Please squash all patches into one, and please use
> devm_hwmon_device_register_with_info to register the hardware
> monitoring device.
> 
> Also please follow the guidelines in
> Documentation/hwmon/submitting-patches.rst.
> 
> Thanks,
> Guenter
>