audiohacked / OpenCorsairLink

Linux and Mac OS support for the CorsairLink Devices
GNU General Public License v2.0
706 stars 125 forks source link

Fix where to read Hydro Asetek fan speeds from #203

Closed jonasmalacofilho closed 5 years ago

jonasmalacofilho commented 5 years ago

Fixes: #142 ("H100i GT V2 fan current/max speed 0")

Proposed changes

Struct fan_ctrl has both speed and speed_rpm fields. The driver for Hydro Asetek devices stores the fan speed in the former, but we were accessing the latter.

This PR only fixes the Hydro Asetek driver. While the speed_rpm field [might simply be some leftover from legacy code](), it is still used in other drivers and, from a quick glance, possibly correctly so.

Types of changes

Checklist