ValveSoftware / SteamOS

SteamOS community tracker
1.57k stars 70 forks source link

Expose CPU clock speed limits in PM firmware #1309

Open badly-drawn-wizards opened 9 months ago

badly-drawn-wizards commented 9 months ago

Following the discussion in this issue: https://gitlab.freedesktop.org/drm/amd/-/issues/2638

The amdgpu drivers for vangogh currently hard code the maximum clock frequency. If a steam deck is overclocked, then the AMD PowerPlay interface can only be set up to this hard coded limit, rather than the true maximum clock.

Steps to reproduce:

Alternatively on a steam deck you can directly use the PowerPlay interface through the linux sysfs and the same will occur.

I am not familiar with the full details so I apologize if I haven't got the precise blocker for amdgpu on this side.

NGnius commented 9 months ago

Since PowerTools is mentioned I'm poking my head here to offer moral support and answer any (relevant) PowerTools-related questions people might have. I'm also sort of familiar with how the sysfs interfaces work since I've had to interact with them.

RodoMa92 commented 9 months ago

Having this fixed should also fix #1243. I hope that the firmware team will respond to this multiple requests for fixing broken firmware stuff, but I never seen them around here as of today.

iTsMute commented 4 months ago

Also running into this issue. Please fix

wantija commented 4 months ago

would love if this eventually gets fixed.

naanlizard commented 1 month ago

This is really annoying on my end, would love a fix

justEhCupcake commented 1 month ago

Would love to have this limitation removed for those who want more out of their hardware and are willing to accept the risks

such a frustrating issue

Skyrent commented 1 month ago

Please fix and let us pass through this limitation.

Stormageddon0 commented 4 weeks ago

Please fix

Dr1fte commented 3 weeks ago

This issue does need to be resolved.

Valve, why allow us (consumers) to overclock and undervolt? What’s the entire point of owning this system. If us consumers, can’t actually overclock to see our own steam decks limits? The difference it can make being able to achieve an additional 500mhz on the cpu side. But yet. We can set these values in the bios. But unable to actually achieve them? Sounds like a bad business decision, to allow this issue to go unnoticed for so long.

Please fix this issue before releasing another stable build. Release it to the next beta build, so us enthusiasts can test. It would be greatly appreciated.

gandalfthegreet commented 1 week ago

Please fix this we want more POWER✊