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

Re: this patch in next X?



On Mon, 17 Jun 2002, Edward Betts wrote:

>Date: Mon, 17 Jun 2002 04:20:19 +0100
>From: Edward Betts <edward@debian.org>
>To: Debian X <debian-x@lists.debian.org>
>Content-Type: text/plain; charset=us-ascii
>Subject: this patch in next X?
>
>Any chance of seeing this patch in the next X?
>
>http://www.xfree86.org/pipermail/xpert/2002-May/017542.html
>
>I am thinking it would fix the video overlay on my laptop.

I'm not ultra familiar with the i810/i830 video driver code,
however the patch included in the above email seems like it might
fix things on i830 but possibly break i810.

Perhaps a better fix would be to read the values from an 
i830RegRec, which would need to be filled in elsewhere first?  I 
see that declared in the headers but not really used much 
anywhere.  Perhaps the driver is incomplete.

I'd at least test this patch on various i810/i830 video hardware
prior to committing it.  Also, the INREG(0x60000) should be
INREG(HTOTAL_A) to be a bit more readable.

Hope this helps.


-- 
Mike A. Harris                  Shipping/mailing address:
OS Systems Engineer             190 Pittsburgh Ave., Sault Ste. Marie,
XFree86 maintainer              Ontario, Canada, P6C 5B3
Red Hat Inc.
http://www.redhat.com           ftp://people.redhat.com/mharris


-- 
To UNSUBSCRIBE, email to debian-x-request@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org



Reply to: