Patch: add quirks for Dell D420

[Frederic Crozat]

Le mardi 22 mai 2007 à 14:21 +0100, Bastien Nocera a écrit :
> On Tue, 2007-05-22 at 14:08 +0100, Bastien Nocera wrote:
> > On Tue, 2007-05-22 at 15:13 +0200, Frederic Crozat wrote:
> > > Le mardi 22 mai 2007 à 13:48 +0100, Bastien Nocera a écrit :
> > > > On Tue, 2007-05-22 at 14:43 +0200, Frederic Crozat wrote:
> > > > > Le mardi 22 mai 2007 à 13:23 +0100, Bastien Nocera a écrit :
> > > > > > On Wed, 2007-05-02 at 23:57 -0400, David Zeuthen wrote:
> > > > > > > On Wed, 2007-05-02 at 14:59 +0200, Frederic Crozat wrote:
> > > > > > > > Even if Bastien has already submitted a similar patch, I prefer to
> > > > > > > > post
> > > > > > > > our patch too, just to be sure I don't forget to send all our patches
> > > > > > > > here :) 
> > > > > > > 
> > > > > > > Committed (with moving things around a bit). Thanks.
> > > > > > 
> > > > > > Could this be revert?
> > > > > > 
> > > > > > It actually breaks resume on my laptop when X is running. What do we do
> > > > > > thee then?
> > > > > 
> > > > > Try replacing the vbestate_restore with vbemode_restore and see if it
> > > > > works.
> > > > 
> > > > Works under X.
> > > 
> > > Could you also test under a text console :
> > > log as root and run pm-suspend --quirk-vbe-post --quirk-vbemode-restore
> > > 
> > > Just to be sure it doesn't regress (I should be able to have a D420
> > > tomorrow to test here too).
> > 
> > It doesn't resume...
> 
> And "pm-suspend --quirk-vbe-post --quirk-vbestate-restore" works on the
> console (it's what's currently in the fdi file)
> 
> Using only vbe post works on both the console and X (I suspended and
> resumed twice under X using just this quirk).
> 
> Won't resume on the console without vbe post.

Bastien is using the new intel driver, which is handling all the
resolution mode change and doesn't not rely on BIOS anymore. And I think
vbestate-restore is probably conflicting with the new driver but is
probably required with the old version of the driver.

I'm not sure how we should deal with this..

-- 
Frederic Crozat <fcrozat at mandriva.com>
Mandriva