Skip to content

Commit

Permalink
drm/vboxvideo: Set up fbdev after registering device; remove error ch…
Browse files Browse the repository at this point in the history
…ecks

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]>
Reviewed-by: Noralf Trønnes <[email protected]>
Acked-by: Sam Ravnborg <[email protected]>
Acked-by: Gerd Hoffmann <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
  • Loading branch information
Thomas Zimmermann committed Apr 8, 2020
1 parent 66459fa commit 3662cd1
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions drivers/gpu/drm/vboxvideo/vbox_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -78,14 +78,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:
Expand Down

0 comments on commit 3662cd1

Please sign in to comment.