-
Notifications
You must be signed in to change notification settings - Fork 63
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
Legion Pro 5 2023 ( R9000P 2023 in China ) support #71
Comments
Here is ACPI tables and WMI entries: In the WMI info, |
@huyz-git Then, please test the following features. We have to try a bit to find the correct configuration. Temperatures and Fan SpeedsPlease read the fan speeds and temperatures (see README) and confirm that they are sensible and change accordingly, e.g. when you put load onto the CPU the CPU temp and the fan speeds should increase, but GPU temp should stay roughly the same. PowermodeWe will start by confirming that power mode change work and to get me more debug info. Please change the power mode with Fn+Q and provide the output of the following commands in each mode. sudo cat /sys/kernel/debug/legion/ecmemoryram | hexdump -C
sudo cat /sys/kernel/debug/legion/ecmemory | hexdump -C
sudo cat /sys/kernel/debug/legion/fancurve Then please also check if the power mode is read correctly in the OS (see README). Then please check if you can set the power mode with the GUI (see README) and the LED changes accordingly. Tests the following features with the GUI. Please check that if works or otherwise give precise information about what does not work. Please try to check that reading and writing works with the software. Please provide the log output from the GUI from the log tab for all the tests. Fn Lock
Win Key
Touchpad
Battery Conservation
Rapid Charging
Charge Output Always On
Display Overdrive
Keyboard BacklightIn order to fix/configure the keyboard backlight and begin with setting fan curves, please put your keyboard backlight to off, medium, or bright. Provide the output of the following commands (in that order) in each backlight state: sudo cat /sys/kernel/debug/legion/fancurve
sudo dmesg | tail -n 30 Fan curveFirst, please give again the output of sudo cat /sys/kernel/debug/legion/fancurve
sudo dmesg | tail -n 30 Second, please use the GUI and enter custom mode. Then change the fan curve (currently only the first row for fan 1), write it to hardware, and read it back. Please report back if it works and provide the log output from the GUI and |
PowermodeThis tarball contains the output of these 3 files at each powermodes: I tested these power modes in stress test, changing power mode to 3 default modes works, and the power limits do changes accordingly. |
Keyboard Backlight |
Fan curveThe "Lock fan controller" option works, the fan can keep at high speed after ending stress test, and return to low speed as soon as I unticked this option.
Fan speed can be read correctly from hw, If I change the fan speed curve in Windows, this module can correctly read these changes. |
Sorry for the crash. I have to develop it without access to the newest models. Please try the newest version again and test the fan curve. I guess it was an array accessed out of bounds. Temperatures: I guess the temperatures for the GPU are a little different because these are different sensors or calibration. There is also a difference on my model (e.g. 35 degrees vs 37 degrees). Greyed out options: Some of them are provided by another driver (idepad)? Maybe in the newer models it does not load or the firmware changes. Is the idepad module loaded (output of Power limits: It's good that changing the power modes work. We can later troubleshoot the power limits. I have not configured/changes the driver for the newest models, yet. Let's do this later and first focus on fan curve etc. Maximum Fan Speed: I have to adapt it to the newest model. We can test it later after I change it. First try to fix the other problems. Y-Logo: Can you turn it on off with Ctrl+U or Ctrl+L? Please turn it off and give the output of |
With the latest commit, changing fan curve works. If I set all the values to 1000 and apply to HW, the fan speed reaches maximum on idle state. I blacklisted For power limits, currently For Y logo, I can turn it on and off with Fn+L. Here is the output:
Y-Logo on:
It seems that these two outputs are exactly the same (except for temperatures). |
For power modes, I do some stress testing, and find out that, If I set power mode to custom mode with this tool, it actually change the power limits to the same as balance mode ( in my model, this mode has 105 watts of cpu maximum power, and 55 watts cpu cross loading power, which is the same as the result of my stress test ). Also, applying fan curve will also change power limits in custom mode. Also, there is a bug with |
Regarding ideapad driver: You could try starting it with the parameter (https://wiki.archlinux.org/title/Kernel_parameters) ideapad_laptop.no_bt_rfkill=0 or ideapad_laptop.no_bt_rfkill=1 (no idea what what setting is the correct one) to fix the problem with the bluetooth/wifi. |
@huyz-git what features are working and not working with the latest version? |
Problem Descripition
This model come with up to AMD Ryzen 7945HX processor and up to NVIDIA 4070 GPU.
By default
legion-laptop
module will not loaded in this model:If I force load it with parameter, it also failed to load due to EC version mismatch:
This is because if the module is forced to load, it will only use the first model. So I modify the source code, and move
KWCN
model to the first one, and then force load it, it finally loads, but with many errors:Model and Debug Info
Distribution: Arch (including version, e.g. Ubuntu 24.04)
Model name: Legion Pro 5 2023 ( R9000P in China )
CPU model: AMD Ryzen 9 7945HX
GPU model: NVIDIA RTX 4060
Keyboard backlight: single color with off/medium/bright
Light in lid or logo: yes
Light at IO-Ports at back: no, only power cable has a light
Output of
sudo dmidecode -t system
. Please remove Serial Number and UUID for privacy:Output of
sudo dmidecode -t bios
:Output of
sudo cat /sys/kernel/debug/legion/fancurve
:The text was updated successfully, but these errors were encountered: