From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751247AbdFEPTD (ORCPT ); Mon, 5 Jun 2017 11:19:03 -0400 Received: from esa1.dell-outbound.iphmx.com ([68.232.153.90]:54681 "EHLO esa1.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751077AbdFEPTC (ORCPT ); Mon, 5 Jun 2017 11:19:02 -0400 From: X-LoopCount0: from 10.175.216.251 X-IronPort-AV: E=Sophos;i="5.39,300,1493701200"; d="scan'208";a="1117670724" To: , CC: , , , , , Subject: RE: [PATCH 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems Thread-Topic: [PATCH 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems Thread-Index: AQHS2maHWt1G2nSnXE27L8qx4q5t0qIWZ5wQ Date: Mon, 5 Jun 2017 15:18:59 +0000 Message-ID: <806f96fe75d84268a570b702490c686c@ausx13mpc120.AMER.DELL.COM> References: <1979543.KIEJ8uyRaT@aspire.rjw.lan> <3454366.uzaJljlWGm@aspire.rjw.lan> <2944956.jYmHlBxMl3@aspire.rjw.lan> In-Reply-To: <2944956.jYmHlBxMl3@aspire.rjw.lan> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.143.242.75] Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v55FJ6KJ027147 > + /* > + * Enable the EC to wake up the system from suspend-to-idle to allow > + * power button events to it wake up. > + */ > + { > + .callback = init_ec_gpe_wakeup, > + .ident = "Dell XPS 13 9360", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "XPS 13 9360"), > + }, > + }, > + { > + .callback = init_ec_gpe_wakeup, > + .ident = "Dell XPS 13 9365", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "XPS 13 9365"), > + }, > + }, > {}, > }; > Although the XPS 9360 and XPS 9365 are currently the only Dell products on the market that do the wakeup this way, this is part of Dell's BIOS and EC codebase. As other Dell products start to ship Windows 10 with Modern Standby and (eventually) Linux with Suspend-to-Idle I'd expect this list to be growing. I know that we talked about this previously in relation to the EC notification of going in to suspend-to-idle and you had instead marked it across all Dell Inc systems. Rather than hardcode a whitelist of platform that can do this, could you do the same thing here and allow EC wakeups across Dell Inc. systems? Thanks,