drm/sun4i: use simpler remove_conflicting_framebuffers(NULL)
authorMichał Mirosław <mirq-linux@rere.qmqm.pl>
Sat, 1 Sep 2018 14:08:51 +0000 (16:08 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 3 Sep 2018 16:18:01 +0000 (18:18 +0200)
Use remove_conflicting_framebuffers(NULL) instead of duplicating it.

Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/7d6d5bc4deac322b1351533c989cb3583e91be49.1535810304.git.mirq-linux@rere.qmqm.pl
drivers/gpu/drm/sun4i/sun4i_drv.c

index dd19d674055c625c20e4e78cfdc61b50fefa6743..3a10368acf939aa649a5d21af3f9bbed3d45b2b3 100644 (file)
@@ -61,22 +61,6 @@ static struct drm_driver sun4i_drv_driver = {
        /* Frame Buffer Operations */
 };
 
-static void sun4i_remove_framebuffers(void)
-{
-       struct apertures_struct *ap;
-
-       ap = alloc_apertures(1);
-       if (!ap)
-               return;
-
-       /* The framebuffer can be located anywhere in RAM */
-       ap->ranges[0].base = 0;
-       ap->ranges[0].size = ~0;
-
-       drm_fb_helper_remove_conflicting_framebuffers(ap, "sun4i-drm-fb", false);
-       kfree(ap);
-}
-
 static int sun4i_drv_bind(struct device *dev)
 {
        struct drm_device *drm;
@@ -119,7 +103,7 @@ static int sun4i_drv_bind(struct device *dev)
        drm->irq_enabled = true;
 
        /* Remove early framebuffers (ie. simplefb) */
-       sun4i_remove_framebuffers();
+       drm_fb_helper_remove_conflicting_framebuffers(NULL, "sun4i-drm-fb", false);
 
        /* Create our framebuffer */
        ret = sun4i_framebuffer_init(drm);