Re: [PATCH] XRANDR: Make sure that hardware cursor is inside crtc all the time.

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

 



On Saturday, 22 March 2008 19:16:16 Keith Packard wrote:
> 
> On Sat, 2008-03-22 at 16:03 +0200, Maxim Levitsky wrote:
> 
> > +	/* Adjust cursor position*/
> > +	xf86_reload_cursors (crtc->scrn->pScreen);
> 
> I suspect a better plan would be to turn off the cursor before setting
> the mode, then reload it afterwards. Otherwise, you may ask the driver
> to position the cursors badly for the current mode.
> 
> Splitting xf86_reload_cursors into two pieces, one which hides them and
> another which loads them should do the trick.
> 
I was thinking about this, but why?
Increasing the resolution leaves the cursor where it is
and decreasing moves it toward upper left corner which is correct for both lower resolution
and higher too.

Besides there is a risk that even disabled cursor can make hardware mad if offscreen
I think it is better to move the cursor in current resolution, and then do the modesetting


Best regards,
	Maxim Levitsky
_______________________________________________
xorg mailing list
xorg@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/xorg

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [X Forum]     [Intel Graphics]     [AMD Graphics]     [Nouveau Driver]     [XFree86]     [XFree86 Newbie]     [IETF Annouce]     [Security]     [Fontconfig]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Video for Linux]     [Linux RAID]

  Powered by Linux