diff mbox

[4/4] drm/vc4: Don't use drm_put_dev

Message ID 20161208110739.24417-4-daniel.vetter@ffwll.ch (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Vetter Dec. 8, 2016, 11:07 a.m. UTC
vc4 already has a proper load sequence, but the unload one needed some
fixups: First unregister, and last drop the final ref.

Cc: Eric Anholt <eric@anholt.net>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
 drivers/gpu/drm/vc4/vc4_drv.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Eric Anholt Dec. 9, 2016, 11:44 p.m. UTC | #1
Daniel Vetter <daniel.vetter@ffwll.ch> writes:

> vc4 already has a proper load sequence, but the unload one needed some
> fixups: First unregister, and last drop the final ref.
>
> Cc: Eric Anholt <eric@anholt.net>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Pulled this one.  Thanks!
diff mbox

Patch

diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c
index 1dab9e5b3689..bfaf4e36f7ae 100644
--- a/drivers/gpu/drm/vc4/vc4_drv.c
+++ b/drivers/gpu/drm/vc4/vc4_drv.c
@@ -277,12 +277,14 @@  static void vc4_drm_unbind(struct device *dev)
 	struct drm_device *drm = platform_get_drvdata(pdev);
 	struct vc4_dev *vc4 = to_vc4_dev(drm);
 
+	drm_dev_unregister(drm);
+
 	if (vc4->fbdev)
 		drm_fbdev_cma_fini(vc4->fbdev);
 
 	drm_mode_config_cleanup(drm);
 
-	drm_put_dev(drm);
+	drm_dev_unref(drm);
 }
 
 static const struct component_master_ops vc4_drm_ops = {