Re: [v2 3/3] hwmon: Add Aspeed ast2600 TACH support

From: Guenter Roeck
Date: Thu Nov 03 2022 - 00:32:39 EST


On Thu, Nov 03, 2022 at 03:52:59AM +0000, Billy Tsai wrote:
>
> Can't I use a min/max RPM to let the driver know a reasonable timeout/polling period when
> the driver is trying to get RPM?
> Beacause that our tach controller have the falg to indicates the hardware detected the change
> in the input signal. I need the proper timout to rule out slow RPMs.
>

If the chip measures the fan speed continuously, why would that ever be a
problem, and why wait in the first place instead of just taking the most
recent result ?

Pretty much every other driver is doing that, so I really don't understand
why that would not work here.

Guenter