search

system76 / thelio-io-windows

WIP Windows driver for System76 Thelio Io
GNU General Public License v3.0
16 stars 8 forks source link

Mira r2 support #13

Open pyrrho opened 2 years ago

pyrrho commented 2 years ago

The most important change in this update is the addition of the "thelio-mira-r2" product version identifier to the switch statement that selects a fan curve.

The other significant change is bumping the toolchain version to stable to keep this project in-line with the system76-power repository. The addition of the #[must_use] decorator to FanCurve::append was added for the same reason.

The rest of the changes will hopefully make the comments and code more readable, but are entirely non-functional.

kundor commented 1 year ago

Thanks, I updated this to also match "thelio-mira-r3" and it worked on my system.

dsnopek commented 1 year ago

I can't personally vouch for all the other changes in this PR, so I made a simpler one that just adds support for thelio-mira-r2 and thelio-mira-r3:

https://github.com/system76/thelio-io-windows/pull/18