Generic fbdev support is a DRM client. Set it up after registering
the new DRM device. Remove the error checks as the driver's probe
function should not depend on a DRM client's state.

Signed-off-by: Thomas Zimmermann <[email protected]>
---
 drivers/gpu/drm/vboxvideo/vbox_drv.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/vboxvideo/vbox_drv.c 
b/drivers/gpu/drm/vboxvideo/vbox_drv.c
index d685ec197fa05..282348e071fe3 100644
--- a/drivers/gpu/drm/vboxvideo/vbox_drv.c
+++ b/drivers/gpu/drm/vboxvideo/vbox_drv.c
@@ -82,14 +82,12 @@ static int vbox_pci_probe(struct pci_dev *pdev, const 
struct pci_device_id *ent)
        if (ret)
                goto err_mode_fini;
 
-       ret = drm_fbdev_generic_setup(&vbox->ddev, 32);
-       if (ret)
-               goto err_irq_fini;
-
        ret = drm_dev_register(&vbox->ddev, 0);
        if (ret)
                goto err_irq_fini;
 
+       drm_fbdev_generic_setup(&vbox->ddev, 32);
+
        return 0;
 
 err_irq_fini:
-- 
2.26.0

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to