The lable fail_connector should placed before fail_encoder since
encoder was initialized before connector. which should also be
called after connector initialization failed.

Signed-off-by: Meng Yi <meng.yi at nxp.com>
---
 drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c 
b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
index c564ec6..759527e 100644
--- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
+++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
@@ -51,9 +51,9 @@ int fsl_dcu_drm_modeset_init(struct fsl_dcu_drm_device 
*fsl_dev)
        drm_kms_helper_poll_init(fsl_dev->drm);

        return 0;
-fail_encoder:
-       fsl_dev->crtc.funcs->destroy(&fsl_dev->crtc);
 fail_connector:
        fsl_dev->encoder.funcs->destroy(&fsl_dev->encoder);
+fail_encoder:
+       fsl_dev->crtc.funcs->destroy(&fsl_dev->crtc);
        return ret;
 }
-- 
2.1.0.27.g96db324

Reply via email to