[MPlayer-dev-eng] Updated vidix driver for ATI cards.

[Xuebin Wu]

Hi,

There are some compile problems under MinGW with this file, and the
following diff may be helpful
The first problem is there is no <sys/mman.h> in MinGW, and I found that
it can be compiled without this header file.
The second problem is the type "u_int" is not defined in <sys/types.h>, so I
use u_int32_t instead.

Index: vidix/radeon_vid.c
===================================================================
--- vidix/radeon_vid.c (revision 23079)
+++ vidix/radeon_vid.c (working copy)
@@ -35,7 +35,10 @@
 #include <string.h>
 #include <math.h>
 #include <inttypes.h>
+
+#ifdef HAVE_SYS_MMAN_H
 #include <sys/mman.h>
+#endif

 #include "config.h"
 #include "libavutil/common.h"
@@ -2284,7 +2287,7 @@

  TopUVLine = ((int)(config->src.y/VertUVSubSample) < 0) ?  0:
(int)(config->src.y/VertUVSubSample);   /* Round rSrcTop down */
  BottomUVLine = (ceil(((config->src.y+config->src.h)/VertUVSubSample)) >
(config->src.h/VertUVSubSample))
- ? (config->src.h/VertUVSubSample)-1 : (u_int)ceil(((config->src.y+config->
src.h)/VertUVSubSample))-1;
+ ? (config->src.h/VertUVSubSample)-1 : (u_int32_t)ceil(((config->
src.y+config->src.h)/VertUVSubSample))-1;

  if (BottomUVLine >= TopUVLine)
  {



2007/4/18, Benjamin Zores <ben at geexbox.org>:
>
> Hi,
>
> The attached patch updates the ATI Rage128/Radeon Vidix driver from
> sf.net upstream vidix. I'd like some people with access to these types
> of GPUs to test the patch and tell me if -vo [cx]vidix still works for
> them (also tell if you're using Rage based or any Radeon card).
>
> It should theorically introduce support for all Radeon cards higher
> than 9600 and nearly all from the latest X series.
>
> No need to say that the patch itself is disgusting (old file replaced
> by the new one), unreadable and, if ever meant to be commited, will be
> applied this way (i can't waste a year trying to split 3000 lines of
> changes in this driver to have it commited piece by piece).
>
> Ben
>
> --
> "My life, and by extension everyone else's is meaningless."
> Bender, Futurama
>
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
>