On Wed, 24 May 2023 at 12:14, Marcin Juszkiewicz <marcin.juszkiew...@linaro.org> wrote: > > arm/sbsa-ref uses Bochs-display graphics card and without it being > present in vga_interfaces "-vga none" argument handling cannot be added. > > Signed-off-by: Marcin Juszkiewicz <marcin.juszkiew...@linaro.org> > --- > hw/pci/pci.c | 2 ++ > include/sysemu/sysemu.h | 2 +- > softmmu/vl.c | 6 ++++++ > 3 files changed, 9 insertions(+), 1 deletion(-) > > diff --git a/hw/pci/pci.c b/hw/pci/pci.c > index 1cc7c89036..edac76ea15 100644 > --- a/hw/pci/pci.c > +++ b/hw/pci/pci.c > @@ -1937,6 +1937,8 @@ PCIDevice *pci_vga_init(PCIBus *bus) > return pci_create_simple(bus, -1, "vmware-svga"); > case VGA_VIRTIO: > return pci_create_simple(bus, -1, "virtio-vga"); > + case VGA_BOCHS: > + return pci_create_simple(bus, -1, "bochs-display"); > case VGA_NONE: > default: /* Other non-PCI types. Checking for unsupported types is > already > done in vl.c. */ > diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h > index 25be2a692e..9713a1b470 100644 > --- a/include/sysemu/sysemu.h > +++ b/include/sysemu/sysemu.h > @@ -29,7 +29,7 @@ extern int autostart; > > typedef enum { > VGA_NONE, VGA_STD, VGA_CIRRUS, VGA_VMWARE, VGA_XENFB, VGA_QXL, > - VGA_TCX, VGA_CG3, VGA_DEVICE, VGA_VIRTIO, > + VGA_TCX, VGA_CG3, VGA_DEVICE, VGA_VIRTIO, VGA_BOCHS, > VGA_TYPE_MAX, > } VGAInterfaceType;
Is it really right that we want to add more support to this legacy-command-line-option interface? We don't list every graphics device we support here... -- PMM