Skip to content
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

Add tested hardware #34

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Conversation

nicos68
Copy link
Contributor

@nicos68 nicos68 commented Nov 2, 2023

I tested the project on my Infinity Book S 14 gen 6 (from 2019) and can confirm it works.

@signaleleven
Copy link

signaleleven commented Nov 14, 2023

I have the same model. I don't see any performance profile other than "default" and I don't seem to be able to add or configure any new one from the UI.
What am I missing?

edit: I was missing a reboot :D

@nicos68
Copy link
Contributor Author

nicos68 commented Nov 14, 2023

What distro are you running ? Do you also have this issue ?

@signaleleven
Copy link

NixOS, and no, it seems to be working fine, and the fan speed reacts to the fan profile I set, at least it looks like it.
TCC had the option to disable and limit the clock of single cores, while I can't seem to do with this, but other than this (expected) difference it seems to work.

@AaronErhardt
Copy link
Owner

I think it would be interesting to add a small binary crate for checking hardware capabilities, for example "tailor-hwcaps". This crate could run some basic checks and print the information in a simple format, so we can address future issues easier. This should also help with resolving #36, because so far this matter has been rather confusing...

@nicos68
Copy link
Contributor Author

nicos68 commented Nov 14, 2023

FYI I have silently been doing tests on this is issue on my side and will have interesting info to share in the coming days.

@AaronErhardt
Copy link
Owner

I've added a small binary for checking your devices capabilities as I suggested earlier: #37

If you want you both could install it and share the output here so we can compare:

cargo install tailor_hwcaps --git https://github.com/AaronErhardt/tuxedo-rs.git --branch hwcaps

@nicos68
Copy link
Contributor Author

nicos68 commented Nov 19, 2023

So since #36 is closed I'd say this can be merged, can't it ?

@nicos68
Copy link
Contributor Author

nicos68 commented Nov 19, 2023

here is my result :

[OK]    Module version: "0.3.9\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"
[OK]    Device interface ID: "clevo_acpi\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"
[ERR]   Model ID: NotAvailable
[OK]    Available ODM performance profiles: ["quiet", "power_saving", "entertainment", "performance"]
[OK]    Default ODM performance profile: "performance"
[OK]    Number of fans: 1
[OK]    Fan temperatures [°C]: [51]
[OK]    Fan speeds [%]: [23]
[OK]    Fan min speed [%]: 20
[OK]    Webcam enabled: true
[INFO]  TDP control is not available
[OK]    Number of LED devices: 1
[OK]    LED device number: 0
[OK]    LED device name: "platform:tuxedo_keyboard"
[OK]    LED device function: "kbd_backlight"
[OK]    LED mode: Monochrome
[OK]    LED device color: Ok(Color { r: 0, g: 0, b: 0 })

@AaronErhardt
Copy link
Owner

Yes, let's merge this. Thanks!

@AaronErhardt AaronErhardt merged commit 116181f into AaronErhardt:main Nov 20, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants