drm/arm/hdlcd: Use drm_fb_cma_fbdev_init/fini()
authorNoralf Trønnes <noralf@tronnes.org>
Fri, 8 Dec 2017 19:37:35 +0000 (20:37 +0100)
committerLiviu Dudau <Liviu.Dudau@arm.com>
Tue, 31 Jul 2018 15:31:50 +0000 (16:31 +0100)
Use drm_fb_cma_fbdev_init() and drm_fb_cma_fbdev_fini() which relies on
the fact that drm_device holds a pointer to the drm_fb_helper structure.
This means that the driver doesn't have to keep track of that.
Also use the drm_fb_helper functions directly.

Cc: Liviu Dudau <liviu.dudau@arm.com>
Cc: Brian Starkey <brian.starkey@arm.com>
Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
Acked-by: Liviu Dudau <liviu.dudau@arm.com>
Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>
drivers/gpu/drm/arm/hdlcd_drv.c
drivers/gpu/drm/arm/hdlcd_drv.h

index 9800a9e388efe8e84d6fd1737cd9d344e08d7ed8..fd5b8c44af144415206802b5a751050f6058be51 100644 (file)
@@ -101,16 +101,9 @@ setup_fail:
        return ret;
 }
 
-static void hdlcd_fb_output_poll_changed(struct drm_device *drm)
-{
-       struct hdlcd_drm_private *hdlcd = drm->dev_private;
-
-       drm_fbdev_cma_hotplug_event(hdlcd->fbdev);
-}
-
 static const struct drm_mode_config_funcs hdlcd_mode_config_funcs = {
        .fb_create = drm_gem_fb_create,
-       .output_poll_changed = hdlcd_fb_output_poll_changed,
+       .output_poll_changed = drm_fb_helper_output_poll_changed,
        .atomic_check = drm_atomic_helper_check,
        .atomic_commit = drm_atomic_helper_commit,
 };
@@ -125,13 +118,6 @@ static void hdlcd_setup_mode_config(struct drm_device *drm)
        drm->mode_config.funcs = &hdlcd_mode_config_funcs;
 }
 
-static void hdlcd_lastclose(struct drm_device *drm)
-{
-       struct hdlcd_drm_private *hdlcd = drm->dev_private;
-
-       drm_fbdev_cma_restore_mode(hdlcd->fbdev);
-}
-
 static irqreturn_t hdlcd_irq(int irq, void *arg)
 {
        struct drm_device *drm = arg;
@@ -247,7 +233,7 @@ static struct drm_driver hdlcd_driver = {
        .driver_features = DRIVER_HAVE_IRQ | DRIVER_GEM |
                           DRIVER_MODESET | DRIVER_PRIME |
                           DRIVER_ATOMIC,
-       .lastclose = hdlcd_lastclose,
+       .lastclose = drm_fb_helper_lastclose,
        .irq_handler = hdlcd_irq,
        .irq_preinstall = hdlcd_irq_preinstall,
        .irq_postinstall = hdlcd_irq_postinstall,
@@ -322,14 +308,9 @@ static int hdlcd_drm_bind(struct device *dev)
        drm_mode_config_reset(drm);
        drm_kms_helper_poll_init(drm);
 
-       hdlcd->fbdev = drm_fbdev_cma_init(drm, 32,
-                                         drm->mode_config.num_connector);
-
-       if (IS_ERR(hdlcd->fbdev)) {
-               ret = PTR_ERR(hdlcd->fbdev);
-               hdlcd->fbdev = NULL;
+       ret = drm_fb_cma_fbdev_init(drm, 32, 0);
+       if (ret)
                goto err_fbdev;
-       }
 
        ret = drm_dev_register(drm, 0);
        if (ret)
@@ -338,10 +319,7 @@ static int hdlcd_drm_bind(struct device *dev)
        return 0;
 
 err_register:
-       if (hdlcd->fbdev) {
-               drm_fbdev_cma_fini(hdlcd->fbdev);
-               hdlcd->fbdev = NULL;
-       }
+       drm_fb_cma_fbdev_fini(drm);
 err_fbdev:
        drm_kms_helper_poll_fini(drm);
 err_vblank:
@@ -367,10 +345,7 @@ static void hdlcd_drm_unbind(struct device *dev)
        struct hdlcd_drm_private *hdlcd = drm->dev_private;
 
        drm_dev_unregister(drm);
-       if (hdlcd->fbdev) {
-               drm_fbdev_cma_fini(hdlcd->fbdev);
-               hdlcd->fbdev = NULL;
-       }
+       drm_fb_cma_fbdev_fini(drm);
        drm_kms_helper_poll_fini(drm);
        component_unbind_all(dev, drm);
        of_node_put(hdlcd->crtc.port);
index 58d75d7e1b2054abc477b3e4ab591526bdd82412..fd438d177b644d6f7d0fde99a32802dfa2bd6dbc 100644 (file)
@@ -9,7 +9,6 @@
 struct hdlcd_drm_private {
        void __iomem                    *mmio;
        struct clk                      *clk;
-       struct drm_fbdev_cma            *fbdev;
        struct drm_crtc                 crtc;
        struct drm_plane                *plane;
 #ifdef CONFIG_DEBUG_FS