Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Allow for only setting Minimum CPU / GPU Clock(s) with no Maximum #106

Open
3 tasks done
syberphunk opened this issue May 25, 2023 · 6 comments
Open
3 tasks done
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@syberphunk
Copy link

Please confirm

  • I have searched existing issues
  • This issue is not a duplicate of an existing one
  • I will fill this out to the best of my ability

Describe what you'd like to be able to do

Steam Deck users are overclocking the GPU and CPU using tools such as https://github.com/DavidS95/Smokeless_UMAF

PowerTools allows us to configure an override to accommodate these higher values for GPU and CPU clock, but a current bug with amdgpu prevents us from setting higher than stock clock speeds.

If we set no maximum value, the Steam Deck with these overclocks in place will automatically scale upwards.

Unfortunately we cannot currently set only a 'minimum' without also setting a 'maximum' and take advantage of these overclock values.

Some games, emulators, etc. need minimum clock values to be set because the automatic scaling of its frequency may not be appropriate for the game or software that's being run, even if it won't scale higher than this minimum value.

Therefore it would be good to be able to set a minimum value, without enforcing a maximum value for the clock speeds.

It would be good to be able to set this option of 'no maximum values for clocks' in the user interface, else editing pt_oc.json would be an alternative workaround to permit it.

Describe alternatives you've considered

There's no real other alternative aside from not setting clock minimum/maximum at all per game, and this removes the ability to hold the APU accountable to run at least at a minimum speed.

Anything else?

No response

@syberphunk syberphunk added the enhancement New feature or request label May 25, 2023
@NGnius NGnius added the help wanted Extra attention is needed label May 25, 2023
@DenoG1997
Copy link

did you know a workaround for this because when I set a custom freq then it will apply as soon I get the slider under 3500 MHz

@NGnius
Copy link
Owner

NGnius commented May 26, 2023

There's always equivalent commands

@syberphunk
Copy link
Author

Having the controls via GUI is convenient and the whole purpose of having decky plugins.

@NGnius
Copy link
Owner

NGnius commented May 26, 2023

Agreed, my suggestion was strictly in the context of being a workaround. There is also a Decky plugin for running commands.

@syberphunk
Copy link
Author

Noting this as related: https://github.com/badly-drawn-wizards/vangogh_oc_fix as observed by JDGBOLT. Not suggesting this is within scope of PowerTools.

@badly-drawn-wizards
Copy link

Just chipping in to say that I have filed an issue. Although I have little hope that it will be implemented and if it does it will be a long while until it trickles down to SteamOS:
https://gitlab.freedesktop.org/drm/amd/-/issues/2638

I hope my fix becomes irrelevant as fast as possible. 🙏

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants