Received-SPF: pass (google.com: domain of linux-kernel+bounces-64823-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Armin Wolf <W_Armin@gmx.de>
To: corentin.chary@gmail.com,
	luke@ljones.dev
Cc: hdegoede@redhat.com,
	ilpo.jarvinen@linux.intel.com,
	platform-driver-x86@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH 5/5] Revert "platform/x86: asus-wmi: Support WMI event queue"
Date: Wed, 14 Feb 2024 08:04:33 +0100
Message-Id: <20240214070433.2677-6-W_Armin@gmx.de>
In-Reply-To: <20240214070433.2677-1-W_Armin@gmx.de>
References: <20240214070433.2677-1-W_Armin@gmx.de>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
UI-OutboundReport: notjunk:1;M01:P0:VggErPqJM9c=;PauDD9Ido5AEVDLIW3VhEGRrO8V
 sDNPHDUGZRec9GOTnqKdSvtNLMfDvyDcShVmaPPNE+AEQFRxMusubSNIlFTvYEpcZS9WYjayg
 jJAeYDbr2F4NKMqxlv57DZHNb66LuXIfw66CZtL136XdjfPOJaCnF9/2R6ijzvauaG5bnjvIX
 7a3jjJk6FHnuORUUBpDINq5LCfjoFKl6WTOUl8ftfmf0/4NDlkFUrwD6p6YGuFvqw9aZ8j4SJ
 7YQund/DFxAq0M6pezSj4VeHLoNFDby61ksAZXvFsu3p5prE8a8rzVg2itwuCHvnZx8eg3oJd
 X0rwaJ1TLfVmbPaa960VpYUPb19F4p9shp0XOfh25n8MbBCWgCrAbopBZMSsgay0/DOOLhaG2
 a2JVf/cj0rXViV1zCUQ3On0bxbQOytXF0rYLdJ0O16kpmZRXwCD//Tc7qBG/6dGzxHrF8cdn3
 o1c59O+KL6dI6a22CJDuzFyTs0OIuj9ygNVe//2NqMwCoJDlhjgLN/MK47FRkebVlV/aXoHnR
 yIGMUlX3Uh/hoIkKwj0mSlVsueTNQXlkdZVRZM4IUTM1pWEHyARv3yqUje290W1ScP1Qhu15Y
 74ql3vMKJqPTQH8he063400q0UIYG/uFQYX23G6bLkDwlkt4KyhzYUY6jFLgRY3hIo1U0B9wv
 Ikb+ouKy/djRt2d0mYr+6omUkcmlRnkJsJ3Kxn4pbLAcaGgSQrYIl76PjrvDdOEg5YyTslGbH
 e+h42LNNDXYrbnkMY1jo/xxofpObcW6YYg3jw+QHNWTwaC998cdeRZTvzPChvmnL5bh5aEcs1
 AdvY018m246TnKtGckaoEoPF/p7B6wUb2gKx0mm8JDfDo=

This reverts commit 1a373d15e283937b51eaf5debf4fc31474c31436.

The WMI core now takes care of draining the event queue if asus-wmi
is not loaded, so the hacky event queue handling code is not needed
anymore.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>
=2D--
 drivers/platform/x86/asus-wmi.c | 71 +++------------------------------
 1 file changed, 5 insertions(+), 66 deletions(-)

diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-w=
mi.c
index 21dee425ea6f..2865af89e95c 100644
=2D-- a/drivers/platform/x86/asus-wmi.c
+++ b/drivers/platform/x86/asus-wmi.c
@@ -101,13 +101,6 @@ module_param(fnlock_default, bool, 0444);
 #define PCI_DEVICE_ID_INTEL_LYNXPOINT_LP_XHCI	0x9c31

 #define ASUS_ACPI_UID_ASUSWMI		"ASUSWMI"
