[MPlayer-cvslog] r24536 - trunk/vidix/nvidia_vid.c

[Reimar Doeffinger]

Hello,
On Mon, Sep 17, 2007 at 03:06:53PM +0200, Guillaume LECERF wrote:
> 2007/9/17, Sascha Sommer <saschasommer at freenet.de>:
> > Yes. It was at least correct for NV03, NV04 and NV15 (arch 10) as far as I can
> > tell.
> 
> Could you please test with the attached patch.

Even without testing I can see there is no way this patch could fix a
value of 3072 to 1024.

> If it doesn't work, I think I'll add a check for the chip
> architecture, and enable this new way of computing y only for chips >
> nv10.

Are you sure the new X-resolution code is any more correct on older cards?
I'd expect it to be pure luck that that it works, at least I doubt
nVidia would have reserved some bits wildly dispersed for resolutions
the card can't even handle anyway...

Greetings,
Reimar Doeffinger