lkml.org 
[lkml]   [2018]   [Mar]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL for 4.9 015/219] drm/sun4i: Fix up error path cleanup for master bind function
    Date
    From: Chen-Yu Tsai <wens@csie.org>

    [ Upstream commit 9d56defb44b15427f4342c543a70fb7886fc06f5 ]

    The master bind function calls numerous drm functions which initialize
    underlying structures. It also tries to bind the various components
    of the display pipeline, some of which may add additional drm objects.

    This patch adds proper cleanup functions in the error path of the
    master bind function.

    This requires the patch "drm/sun4i: Move drm_mode_config_cleanup call
    to main driver", which splits out drm_mode_config_cleanup from
    sun4i_framebuffer_free so we can call it separately.

    Signed-off-by: Chen-Yu Tsai <wens@csie.org>
    Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
    Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
    ---
    drivers/gpu/drm/sun4i/sun4i_drv.c | 16 +++++++++++-----
    1 file changed, 11 insertions(+), 5 deletions(-)

    diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
    index 1feec34ca9dd..9e77fc034e0a 100644
    --- a/drivers/gpu/drm/sun4i/sun4i_drv.c
    +++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
    @@ -145,7 +145,7 @@ static int sun4i_drv_bind(struct device *dev)
    ret = component_bind_all(drm->dev, drm);
    if (ret) {
    dev_err(drm->dev, "Couldn't bind all pipelines components\n");
    - goto free_drm;
    + goto cleanup_mode_config;
    }

    /* Create our layers */
    @@ -153,7 +153,7 @@ static int sun4i_drv_bind(struct device *dev)
    if (IS_ERR(drv->layers)) {
    dev_err(drm->dev, "Couldn't create the planes\n");
    ret = PTR_ERR(drv->layers);
    - goto free_drm;
    + goto cleanup_mode_config;
    }

    /* Create our CRTC */
    @@ -161,7 +161,7 @@ static int sun4i_drv_bind(struct device *dev)
    if (!drv->crtc) {
    dev_err(drm->dev, "Couldn't create the CRTC\n");
    ret = -EINVAL;
    - goto free_drm;
    + goto cleanup_mode_config;
    }
    drm->irq_enabled = true;

    @@ -173,7 +173,7 @@ static int sun4i_drv_bind(struct device *dev)
    if (IS_ERR(drv->fbdev)) {
    dev_err(drm->dev, "Couldn't create our framebuffer\n");
    ret = PTR_ERR(drv->fbdev);
    - goto free_drm;
    + goto cleanup_mode_config;
    }

    /* Enable connectors polling */
    @@ -181,10 +181,16 @@ static int sun4i_drv_bind(struct device *dev)

    ret = drm_dev_register(drm, 0);
    if (ret)
    - goto free_drm;
    + goto finish_poll;

    return 0;

    +finish_poll:
    + drm_kms_helper_poll_fini(drm);
    + sun4i_framebuffer_free(drm);
    +cleanup_mode_config:
    + drm_mode_config_cleanup(drm);
    + drm_vblank_cleanup(drm);
    free_drm:
    drm_dev_unref(drm);
    return ret;
    --
    2.14.1
    \
     
     \ /
      Last update: 2022-09-17 16:04    [W:4.617 / U:0.160 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site