On 11/25/21 5:14 AM, Eugene Shalygin wrote:
Dear Andy, Denis, and Günter, Denis worked on my code with the first attempt to read EC sensors from ASUS motherboards and submitted it as a driver named "asus_wmi_ec_sensors", which is not in hwmon-next. Now he adds the ACPI lock feature to support other motherboards, and I have another iteration of the EC sensor driver under development (needs some polishment) that utilizes the same concept (ACPI lock instead of WMI methods), which is smaller, cleaner and faster than the WMI-based one. I'm going to submit it to the mainline too. I think it should replace the WMI one. In anticipation of that, can we change the name of the accepted driver (asus_wmi_ec_sensors -> asus_ec_sensors) now, in order to not confuse users in the next version and to remove implementation detail from the module name? The drivers provide indistinguishable data to HWMON.
Two drivers with the same functionality ? What would be the benefit of that ? Why not drop the current one entirely instead ? Guenter
