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

Add Helios 300 PH315-52-748u to supported devices #1

Closed
RuiGuilherme opened this issue Jun 6, 2021 · 6 comments
Closed

Add Helios 300 PH315-52-748u to supported devices #1

RuiGuilherme opened this issue Jun 6, 2021 · 6 comments

Comments

@RuiGuilherme
Copy link

RuiGuilherme commented Jun 6, 2021

Kernel: 5.12.9-arch1-1
OS: Arch Linux x86_64
Device: Acer Predator Helios 300 PH315-52-748u

  1. All mode works fine.
  2. Speed control works fine
  3. Brightness control works fine (but has one issue with mode).
  4. Direction control works fine
  5. cR, cG and cB works fine

I just had one issue:
If you set keyboard brightness using -b the mode will be set to 3 automatically. (Control brightness using Fn + F9/F10 solve this)

sudo dmidecode | grep "Product Name" -B 2 -A 4

System Information
        Manufacturer: Acer
        Product Name: Predator PH315-52
        Version: V1.12
        Serial Number: NHQ62AL001016D78C99501
        UUID: f87dea2b-75a4-471d-ae8a-586741a05036
        Wake-up Type: Power Switch
--
Base Board Information
        Manufacturer: CFL
        Product Name: Covini_CFS
        Version: V1.12
        Serial Number: NBQ5311004016018DE9501
        Asset Tag: Type2 - Board Asset Tag
        Features:

sudo cat /sys/firmware/acpi/tables/DSDT > dsdt.aml

dsdt.aml.zip (Github don't accept .aml file)

@JafarAkhondali
Copy link
Owner

Thank for the repor @RuiGuilherme, glad it worked

I understand the problem with brightness, that's because the script is minimal and doesn't store\use the state.
Actually I was hoping to merge the changes into Linux kernel, but some required functionalities for implementations are missing so I'm not sure to wait for them, or develop this repo.

@JafarAkhondali
Copy link
Owner

JafarAkhondali commented Aug 15, 2021

@RuiGuilherme I've added Turbo mode support as well, can you kindly please test the updated version? (at your own risk 😆 )

@RuiGuilherme
Copy link
Author

RuiGuilherme commented Aug 15, 2021

Nice! Apparently works fine and the fans use the maximum speed.

Later I will check if the UnderVolt is being removed and the overclock is being applied.

@RuiGuilherme
Copy link
Author

RuiGuilherme commented Aug 16, 2021

Strange because this notebook should have an undervolt of -0.125v; Maybe I'm going crazy?

Using turbo mode:

# sudo intel-undervolt read
CPU (0): -0.00 mV
GPU (1): -0.00 mV
CPU Cache (2): -0.00 mV
System Agent (3): -0.00 mV
Analog I/O (4): -0.00 mV

Normal mode:

# sudo intel-undervolt read
CPU (0): -0.00 mV
GPU (1): -0.00 mV
CPU Cache (2): -0.00 mV
System Agent (3): -0.00 mV
Analog I/O (4): -0.00 mV

This application may not read my CPU status correctly as it hasn't been updated in two years: intel-undervolt

@JafarAkhondali
Copy link
Owner

Probably because it hasn't received the update. I think our CPU was build after the last commit of that repository.
btw I think turbo for helios 300 works only on GPU (not sure)

@RuiGuilherme
Copy link
Author

RuiGuilherme commented Aug 16, 2021

Hup! You are right. Just overclock GPU, but at the time I bought this notebook it came with Windows, and it had an undervolt on the i7 9750H and Turbo Mode "Disable" this undervolt.

I don't remember the exact values because I bought this notebook in 2020, but I had used this thead to reverse the undervolt without using turbo mode.
https://www.techpowerup.com/forums/threads/undervolt-an-i7-9750h-with-throttlestop.268380/

But it's like I said, maybe I'm going crazy haha

Edit:
I'm going to use a Windows To-Go to see this issue and confirm it.

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