RE: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-20 Thread Zheng, Lv
Hi, Rafael > From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. > Wysocki > Subject: Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from > suspend-to-idle on recent Dell systems > > On Tue, Jun 20, 2017 at 2:

RE: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-20 Thread Zheng, Lv
Hi, > From: rjwyso...@gmail.com [mailto:rjwyso...@gmail.com] On Behalf Of Rafael J. > Wysocki > Subject: Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from > suspend-to-idle on recent Dell systems > > On Tue, Jun 20, 2017 at 1:37 AM, Zheng, Lv wrote: > > Hi, Raf

Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-20 Thread Rafael J. Wysocki
On Tue, Jun 20, 2017 at 4:00 AM, Linus Torvalds wrote: > On Tue, Jun 20, 2017 at 9:13 AM, Rafael J. Wysocki wrote: >> >> My understanding is that Windows uses the ACPI_FADT_LOW_POWER_S0 flag. >> It generally enables non-S3 suspend/resume when this flag is set and >> it doesn't touch S3 then. Kee

Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-19 Thread Linus Torvalds
On Tue, Jun 20, 2017 at 9:13 AM, Rafael J. Wysocki wrote: > > My understanding is that Windows uses the ACPI_FADT_LOW_POWER_S0 flag. > It generally enables non-S3 suspend/resume when this flag is set and > it doesn't touch S3 then. Keeping the EC GPE (and other GPEs for that > matter) enabled ove

Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-19 Thread Rafael J. Wysocki
On Tue, Jun 20, 2017 at 2:07 AM, Linus Torvalds wrote: > On Tue, Jun 20, 2017 at 5:53 AM, Rafael J. Wysocki wrote: >> >> -> v2: Added acpi_sleep=no_ec_wakeup to prevent EC events from waking up >> the system from s2idle on systems where they do that by default. > > This seems a big hack

Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-19 Thread Linus Torvalds
On Tue, Jun 20, 2017 at 5:53 AM, Rafael J. Wysocki wrote: > > -> v2: Added acpi_sleep=no_ec_wakeup to prevent EC events from waking up > the system from s2idle on systems where they do that by default. This seems a big hacky. Is there no way to simply make acpi_ec_suspend() smarter whi

Re: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-19 Thread Rafael J. Wysocki
On Tue, Jun 20, 2017 at 1:37 AM, Zheng, Lv wrote: > Hi, Rafael > >> From: linux-acpi-ow...@vger.kernel.org >> [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. >> Wysocki >> Subject: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-i

RE: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-19 Thread Zheng, Lv
Hi, Rafael > From: linux-acpi-ow...@vger.kernel.org > [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J. > Wysocki > Subject: [PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on > recent Dell systems > > From: Rafael J. Wysocki > &g

[PATCH v2 3/3] ACPI / sleep: EC-based wakeup from suspend-to-idle on recent Dell systems

2017-06-19 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Some recent Dell laptops, including the XPS13 model numbers 9360 and 9365, cannot be woken up from suspend-to-idle by pressing the power button which is unexpected and makes that feature less usable on those systems. Moreover, on the 9365 ACPI S3 (suspend-to-RAM) is not e