> -----Original Message----- > From: Srinivas Pandruvada [mailto:srinivas.pandruv...@linux.intel.com] > Sent: Friday, June 23, 2017 11:06 AM > To: Limonciello, Mario <mario_limoncie...@dell.com>; r...@rjwysocki.net; > linux- > a...@vger.kernel.org > Cc: linux...@vger.kernel.org; andriy.shevche...@linux.intel.com; > dvh...@infradead.org; linux-kernel@vger.kernel.org; > mika.westerb...@linux.intel.com; t...@oneshoeco.com; > jerome.debreta...@gmail.com; torva...@linux-foundation.org; > lv.zh...@intel.com > Subject: Re: [PATCH v2] ACPI / sleep: EC-based wakeup from suspend-to-idle on > recent systems > > On Fri, 2017-06-23 at 15:37 +0000, mario.limoncie...@dell.com wrote: > > [...] > > > > > > > +#define ACPI_LPS0_SCREEN_ON 4 > > > +#define ACPI_LPS0_ENTRY 5 > > > +#define ACPI_LPS0_EXIT 6 > > The spec you shared also defines device constraints (function 1). It > > would be very > > useful if these constraints could be parsed and compared against the > > actual power > > states of devices on the system at least for debugging purposes. I'm > > not sure if you > > already had a plan for that in a future series. > > > For debug purpose, I have worked on a patch to dump the constraint > table in debugfs. But in the freeze path whether we meet the > constraints or not will not make any difference, other than for just > debugging. > > Thanks, > Srinivas
Right that was what I thought would be most interesting. You can potentially output to syslog as a last step going down what isn't in the right state to match the constraint table.