[PATCH] intel: Hide cursor before switching modes

Maxim Levitsky maximlevitsky at gmail.com
Sun Mar 30 12:41:44 PDT 2008


On Sunday, 23 March 2008 09:23:04 Maxim Levitsky wrote:
> On Saturday, 22 March 2008 23:26:28 Keith Packard wrote:
> > 
> > On Sat, 2008-03-22 at 22:14 +0200, Maxim Levitsky wrote:
> > 
> > > diff --git a/src/i830_display.c b/src/i830_display.c
> > > index 5e52aac..0c22a0a 100644
> > > --- a/src/i830_display.c
> > > +++ b/src/i830_display.c
> > > @@ -905,6 +905,7 @@ i830_crtc_prepare (xf86CrtcPtr crtc)
> > >      if (i830_use_fb_compression(crtc))
> > >          i830_disable_fb_compression(crtc);
> > >      crtc->funcs->dpms (crtc, DPMSModeOff);
> > > +    crtc->funcs->hide_cursor (crtc);
> > >  }
> > 
> > heh. I had thought up almost exactly the same patch just after mailing
> > you last time. I think this might be a better order for some intel chips
> > which really don't like being poked while the clocks aren't running:
> > 
> ....
> > If you can test this, I'll stuff it into the driver.
> > 
> I tested this, everything works fine.
> I guess that this annoying bug is finally fixed.
Don't see this fix commited, why?

Best regards,
	Maxim Levitsky



More information about the xorg mailing list