lkml.org 
[lkml]   [2016]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Subject[PATCH 08/14] GPU-DRM-OMAP: Rename a jump label in dmm_txn_commit()
    From
    Date
    From: Markus Elfring <elfring@users.sourceforge.net>
    Date: Wed, 21 Sep 2016 17:32:42 +0200

    Adjust a jump target so that redundant checks can be avoided at the end.

    Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
    ---
    drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
    index 5f6f21b..c8ced158 100644
    --- a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
    +++ b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
    @@ -269,7 +269,7 @@ static int dmm_txn_commit(struct dmm_txn *txn, bool wait)
    if (!txn->last_pat) {
    dev_err(engine->dmm->dev, "need at least one txn\n");
    ret = -EINVAL;
    - goto cleanup;
    + goto release_engine;
    }

    txn->last_pat->next_pa = 0;
    @@ -281,7 +281,7 @@ static int dmm_txn_commit(struct dmm_txn *txn, bool wait)
    ret = wait_status(engine, DMM_PATSTATUS_READY);
    if (ret) {
    ret = -EFAULT;
    - goto cleanup;
    + goto release_engine;
    }

    /* mark whether it is async to denote list management in IRQ handler */
    @@ -301,9 +301,9 @@ static int dmm_txn_commit(struct dmm_txn *txn, bool wait)
    }
    }

    -cleanup:
    /* only place engine back on list if we are done with it */
    if (ret || wait)
    + release_engine:
    release_engine(engine);

    return ret;
    --
    2.10.0
    \
     
     \ /
      Last update: 2016-09-23 00:00    [W:3.978 / U:0.288 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site