PowerStation icon indicating copy to clipboard operation
PowerStation copied to clipboard

Published 20 hours ago • verified publisher icon ShadowBlip

Use asusctl to change TDP of supported devices

Open Radiicall opened this issue 10 months ago • 4 comments

Enables the use of asusctl to change TDP on supported devices (e.g. the ROG Ally)

Radiicall avatar Apr 09 '24 07:04 Radiicall

What all needs doing for this PR to move to ready?

pastaq avatar May 02 '24 02:05 pastaq

What all needs doing for this PR to move to ready?

I'm having issues setting TDP on latest unstable release of chimera and havent had time to look into why, i thought it may have been related to ryzenadj but even after reverting the thermal throttle changes it didnt change anything

Radiicall avatar May 02 '24 08:05 Radiicall

Is this functionality something still being considered? I know i havent looked at this in a while but i could probably get around to it

Radiicall avatar Aug 07 '24 20:08 Radiicall

Yeah, I meant to ask about it. Some of the endpoints will actually be exposed in sysfs soon, so that might be a better course. Also, we should add the platform_profile from ACPI and include safety checks to ensure if TDP is adjusted above/below certain thresholds that the platform_profile is automatically set.

pastaq avatar Aug 08 '24 04:08 pastaq

As we can use the interfaces provided by asusctl directly and it reduces the software requirements, #12 is a better overall solution. It also solves the fallback issue by making the AMDGPU TDPDevice interface separate from RyzenAdj and Asus WMI interfaces.

pastaq avatar Sep 08 '24 03:09 pastaq