Re: Current suspend issues on Lenovo X200

[Nico -telmich- Schottelius]

Jesse Barnes [Fri, Oct 24, 2008 at 11:23:17AM -0700]:
> On Friday, October 24, 2008 7:28 am Andres Freund wrote:
> > Hi Nico,
> >
> > On Thursday 23 October 2008 22:43:26 Nico -telmich- Schottelius wrote:
> > >  - 2.6.27-next-20081023:
> > >    -> no X running (because xorg does not work with that kernel)
> >
> > I think what you need is one rather short fix for the intel xorg driver:
> >
> > diff --git a/src/i830_driver.c b/src/i830_driver.c
> > index c1d61f4..eaf5d27 100644
> > --- a/src/i830_driver.c
> > +++ b/src/i830_driver.c
> > @@ -502,8 +502,8 @@ I830DetectMemory(ScrnInfoPtr pScrn)
> >     range = gtt_size + 4;
> >
> >     /* new 4 series hardware has seperate GTT stolen with GFX stolen */
> > - if (IS_G4X(pI830))
> > - range = 0;
> > + if (IS_G4X(pI830) || IS_GM45(pI830))
> > + range = 4;
> >
> >     if (IS_I85X(pI830) || IS_I865G(pI830) || IS_I9XX(pI830)) {
> >        switch (gmch_ctrl & I855_GMCH_GMS_MASK) {
> >
> > (Out of the "[git pull] agp patches for 2.6.28-rc1" thread)
> 
> Something like this might also help.
> 
> G4x chips do their own memory arbitration management, so we shouldn't trying
> to save/restore the DSPARB reg.
> 

I'm a bit confused. Which of both patches against 2.6.28-rc1 should I try?

Nico

Attachment: signature.asc
Description: Digital signature