On Wednesday 07 March 2018 07:50 PM, Guenter Roeck wrote:
On 03/07/2018 01:47 AM, Rajkumar Rampelli wrote:
I am planning to add tachometer support in pwm-fan.c driver
(drivers/hwmon/) instead of adding new generic-pwm-tachometer.c
driver. Measuring RPM value will be done in pwm-fan driver itself
using pwm capture feature and will add new sysfs attributes under
this driver to report rpm value of fan.
While I am not opposed to ABI changes, the merits of those would
need to be
discussed on the mailing list. But replacing "fan1_input" with "rpm" is
not an acceptable ABI change, even if it may measure something that
but isn't a fan.
If this _is_ in fact supposed to be used for something else but
would have to discuss what that might be, and if hwmon is the
subsystem to measure and report it. This does to some degree lead
my concern of having the "fan" part of this patch series in the pwm
I am still not sure if that makes sense.
There is an existing attribute to report the RPM of fans. It is called
"replacing "fan1_input" with "rpm" is not an acceptable ABI change"
The RPM is measured speed via PWM signal capture which is output from fan.
So should we have the fan[1..n]_output_rpm?
To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html