-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Thelio R2 listed as unsupported #5
Comments
Found the line in main.rs limiting it to Mira. |
(if you do not use 264b41a your line numbers may differ) Thanks for details... 😐 |
Didn’t add the details because I wasn’t sure if this would be a good thing or not. If the fan curves are specific for the Mira, it might have different impacts on the other cases that have different cooling characteristics. @jackpot51 or someone from System76 should weigh in there. |
The fan curve seems identical to the Linux driver, so while I would be worried about other things... not so much the fan profile. As far as I'm aware there's only the one I/O board, identical between models? Would be nice to get some official word though. |
Oh I should mention there are other fan profiles defined, such as the HEDT? Can we keep this issue open? It's still outstanding as they have a bunch of thelio models and this driver only has the one in it. |
Agreed, reopening. If nothing else, to get some official guidance. |
I've done a little digging and the only model-specific stuff I can find in the Linux daemon is the fan profile. You can see the model-to-profile mapping here and the fan profiles here If they're open to PRs, I could actually go and make sure the fan curves are identical and add the missing entries? Though I can't actually test anything myself, yet. Are you willing to give it a shot, if I forked and added the change? Interesting to note that on the Linux driver they just fail to the standard curve instead of bailing out entirely. Also maybe interesting to see that the mira is absent over there? In the very least though this tells you which profile to use for your hardware 🤣 |
I already moved back to Pop_OS full time on the Thelio I have given Proton's to the point that I no longer need Windows on this desktop, otherwise I'd be willing to test myself. FWIW, I just removed the model check and the Mira profile's didn't seem to cause my system to run hotter than usual, which is saying a lot with a Ryzen 3900XT and a RX 5700XT in the Thelio. 😄 |
I received my hardware today, so I should be able to play with this myself soonish :) |
I had a different system/version string (see #7) and had to bump the rust compiler version to use the syntax that allowed a minimal code change... But anyway, it worked. If you want to see what that looks like, the commit is on my fork currently. Assuming a positive result from #7 I plan to submit a PR. |
I realize this is a WIP, is Thelio R2 supported? Fails on service start:
From Application event log:
unsupported sys_vendor 'System76' and product_version 'thelio-r2'
Custom {
kind: Other,
error: "unsupported sys_vendor 'System76' and product_version 'thelio-r2'",
}
From System Event Log:
The System76 Thelio Io service terminated unexpectedly. It has done this 3 time(s).
The text was updated successfully, but these errors were encountered: