> --- a/hw/vfio/pci.c > +++ b/hw/vfio/pci.c > @@ -3080,8 +3080,10 @@ static void vfio_realize(PCIDevice *pdev, Error > **errp) > error_setg(errp, "xres and yres properties require > display=on"); > goto out_teardown; > } > - if (vdev->dpy->edid_regs == NULL) { > - error_setg(errp, "xres and yres properties need edid support"); > + if (vdev->dpy->edid_regs == NULL && !vdev->enable_ramfb) { > + error_setg(errp, > + "xres and yres properties need edid support" > + " or ramfb=on"); > goto out_teardown; > } > }
I don't think this is useful. We should continue to allow xres and yres only in case the vfio device actually has edid support. cheers, Gerd