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

document support for x1e4 #318

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kylebakerio
Copy link

documenting that this is working for me. I've commented on an issue the basis for my experience that it's working.

of note: I didn't try undervolting, as I know that isn't supported on my platform.

also: if you have any steps you'd like me to take to confirm this is working before merging this request, let me know.

documenting that this is working for me. I've commented on an issue the basis for my experience that it's working.

of note: I didn't try undervolting, as I know that isn't supported on my platform. 

also: if you have any steps you'd like me to take to confirm this is working before merging this request, let me know.
@kylebakerio
Copy link
Author

I did some benchmarking and was able to observe > 60 watts sustained on a 30 minute benchmark. 80+ watts on spikes. scores on all cpu benchmarks went up substantially.

@kylebakerio
Copy link
Author

Also, worth noting: in your readme, you describe specific watts, but since this is now generalizable to many platforms, those don't apply. I uset his tool to set PL1 and PL2 to 100w (like modern motherboard manufacturers) to basically force my system to limit itself only according to thermals, which removes artificial and unnecessary performance limitations.

@kylebakerio
Copy link
Author

kylebakerio commented Dec 7, 2022

Also worth noting: settings these much lower than factory allows me to get excellent battery performance as well.

Will be testing on X1E5 soon! This project is excellent, does mainlining some of this stuff into kernel make sense? Maybe a GUI?

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.

1 participant