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, 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 <lv.zh...@intel.com&g

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

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.

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 >

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

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

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

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

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

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-idle >> on

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-idle >> on recent Dell systems >> >>

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 > >

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 > > Some recent Dell laptops,