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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux