Re: [PATCH 03/10] hwmon: (amc6821) Rename fan1_div to fan1_pulses

From: Quentin Schulz
Date: Mon Jul 01 2024 - 07:08:52 EST


Hi Guenter,

On 6/28/24 5:13 PM, Guenter Roeck wrote:
The chip does not have a fan divisor. What it does have is a configuration
to set either 2 or 4 pulses per fan rotation. Rename the attribute to
reflect its use. Update documentation accordingly.

Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>

Good catch!

Reviewed-by: Quentin Schulz <quentin.schulz@xxxxxxxxx>

Thanks,
Quentin