[MPlayer-dev-eng] [PATCH] autodetect screen size for vo_mga fullscreen

[Jason Lunz]

dalias at aerifal.cx said:
> the problem with your patch is that "/dev/fb0" might not be the same
> device mga_vid is using.

that's true, but my patch only changes the default. If the value
detected from /dev/fb0 isn't right, you can still put in whatever you
like with -screenw and -screenh. So you're no worse off than before.

> unfortunately, i don't know any way around this problem without
> looking up pci bus id's and stuff, and even then with a multihead
> card, i think there can be two separate fb devices for the card, so
> it'd be hard to tell which is attached to mga_vid.

This would be ideal. Can someone tell me in what situation you would use
something other than /dev/fb0? I don't know when you'd even have another
framebuffer device than fb0, unless you have a multihead configuration.
Would another command-line option for the fb device help? is there
another more elegant way to detect which device to use?

> imo, the best solution, if we really want this functionality (which
> would be nice, although my mpconsole script handles it ok), would be
> to make the mga_vid driver actually read and export the screen
> dimensions from the device it's using. i'm not sure how easily this
> could be done, however.

yeah, that'd be nice. can anyone who knows more about mga_vid comment?
Anyway, notice that TOOLS/mpconsole is no better than my patch because
it uses fbset and fbset defaults to using /dev/fb0.

Jason