drm/hisilicon: Use managed mode-config init
authorTian Tao <tiantao6@hisilicon.com>
Mon, 7 Dec 2020 11:16:17 +0000 (19:16 +0800)
committerTian Tao <tiantao6@hisilicon.com>
Wed, 9 Dec 2020 00:33:31 +0000 (19:33 -0500)
Using drmm_mode_config_init() sets up managed release of modesetting
resources.

v2:
Remove the unused structure member variable mode_config_initialized.

Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/1607339778-20460-2-git-send-email-tiantao6@hisilicon.com
drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c
drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h

index 3687753dba9addfa9207e9cc6ddd87a132d5bf29..7f01213d3684298bef5c3409710ef233b81b3df2 100644 (file)
@@ -96,8 +96,9 @@ static int hibmc_kms_init(struct hibmc_drm_private *priv)
        struct drm_device *dev = &priv->dev;
        int ret;
 
-       drm_mode_config_init(dev);
-       priv->mode_config_initialized = true;
+       ret = drmm_mode_config_init(dev);
+       if (ret)
+               return ret;
 
        dev->mode_config.min_width = 0;
        dev->mode_config.min_height = 0;
@@ -125,14 +126,6 @@ static int hibmc_kms_init(struct hibmc_drm_private *priv)
        return 0;
 }
 
-static void hibmc_kms_fini(struct hibmc_drm_private *priv)
-{
-       if (priv->mode_config_initialized) {
-               drm_mode_config_cleanup(&priv->dev);
-               priv->mode_config_initialized = false;
-       }
-}
-
 /*
  * It can operate in one of three modes: 0, 1 or Sleep.
  */
@@ -262,7 +255,6 @@ static int hibmc_unload(struct drm_device *dev)
        drm_atomic_helper_shutdown(dev);
 
        pci_disable_msi(dev->pdev);
-       hibmc_kms_fini(priv);
        dev->dev_private = NULL;
        return 0;
 }
index a49c10e3ec34fe8d4178b5a1235ce422fd5e89f2..7d263f4d707845858a7f9f6253efe8a6d1c974be 100644 (file)
@@ -42,7 +42,6 @@ struct hibmc_drm_private {
        struct drm_crtc crtc;
        struct drm_encoder encoder;
        struct hibmc_connector connector;
-       bool mode_config_initialized;
 };
 
 static inline struct hibmc_connector *to_hibmc_connector(struct drm_connector *connector)