summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorShyam Sundar S K <Shyam-sundar.S-k@amd.com>2023-01-25 15:29:34 +0530
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2023-02-09 11:28:06 +0100
commitbb4a5cef532d6948c34086b588b46f650bc96ead (patch)
treeecedfa3a266591e5e2570a6e2d725de575677c86
parent3be069f42a7b79d3149194f21cdf24bf23864cac (diff)
downloadlinux-bb4a5cef532d6948c34086b588b46f650bc96ead.tar.gz
linux-bb4a5cef532d6948c34086b588b46f650bc96ead.tar.bz2
linux-bb4a5cef532d6948c34086b588b46f650bc96ead.zip
platform/x86/amd/pmf: update to auto-mode limits only after AMT event
[ Upstream commit 3dfe28c936f87373a2b6ada750be4c52c0f249f3 ] Auto-mode thermal limits should be updated only after receiving the AMT event. But due to a bug in the older commit, these settings were getting applied during the auto-mode init. Fix this by removing amd_pmf_set_automode() during auto-mode initialization. Fixes: 3f5571d99524 ("platform/x86/amd/pmf: Add support for Auto mode feature") Suggested-by: Patil Rajesh Reddy <Patil.Reddy@amd.com> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com> Link: https://lore.kernel.org/r/20230125095936.3292883-4-Shyam-sundar.S-k@amd.com Reviewed-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r--drivers/platform/x86/amd/pmf/auto-mode.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/platform/x86/amd/pmf/auto-mode.c b/drivers/platform/x86/amd/pmf/auto-mode.c
index 644af42e07cf..85ce9ed25b85 100644
--- a/drivers/platform/x86/amd/pmf/auto-mode.c
+++ b/drivers/platform/x86/amd/pmf/auto-mode.c
@@ -299,7 +299,5 @@ void amd_pmf_deinit_auto_mode(struct amd_pmf_dev *dev)
void amd_pmf_init_auto_mode(struct amd_pmf_dev *dev)
{
amd_pmf_load_defaults_auto_mode(dev);
- /* update the thermal limits for Automode */
- amd_pmf_set_automode(dev, config_store.current_mode, NULL);
amd_pmf_init_metrics_table(dev);
}