-#define ASUS_ACPI_UID_ATK		"ATK"
-
-#define WMI_EVENT_QUEUE_SIZE		0x10
-#define WMI_EVENT_QUEUE_END		0x1
-#define WMI_EVENT_MASK			0xFFFF
-/* The WMI hotkey event value is always the same. */
-#define WMI_EVENT_VALUE_ATK		0xFF

 #define WMI_EVENT_MASK			0xFFFF

@@ -219,7 +212,6 @@ struct asus_wmi {
 	int dsts_id;
 	int spec;
 	int sfun;
-	bool wmi_event_queue;

 	struct input_dev *inputdev;
 	struct backlight_device *backlight_device;
@@ -4019,50 +4011,14 @@ static void asus_wmi_handle_event_code(int code, s=
truct asus_wmi *asus)
 static void asus_wmi_notify(u32 value, void *context)
 {
 	struct asus_wmi *asus =3D context;
-	int code;
-	int i;
-
-	for (i =3D 0; i < WMI_EVENT_QUEUE_SIZE + 1; i++) {
-		code =3D asus_wmi_get_event_code(value);
-		if (code < 0) {
-			pr_warn("Failed to get notify code: %d\n", code);
-			return;
-		}
-
-		if (code =3D=3D WMI_EVENT_QUEUE_END || code =3D=3D WMI_EVENT_MASK)
-			return;
-
-		asus_wmi_handle_event_code(code, asus);
-
-		/*
-		 * Double check that queue is present:
-		 * ATK (with queue) uses 0xff, ASUSWMI (without) 0xd2.
-		 */
-		if (!asus->wmi_event_queue || value !=3D WMI_EVENT_VALUE_ATK)
-			return;
-	}
+	int code =3D asus_wmi_get_event_code(value);

-	pr_warn("Failed to process event queue, last code: 0x%x\n", code);
-}
-
-static int asus_wmi_notify_queue_flush(struct asus_wmi *asus)
-{
-	int code;
-	int i;
-
-	for (i =3D 0; i < WMI_EVENT_QUEUE_SIZE + 1; i++) {
-		code =3D asus_wmi_get_event_code(WMI_EVENT_VALUE_ATK);
-		if (code < 0) {
-			pr_warn("Failed to get event during flush: %d\n", code);
-			return code;
-		}
-
-		if (code =3D=3D WMI_EVENT_QUEUE_END || code =3D=3D WMI_EVENT_MASK)
-			return 0;
+	if (code < 0) {
+		pr_warn("Failed to get notify code: %d\n", code);
+		return;
 	}

-	pr_warn("Failed to flush event queue\n");
-	return -EIO;
+	asus_wmi_handle_event_code(code, asus);
 }

 /* Sysfs ****************************************************************=
******/
@@ -4302,23 +4258,6 @@ static int asus_wmi_platform_init(struct asus_wmi *=
asus)
 		asus->dsts_id =3D ASUS_WMI_METHODID_DSTS;
 	}

-	/*
-	 * Some devices can have multiple event codes stored in a queue before
-	 * the module load if it was unloaded intermittently after calling
-	 * the INIT method (enables event handling). The WMI notify handler is
-	 * expected to retrieve all event codes until a retrieved code equals
-	 * queue end marker (One or Ones). Old codes are flushed from the queue
-	 * upon module load. Not enabling this when it should be has minimal
-	 * visible impact so fall back if anything goes wrong.
-	 */
-	wmi_uid =3D wmi_get_acpi_device_uid(asus->driver->event_guid);
-	if (wmi_uid && !strcmp(wmi_uid, ASUS_ACPI_UID_ATK)) {
-		dev_info(dev, "Detected ATK, enable event queue\n");
-
-		if (!asus_wmi_notify_queue_flush(asus))
-			asus->wmi_event_queue =3D true;
-	}
-
 	/* CWAP allow to define the behavior of the Fn+F2 key,
 	 * this method doesn't seems to be present on Eee PCs */
 	if (asus->driver->quirks->wapf >=3D 0)
=2D-
2.39.2