On Thu, 23 Jan 2003 [EMAIL PROTECTED] wrote:

>   Does anybody see problems with the attached patch?
>   I think there is some misnaming in the register field names, i.e. instead
> of "pInt->ax" maybe it should now be called "pInt->eax", anyway, this patch
> does not change the int10 infrastructure, and allow the vesa driver to work
> with some SiS cards that stopped working some time ago.
>   This isn't a very important patch as there is a sis driver, but I feel
> better knowing it is working...

> (I have sent a smaller patch to Egbert Eich a few days ago, but maybe the
> email was lost, sorry if you are looking at it Egbert...)

I'll look this over.  BTW, ...

Index: xc/programs/Xserver/hw/xfree86/os-support/vbe/vbe.c
===================================================================
RCS file: /opt/XFree86/xc/programs/Xserver/hw/xfree86/os-support/vbe/vbe.c,v
retrieving revision 1.23
diff -u -r1.23 vbe.c
--- xc/programs/Xserver/hw/xfree86/os-support/vbe/vbe.c 18 Jan 2003 15:22:35 -0000     
 1.23
+++ xc/programs/Xserver/hw/xfree86/os-support/vbe/vbe.c 23 Jan 2003 01:43:45 -0000

        [...]

@@ -1066,6 +1067,6 @@
        break;
     }
     xf86ExecX86int10(pVbe->pInt10);
-    return (pVbe->pInt10->ax == 0x4f);
+    return (R16(pVbe->pInt10->ax == 0x4f));
 }


... should be ...

+    return (R16(pVbe->pInt10->ax) == 0x4f);

Marc.

+----------------------------------+-----------------------------------+
|  Marc Aurele La France           |  work:   1-780-492-9310           |
|  Computing and Network Services  |  fax:    1-780-492-1729           |
|  352 General Services Building   |  email:  [EMAIL PROTECTED]          |
|  University of Alberta           +-----------------------------------+
|  Edmonton, Alberta               |                                   |
|  T6G 2H1                         |     Standard disclaimers apply    |
|  CANADA                          |                                   |
+----------------------------------+-----------------------------------+
XFree86 Core Team member.  ATI driver and X server internals.

_______________________________________________
Devel mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/devel

Reply via email to