Re: [Qemu-devel] [PATCH] Add resolutions via the command-line

[Benjamin Herrenschmidt <benh@kernel.crashing.org>]

On Tue, 2016-09-20 at 22:54 -0400, G 3 wrote:
> You really want to remove the included list of resolutions? I was  
> thinking about adding a lot more built-in resolutions in another  
> patch. A built-in list is very convenient.

I mean remove it from the driver and put it in OpenBIOS instead.

Ie. have openbios build the resolution list from the combination of what you put
in the options node and its own built-in list.

> As for the options node problem. We need a way to add resolutions  
> from the command-line for a Mac OS X guest. Using "-prom-env" isn't  
> an option because of some unknown bug.

Maybe, I can look into it later, but the above would fix it nicely.

>  I suppose it is possible to  
> fix this bug - if it is a bug. Another idea I have is to add a  
> completely different command-line option to QEMU just for  
> resolutions. The user could specify resolutions like this: -video- 
> resolutions 640x480,1024x768,1280x700.... This feature could place  
> the list of resolutions inside a property of the VGA device node. I  
> am betting the VGA driver could access this node and parse the list  
> of resolutions. So this: qemu -video-resolutions ---> OpenBIOS VGA  
> device node ---> VGA driver
> 
> Would anyone have another idea they would like to share?

As I said. Have OpenBIOS build the list of resolutions and put it in a
property (in binary, not ASCII form so the driver doesn't have to do
the mess it does now) in the DT node of the device itself.

Cheers,
Ben.