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
Platform profile: switch fan profile #145
Platform profile: switch fan profile #145
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@qzed , filed a draft PR for the additions to the platform profile, still some outstanding questions here and there on the approach, but the overall functionality is there.
54d09a8
to
274e706
Compare
I've marked this as ready for review. I cleaned up some comments here and there, and renamed the node in the registry. I've also squashed the commits and put a reasonable commit message / title on it. |
274e706
to
1f32229
Compare
1f32229
to
5000708
Compare
Change naming from tmp to platform profile to clarify the module may interact with both the TMP and FAN subystems. Add functionality that switches the fan profile when the platform profile is changed. Signed-off-by: Ivor Wanders <ivor@iwanders.net>
5000708
to
58baf8a
Compare
@StollD if you have some time, feel free to merge this and add it to the patches. Otherwise I'll try to make some time, but I'm not sure I can get to much before March 15th. |
Upstream lore entry for v1. |
I cherry-picked this to our 6.6 and 6.7 branches. Thank you! |
Changes: - Add driver for changing the fan profile (Surface Pro 9) Links: - kernel: linux-surface/kernel@fcc3a67 - fan profile driver: linux-surface/kernel#145
Changes: - Add driver for changing the fan profile (Surface Pro 9) - Added fixes for IPU3 cameras - Reverted DW9719 driver to previous version (Surface Go 2) Links: - kernel: linux-surface/kernel@530f557 - fan profile driver: linux-surface/kernel#145 - camera fixes: linux-surface/kernel#146
Change naming from tmp to platform profile to clarify the module may interact with both the TMP and FAN subystems. Add functionality that switches the fan profile when the platform profile is changed when a fan is present. Signed-off-by: Ivor Wanders <ivor@iwanders.net> Link: #145 Reviewed-by: Maximilian Luz <luzmaximilian@gmail.com> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Change naming from tmp to platform profile to clarify the module may interact with both the TMP and FAN subystems. Add functionality that switches the fan profile when the platform profile is changed when a fan is present. Signed-off-by: Ivor Wanders <ivor@iwanders.net> Link: #145 Reviewed-by: Maximilian Luz <luzmaximilian@gmail.com> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Change naming from tmp to platform profile to clarify the module may interact with both the TMP and FAN subystems. Add functionality that switches the fan profile when the platform profile is changed when a fan is present. Signed-off-by: Ivor Wanders <ivor@iwanders.net> Link: #145 Reviewed-by: Maximilian Luz <luzmaximilian@gmail.com> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Change naming from tmp to platform profile to clarify the module may interact with both the TMP and FAN subystems. Add functionality that switches the fan profile when the platform profile is changed when a fan is present. Signed-off-by: Ivor Wanders <ivor@iwanders.net> Link: linux-surface/kernel#145 Reviewed-by: Maximilian Luz <luzmaximilian@gmail.com> Link: https://lore.kernel.org/r/20240314223733.6236-2-ivor@iwanders.net Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Followup from #144 that adds support to switch the fan profile whenever the platform profile is switched.
Values recorded with IRPMon as sent to the FAN together with the numbers sent to the TMP module:
I've confirmed the changes in this PR using the following methodology;
echo balanced-performance > /sys/firmware/acpi/platform_profile
for normal)stress -c 12
.stress -c 12
after steady state has been reached, plus a bit (didn't time this). Mark time.This obtains us the following fan curves:
We clearly get 4 different fan max speeds, with performance as the highest level.