> -----Original Message-----
> From: Philippe Mathieu-Daudé [mailto:[email protected]]
> Sent: 12 April 2019 17:37
> To: Marc-André Lureau <[email protected]>; [email protected]
> Cc: Markus Armbruster <[email protected]>; Anthony Perard 
> <[email protected]>; Paul Durrant
> <[email protected]>; Stefano Stabellini <[email protected]>; Gerd 
> Hoffmann
> <[email protected]>; Paolo Bonzini <[email protected]>; Philippe 
> Mathieu-Daudé <[email protected]>
> Subject: [PATCH] vl: Add missing descriptions to the VGA adapters list
> 
> Some VGA adapters do not contain an helpful description,
> this can be confusing:
> 
>   $ qemu-system-arm -M virt -vga help
>   none
>   std                  standard VGA
>   cirrus               Cirrus VGA (default)
>   vmware               VMWare SVGA
>   xenfb
> 
> Add a description to the missing adapters:
> 
>   $ qemu-system-arm -M virt -vga help
>   none                 no graphic card
>   std                  standard VGA
>   cirrus               Cirrus VGA (default)
>   vmware               VMWare SVGA
>   xenfb                Xen paravirtualized framebuffer
> 
> Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
> ---
> Based-on: <[email protected]>

Reviewed-by: Paul Durrant <[email protected]>

> ---
>  vl.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/vl.c b/vl.c
> index 840c45c00fa..dfeda8bd7fc 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2029,6 +2029,7 @@ typedef struct VGAInterfaceInfo {
>  static const VGAInterfaceInfo vga_interfaces[VGA_TYPE_MAX] = {
>      [VGA_NONE] = {
>          .opt_name = "none",
> +        .name = "no graphic card",
>      },
>      [VGA_STD] = {
>          .opt_name = "std",
> @@ -2067,6 +2068,7 @@ static const VGAInterfaceInfo 
> vga_interfaces[VGA_TYPE_MAX] = {
>      },
>      [VGA_XENFB] = {
>          .opt_name = "xenfb",
> +        .name = "Xen paravirtualized framebuffer",
>      },
>  };
> 
> --
> 2.20.1

Reply via email to