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

Fan curve adjustable from OS (Linux) #424

Closed
macpijan opened this issue Apr 24, 2023 · 4 comments
Closed

Fan curve adjustable from OS (Linux) #424

macpijan opened this issue Apr 24, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request novacustom_ns5x/7x_adl NovaCustom NS5x/7xPU (12th Gen) novacustom_ns5x/7x_tgl NovaCustom NS5x/7xMU (11th Gen) novacustom_ns7x_tgl NovaCustom NS7xMU (11th Gen) novacustom_nv4x_adl NovaCustom NV4xPZ (12th Gen) novacustom_nv4x_tgl NovaCustom NV4xMx (11th Gen)

Comments

@macpijan
Copy link
Contributor

The problem you're addressing (if any)

The fan curve is not adjustable from OS level. It requires recompliation of EC firmware, or selecting one of the profiles ine the firmware menu: https://docs.dasharo.com/unified/novacustom/fan-profiles/

Describe the solution you'd like

I'd like that the fan curve can be configured from Linux, so I can use sysfs programming interface in my scripts, or GUI applications, to control the fan curve.

Where is the value to a user, and who might that user be?

Describe alternatives you've considered

None

Additional context

Some work was already done here by community contributor: pop-os/system76-acpi-dkms#9 (comment)
It seems it lacked testing and review, and maybe we could step in here with some support.

@macpijan macpijan added novacustom_nv4x_tgl NovaCustom NV4xMx (11th Gen) novacustom_ns5x/7x_tgl NovaCustom NS5x/7xMU (11th Gen) novacustom_ns7x_tgl NovaCustom NS7xMU (11th Gen) novacustom_ns5x/7x_adl NovaCustom NS5x/7xPU (12th Gen) novacustom_nv4x_adl NovaCustom NV4xPZ (12th Gen) labels Apr 24, 2023
@macpijan macpijan added the enhancement New feature or request label Apr 24, 2023
@wessel-novacustom
Copy link

No need from our side for this feature. Users can set the fan curve they like in the UEFI firmware settings. Moreover, they can temporarily switch to 100% fan speed with the shortcut Fn + 1. I think that is enough :-)

@BeataZdunczyk BeataZdunczyk closed this as not planned Won't fix, can't repro, duplicate, stale Apr 24, 2024
@tlaurion
Copy link

@wessel-novacustom but if not exposed through kernel (again system76_acpi driver I guess) then Heads will not be able to permit to tweak those settings?

@wessel-novacustom
Copy link

@wessel-novacustom but if not exposed through kernel (again system76_acpi driver I guess) then Heads will not be able to permit to tweak those settings?

I think that is correct. Ping @macpijan

So we need #820

@macpijan
Copy link
Contributor Author

macpijan commented Apr 25, 2024

Yes, please see my comment here: #820 (comment)

It is challenging to please Linux, Windows, and now also heads users with a single firmware base. We were aware together with @wessel-novacustom of the fact, that some features implemented in the edk2 (UEFI firmware variant) will not be available in the heads firmware variant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request novacustom_ns5x/7x_adl NovaCustom NS5x/7xPU (12th Gen) novacustom_ns5x/7x_tgl NovaCustom NS5x/7xMU (11th Gen) novacustom_ns7x_tgl NovaCustom NS7xMU (11th Gen) novacustom_nv4x_adl NovaCustom NV4xPZ (12th Gen) novacustom_nv4x_tgl NovaCustom NV4xMx (11th Gen)
Projects
None yet
Development

No branches or pull requests

5 participants