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

plugin not working on legion go with catchyOS #55

Open
2 tasks done
ppodevlabs opened this issue Jan 29, 2025 · 1 comment
Open
2 tasks done

plugin not working on legion go with catchyOS #55

ppodevlabs opened this issue Jan 29, 2025 · 1 comment

Comments

@ppodevlabs
Copy link

ppodevlabs commented Jan 29, 2025

Is there an existing issue for this?

  • I have searched the existing issues

Have you updated to the lateset version of the plugin?

  • I am on the latest version of the plugin

Bug Description

Plugin seems to not be working fine on my legion go with catchyOS. I've installed ryzenadj

❯ which ryzenadj
/usr/bin/ryzenadj

i tested ryzenadj is working fine with

❯ sudo ryzenadj --stapm-limit 15000 --slow-limit 15000 --fast-limit 15000
Sucessfully set stapm_limit to 15000
Sucessfully set fast_limit to 15000
Sucessfully set slow_limit to 15000

But when in gaming mode and using the plugin, TDP limits seems to not take effect as i do see my legion go going over the TDP limit i set of 30w and per game configurations are ignored, causing the legion go to turn off.

Log does not show any erors, see below

am I missing anything?

Steps To Reproduce

2025-01-29 10.58.42.log

Hardware information

Legion Go

- Distro: CachyOS
- CPU: AMD Ryzen Z1 Extreme
- GPU: AMD

Logs, screenshots, or anything else

No response

@aarron-lee
Copy link
Owner

For the Legion Go specifically, you actually need the acpi_call module for to correctly set TDP via Lenovo's WMI calls.

TDP control on the Legion Go is not possible with ryzenadj.

There is a work-in-progress driver for the Legion Go + Legion Go S that will formally add TDP controls without workarounds such as acpi_call, but I'll need to update the plugin once that driver is formally available.

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

No branches or pull requests

2 participants