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

tcc shows fan speed is 0, even though it is noisy #392

Open
harridu opened this issue Jun 7, 2024 · 4 comments
Open

tcc shows fan speed is 0, even though it is noisy #392

harridu opened this issue Jun 7, 2024 · 4 comments

Comments

@harridu
Copy link

harridu commented Jun 7, 2024

After wakeup from hibernate the tuxedo control center claims the fan is at 0%, even though the fan is noisy. Before hibernate tcc showed a fan speed of 20% due to regular load.

Changing the profile or connecting the power brick doesn't help. I could reproduce this on 2 Infinitybooks (by now).

Platform is Debian 12, kernel 6.1.90, tcc is 2.1.8, tuxedo-drivers is version 4.5.0. Hardware is a shiny new Tuxedo Infinitybook Pro16.

The TUXEDO default profile has been chosen.

@harridu
Copy link
Author

harridu commented Jun 10, 2024

I have the impression that loading the profiles is broken after wakeup from hibernate.

@nickma82
Copy link

nickma82 commented Jun 18, 2024

Same here. Fan Speed in tuxedo-control-center is shown as 0% while fan is spinning audibly.

@harridu Ad "broken after wakeup": I can confirm, that after a fresh restart the fan works as expected.

Hardware/Software

  • InfinityBook-Pro-14-Gen8 (without NVidia card)
  • TuxedoOS 3 22.04.4
  • tuxedo-control-center 2.1.11-2
  • tuxedo-drivers 4.5.0
  • Kernel 6.5.0-10040-tuxedo

@tuxedoder
Copy link
Contributor

tuxedoder commented Jun 18, 2024

Currently for the fan api which the IB Pro uses, tcc shows what it thinks it configured and not the actual sensor value and shows 0 if fans are disabled globally.

const currentSpeedPercent: ObjWrapper<number> = { value: 0 };
// const speedReadSuccess = ioAPI.getFanSpeedPercent(fanIndex, currentSpeedPercent);
tempSensorAvailable.push(tempReadSuccess);
fanTimestamps.push(Date.now());
fanSpeedsRead.push(currentSpeedPercent.value);

if (fanTemps[i] === -1) {
currentSpeed = -1;
} else if (useFanControl) {
currentSpeed = fanSpeedsSet[i];
} else {
currentSpeed = fanSpeedsRead[i];
}

I didn't write the initial fan control and I am not sure why it is like this, but I wanted to change it later. Otherwise fan control seems to work on my IB Pro 16 Gen 7. Fans might spin on wakeup since fans can't be instantly controlled and are in auto mode for a bit, but afterwards the fans seem to apply correctly after they get controlled by tccd or rather tuxedo-drivers. Keep in mind that fans also need some time to reach a value and how much depends on the device.

I was trying to rewrite the fan control inside tccd since a while and it seems to work fine there since I show sensor values. Based on the information I have, I think it is most likely just a display discrepancy in the dashboard and should be fixed with my rewrite. No ETA yet on when the new fan control will be public.

@nickma82
Copy link

nickma82 commented Jun 19, 2024

@tuxedoder Ad "tcc shows what it thinks it configured":
That's possible an printing the desired state would in-general be okay, IF applied to the hardware in a reliable way...

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

3 participants