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

ec/system76/ec: Add config for 2nd fan without dGPU #222

Merged
merged 2 commits into from
Jul 17, 2024
Merged

Conversation

crawfxrd
Copy link
Member

@crawfxrd crawfxrd commented May 1, 2024

The darp10 has a second fan but no dGPU. The NFAN Method must exist, so use the default hwmon names of "fan1" and "fan2" for labels.

The RPM calculation depends on the EC configuration. The EC is now responsible for the calculation, and the RAM values have the actual RPM values.

Requires: system76/ec#474

@crawfxrd crawfxrd marked this pull request as ready for review May 9, 2024 15:06
@crawfxrd crawfxrd requested review from a team May 9, 2024 15:06
This calculation depends on the EC configuration. The EC is now
responsible for the calculation, and the RAM values have the actual RPM
values.

Change-Id: Ifabbcb5922643e218ed7c4984b1d41aee841bb39
Signed-off-by: Tim Crawford <[email protected]>
The darp10 has a second fan but no dGPU. The NFAN Method must exist, so
use the default hwmon names of "fan1" and "fan2" for labels.

Change-Id: Icde5dec82262d9262b046c1557167801af8e5cb2
Signed-off-by: Tim Crawford <[email protected]>
@crawfxrd crawfxrd changed the title ec/system76/ec: Remove RPM calculation ec/system76/ec: Add config for 2nd fan without dGPU Jul 5, 2024
@crawfxrd crawfxrd requested a review from jackpot51 July 5, 2024 22:11
Copy link

@XV-02 XV-02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to be functioning without issues. Good to go.

@crawfxrd crawfxrd merged commit bb712a7 into system76 Jul 17, 2024
@crawfxrd crawfxrd deleted the acpi-rpm branch July 17, 2024 22:29
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

Successfully merging this pull request may close these issues.

3 participants