Since components have their own cleanup routines calling
drm_mode_config_cleanup before component_unbind_all causes errors
due to double free of KMS objects. The patch fixes it by changing
de-initialization order. Now it is exactly opposite to init order.

Signed-off-by: Andrzej Hajda <a.hajda at samsung.com>
---
 drivers/gpu/drm/exynos/exynos_drm_drv.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c 
b/drivers/gpu/drm/exynos/exynos_drm_drv.c
index e11c4d6..b2c710a 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_drv.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_drv.c
@@ -136,14 +136,14 @@ static int exynos_drm_unload(struct drm_device *dev)
        exynos_drm_device_subdrv_remove(dev);

        exynos_drm_fbdev_fini(dev);
-       drm_vblank_cleanup(dev);
        drm_kms_helper_poll_fini(dev);
-       drm_mode_config_cleanup(dev);

+       component_unbind_all(dev->dev, dev);
+       drm_vblank_cleanup(dev);
+       drm_mode_config_cleanup(dev);
        drm_release_iommu_mapping(dev);
-       kfree(dev->dev_private);

-       component_unbind_all(dev->dev, dev);
+       kfree(dev->dev_private);
        dev->dev_private = NULL;

        return 0;
-- 
1.9.1

Reply via email to