platform/x86: thinkpad_acpi: Disable DYTC CQL mode around switching to balanced mode
authorHans de Goede <hdegoede@redhat.com>
Sun, 21 Mar 2021 11:31:08 +0000 (12:31 +0100)
committerHans de Goede <hdegoede@redhat.com>
Sun, 21 Mar 2021 16:39:27 +0000 (17:39 +0100)
commite157c95f589409d32e6e245f6f185069a8c29cbd
treedeb7b30176863df38173a715a262036628a45171
parent12dde4d11e6d9c61f89496d358505ec3197a67f9
platform/x86: thinkpad_acpi: Disable DYTC CQL mode around switching to balanced mode

Testing has shown that setting /sys/firmware/acpi/platform_profile to
"balanced" when /sys/bus/platform/devices/thinkpad_acpi/dytc_lapmode
reports 1, causes dytc_lapmode to get reset to 0 and then it becomes
stuck at 0 for aprox. 30 minutes even if the laptop is used on a lap.

Disabling CQL (when enabled) before issuing the DYTC_CMD_RESET to get
back to balanced mode and re-enabling it afterwards again, like the
code already does when switching to low-power / performance mode fixes
this.

Fixes: c3bfcd4c6762 ("platform/x86: thinkpad_acpi: Add platform profile support")
Tested-by: Mark Pearson <markpearson@lenovo.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20210321113108.7069-1-hdegoede@redhat.com
drivers/platform/x86/thinkpad_acpi.c