From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4A6DD28137C; Sun, 1 Jun 2025 23:29:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748820555; cv=none; b=I1xe2r96MbAGVbrONG/2U+BhQ6G40xe3Uur/7ZBOK8jXO0GMXfNE2je0KXLOqgpryX/aMmeWHyOxphLuxseIYHm1vkAW7pmlZ/PIVbz9bK9h2VbRYyYlVV0KO44sVgagSnDfqsNg4a85liOxgiH6/5U002et6QvwkeY5r35bUhM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748820555; c=relaxed/simple; bh=oMK/FszmytbQW76F8NDQuGeWt7/Tzt8bKVFy+DSymnY=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version:Content-Type; b=oIi7kZdxolZNEzxfO3+iu+8PB8fkVkoZuYKlbRbuz+tipjUVGsH5bpptiG7dwuuxWr4dixsf71yi2oyM7tLpi2H4pQWG0yjDm3rJ7lXO65k22ymMsf9u/2mGbo51cnQIMsFHxFi5IhD0taUM194/vki84BRcQkxcTDyQEXP/YvQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=A9UG7+1U; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="A9UG7+1U" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4186EC4CEF2; Sun, 1 Jun 2025 23:29:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1748820555; bh=oMK/FszmytbQW76F8NDQuGeWt7/Tzt8bKVFy+DSymnY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=A9UG7+1UlvnLcSAgCG+CQBvAbKGh3+J2g9qgmbF5Gu0/ae/QqRS1U9EqsIBPMAtI5 aN04kaUPsRdAtp56ubzsaWlZtLH0Hm04nSLBuv6Qe6LrbeRxKz5yhgMIjU1nKJhsgd I4hTuW/bAtqIP4Ll51LIUDP0fHCwNlbhVO6TL/yaqVJVIV1czTckZWU11tp+v2sVjl ltnR6gr1w4jijJ3N8JhBawHGCm237VJVG33sCJEC3FkkuNp1MPmPxt+CQquj3SlP9H Su4i5VtXG4SqGIPKml/tnt6PpYMAMrfncvOkOACcL6jvIn6lBwulTvrG5YHIzE7Ku6 HFKY17gjii1wQ== From: Sasha Levin To: patches@lists.linux.dev, stable@vger.kernel.org Cc: Werner Sembach , "Rafael J . Wysocki" , Sasha Levin , rafael@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH AUTOSEL 6.15 100/110] ACPI: EC: Add device to acpi_ec_no_wakeup[] qurik list Date: Sun, 1 Jun 2025 19:24:22 -0400 Message-Id: <20250601232435.3507697-100-sashal@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20250601232435.3507697-1-sashal@kernel.org> References: <20250601232435.3507697-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.15 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: Werner Sembach [ Upstream commit 9cd51eefae3c871440b93c03716c5398f41bdf78 ] Add the TUXEDO InfinityBook Pro AMD Gen9 to the acpi_ec_no_wakeup[] quirk list to prevent spurious wakeups. Signed-off-by: Werner Sembach Link: https://patch.msgid.link/20250508111625.12149-1-wse@tuxedocomputers.com Signed-off-by: Rafael J. Wysocki Signed-off-by: Sasha Levin --- The search failed because this is the current version in the kernel tree, but the commit we're analyzing hasn't been applied yet. Let me analyze this commit based on the information provided. ## Analysis **YES** This commit should be backported to stable kernel trees. ### Detailed Analysis: **1. Nature of the Fix:** The commit adds the "TUXEDO InfinityBook Pro AMD Gen9" laptop to the `acpi_ec_no_wakeup[]` DMI quirk list. This is a targeted fix for spurious wakeups from suspend-to-idle (s2idle) caused by EC (Embedded Controller) interrupts on a specific laptop model. **2. Code Changes:** The change is minimal and surgical: - Adds a single new DMI entry to an existing quirk table - Uses `DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx")` to identify the specific device - Includes a comment identifying the device clearly - No functional code logic changes - only data structure modification **3. Comparison with Historical Similar Commits:** Looking at the historical commits provided: - **Similar Commit #1 (NO):** Added HP ZHAN 66 Pro - marked as NO for backporting - **Similar Commit #2 (YES):** Fixed ThinkPad X1 Carbon 6th DMI matching - marked as YES - **Similar Commit #3 (NO):** Added ThinkPad X1 Yoga 3rd - marked as NO - **Similar Commit #4 (YES):** GPIO quirk for GPD G1619-04 with `Cc: stable@vger.kernel.org` - marked as YES - **Similar Commit #5 (YES):** Removed duplicate entry - marked as YES **4. Stable Tree Criteria Assessment:** ✅ **Fixes important user- visible bug:** Spurious wakeups significantly impact user experience and battery life ✅ **Small and contained change:** Only adds one DMI table entry, no logic changes ✅ **Low risk of regression:** DMI quirks are device-specific and don't affect other systems ✅ **Well-tested pattern:** The `acpi_ec_no_wakeup` mechanism is well-established and widely used ✅ **Critical subsystem appropriately handled:** ACPI EC is critical but this change is isolated to one device **5. Risk Assessment:** - **Risk to target device:** Minimal - disables EC wakeup which is the intended behavior to fix spurious wakeups - **Risk to other devices:** None - DMI matching is device-specific - **Risk of regression:** Very low - follows established pattern used by many other laptop models **6. Precedent Analysis:** The pattern shows that while some individual device additions were marked as NO (commits #1 and #3), those that include proper documentation, clear problem statements, or fix significant user issues tend to be marked YES (commits #2, #4, #5). This commit includes a clear problem statement ("prevent spurious wakeups") and follows the exact same pattern as successful backports. **7. User Impact:** Spurious wakeups during suspend-to-idle directly impact: - Battery life (system doesn't stay in low-power state) - User experience (unexpected wake-ups) - Power management effectiveness This is exactly the type of user-facing power management issue that stable trees aim to address. The commit is an ideal candidate for backporting: it's a small, safe, device-specific fix for a real user problem that follows well-established patterns with minimal risk. drivers/acpi/ec.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c index 3c5f34892734e..6f4203716b53f 100644 --- a/drivers/acpi/ec.c +++ b/drivers/acpi/ec.c @@ -2329,6 +2329,12 @@ static const struct dmi_system_id acpi_ec_no_wakeup[] = { DMI_MATCH(DMI_PRODUCT_NAME, "83Q3"), } }, + { + // TUXEDO InfinityBook Pro AMD Gen9 + .matches = { + DMI_MATCH(DMI_BOARD_NAME, "GXxHRXx"), + }, + }, { }, }; -- 2.39.5