From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH 0/3] ACPI / sleep: Support power button wakeup from S2I on recent Dell laptops Date: Tue, 13 Jun 2017 02:22:37 +0200 Message-ID: References: <88366b425cce41358ae8f88e43f3d201@ausx13mpc120.AMER.DELL.COM> <1872413.KDFIRZOQLU@aspire.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-it0-f67.google.com ([209.85.214.67]:34037 "EHLO mail-it0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752313AbdFMAWi (ORCPT ); Mon, 12 Jun 2017 20:22:38 -0400 In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: =?UTF-8?B?SsOpcsO0bWUgZGUgQnJldGFnbmU=?= Cc: "Rafael J. Wysocki" , "Rafael J. Wysocki" , Mario Limonciello , ACPI Devel Maling List , Linux PM , Andy Shevchenko , Darren Hart , Srinivas Pandruvada , Mika Westerberg On Tue, Jun 13, 2017 at 2:00 AM, J=C3=A9r=C3=B4me de Bretagne wrote: >>> One more ouput for tonight. After doing a diff of the acpidump from >>> both BIOS versions, I've found explicit references to the two 0xCE and >>> 0xCF values right in the delta, in the dsdt.dsl files. >> >> Right. That's what my last message was about. :-) > > Indeed, our messages have crossed each other :-) > >>> I'm new to ACPI but could that ECBT mean "EC Button" maybe? >> >> That's just a method name which may or may not mean the above. >> >> \_SB.PCI0.LPCB.ECDV is the EC, so it is all related. >> >> However, the bottom line is that there is no way for the kernel at all >> to know in advance that these events are going to be generated. > > Ok, so I understand that this is totally device-specific :-( Well, device-and-BIOS-version-specific even. > Would a patch to handle this specific case be acceptable in the kernel th= ough? We have quirks. :-) That said I think intel-hid could check the 0xCE event unconditionally in the "wakeup" mode, but that will be a separate patch on top of my series. Thanks, Rafael