Commit 2abe1f25 authored by Wei Yongjun's avatar Wei Yongjun Committed by Rob Clark

drm/msm/dsi: Fix return value check in msm_dsi_host_set_display_mode()

In case of error, the function drm_mode_duplicate() returns NULL
pointer not ERR_PTR(). The IS_ERR() test in the return value check
should be replaced with NULL test.
Signed-off-by: default avatarWei Yongjun <yongjun_wei@trendmicro.com.cn>
Reviewed-by: default avatarArchit Taneja <architt@codeaurora.org>
Signed-off-by: default avatarRob Clark <robdclark@gmail.com>
parent f1427016
...@@ -2278,9 +2278,9 @@ int msm_dsi_host_set_display_mode(struct mipi_dsi_host *host, ...@@ -2278,9 +2278,9 @@ int msm_dsi_host_set_display_mode(struct mipi_dsi_host *host,
} }
msm_host->mode = drm_mode_duplicate(msm_host->dev, mode); msm_host->mode = drm_mode_duplicate(msm_host->dev, mode);
if (IS_ERR(msm_host->mode)) { if (!msm_host->mode) {
pr_err("%s: cannot duplicate mode\n", __func__); pr_err("%s: cannot duplicate mode\n", __func__);
return PTR_ERR(msm_host->mode); return -ENOMEM;
} }
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment