Re: [PATCH] ACPI: suspend: don't let device _PS3 failure prevent suspend

[yakui_zhao <yakui.zhao@intel.com>]

On Mon, 2009-05-11 at 23:21 +0800, Rafael J. Wysocki wrote:
> On Monday 11 May 2009, yakui_zhao wrote:
> > On Mon, 2009-05-11 at 04:48 +0800, Witold Szczeponik wrote:
> > > Henrique de Moraes Holschuh schrieb:
> > > 
> > > > On Fri, 08 May 2009, Len Brown wrote:
> > > >> Or perhaps we should just shrug off the _PS3 failure
> > > >> and carry on with _DIS like we used to -- which is
> > > >> what this patch does.
> > > > 
> > > > Maybe "printk and carry on" would be better?
> > > > 
> > > 
> > > And there is very similar code just a few lines above: where we turn on 
> > > the device by setting it to D0.  There, too, we may want to skip the 
> > > check whether or not the transition was successful or not.
> > What you said is right.
> > It sounds reasonable that we skip the power state check while transiting
> > it to D0 state.
> > 
> > In fact the power state check can be skipped by adding the boot option
> > of "acpi.power_nocheck=1".
> 
> Can we avoid adding the boot option?  I'd very much prefer not to add boot
> options if not really necessary.
One is to add it into the DMI power check table so that the default
value of power_nocheck is 1. 

Another is that the default value of power_nocheck is changed to 1
instead of 0. 

In such case the power state check will be skipped in course of power
transition.

Is this OK?
> 
> Rafael