drm/hisilicon/hibmc: Fix wrong pointer passed to PTR_ERR()
authorWei Yongjun <weiyongjun1@huawei.com>
Thu, 12 Jan 2017 15:19:21 +0000 (15:19 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 23 Jan 2017 08:02:28 +0000 (09:02 +0100)
PTR_ERR should access the value just tested by IS_ERR, otherwise
the wrong error code will be returned.

Fixes: d1667b86795a ("drm/hisilicon/hibmc: Add support for frame buffer")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20170112151921.16538-1-weiyj.lk@gmail.com
drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c

index 7a6957ae4b44d07b540cb7c77fca821c8fb2cab2..16fe79053ee1ebf57112e0189d62e0b394a510da 100644 (file)
@@ -121,7 +121,7 @@ static int hibmc_drm_fb_create(struct drm_fb_helper *helper,
 
        hi_fbdev->fb = hibmc_framebuffer_init(priv->dev, &mode_cmd, gobj);
        if (IS_ERR(hi_fbdev->fb)) {
-               ret = PTR_ERR(info);
+               ret = PTR_ERR(hi_fbdev->fb);
                DRM_ERROR("failed to initialize framebuffer: %d\n", ret);
                goto out_release_fbi;
        }