Received-SPF: pass (google.com: domain of linux-kernel+bounces-129785-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: Gergo Koteles <soyer@irl.hu>
To: Ike Panhc <ike.pan@canonical.com>,
  Hans de Goede <hdegoede@redhat.com>,
  =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>
Cc: platform-driver-x86@vger.kernel.org,
  linux-kernel@vger.kernel.org, Gergo Koteles <soyer@irl.hu>
Subject: [PATCH] platform/x86: ideapad-laptop: switch platform profiles using thermal management key
Date: Wed,  3 Apr 2024 15:09:07 +0200
Message-ID: <85254ce8e87570c05e7f04d6507701bef954ed75.1712149429.git.soyer@irl.hu>
Precedence: bulk
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Ideapad laptops have thermal management or performance mode switch key
(Fn + Q). Now it sends KEY_PROG4.

Switch platform profiles instead, like the ThinkPad ACPI driver.

Tested on Yoga7 14ARB7.

Signed-off-by: Gergo Koteles <soyer@irl.hu>
---
 drivers/platform/x86/ideapad-laptop.c | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/drivers/platform/x86/ideapad-laptop.c b/drivers/platform/x86/ideapad-laptop.c
index 901849810ce2..6310011cc1b3 100644
--- a/drivers/platform/x86/ideapad-laptop.c
+++ b/drivers/platform/x86/ideapad-laptop.c
@@ -1071,7 +1071,6 @@ static const struct key_entry ideapad_keymap[] = {
 	{ KE_KEY,  16, { KEY_PROG1 } },
 	{ KE_KEY,  17, { KEY_PROG2 } },
 	{ KE_KEY,  64, { KEY_PROG3 } },
-	{ KE_KEY,  65, { KEY_PROG4 } },
 	{ KE_KEY,  66, { KEY_TOUCHPAD_OFF } },
 	{ KE_KEY,  67, { KEY_TOUCHPAD_ON } },
 	{ KE_KEY, 128, { KEY_ESC } },
@@ -1181,8 +1180,27 @@ static void ideapad_check_special_buttons(struct ideapad_private *priv)
 		switch (bit) {
 		case 6:	/* Z570 */
 		case 0:	/* Z580 */
-			/* Thermal Management button */
-			ideapad_input_report(priv, 65);
+			/* Thermal Management / Performance Mode button */
+			switch (priv->dytc->current_profile) {
+			case PLATFORM_PROFILE_LOW_POWER:
+				dytc_profile_set(&priv->dytc->pprof,
+						 PLATFORM_PROFILE_BALANCED);
+				break;
+			case PLATFORM_PROFILE_BALANCED:
+				dytc_profile_set(&priv->dytc->pprof,
+						 PLATFORM_PROFILE_PERFORMANCE);
+				break;
+			case PLATFORM_PROFILE_PERFORMANCE:
+				dytc_profile_set(&priv->dytc->pprof,
+						 PLATFORM_PROFILE_LOW_POWER);
+				break;
+			default:
+				dev_warn(&priv->platform_device->dev,
+					 "Unexpected platform profile %d",
+					 priv->dytc->current_profile);
+			}
+			/* Notify user space the profile changed */
+			platform_profile_notify();
 			break;
 		case 1:
 			/* OneKey Theater button */

base-commit: 39cd87c4eb2b893354f3b850f916353f2658ae6f
-- 
2.44.0