diff options
author | Abhinav Kumar <abhinavk@codeaurora.org> | 2018-04-16 22:05:30 -0400 |
---|---|---|
committer | Rob Clark <robdclark@gmail.com> | 2018-06-03 16:48:59 -0400 |
commit | 79ebc86c41959113009e3d9b17dc2d4547512ac9 (patch) | |
tree | 8f7e77d83a6485f1c71d442f45cf6ff14b29633e | |
parent | 6d08b06e67cd117f6992c46611dfb4ce267cd71e (diff) |
drm/msm/dsi: check return value for video done waits
Check for the return value of wait for video
done waits and print appropriate error message.
Signed-off-by: Abhinav Kumar <abhinavk@codeaurora.org>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Rob Clark <robdclark@gmail.com>
-rw-r--r-- | drivers/gpu/drm/msm/dsi/dsi_host.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c index 7a03a9489708..93a3cdde72ef 100644 --- a/drivers/gpu/drm/msm/dsi/dsi_host.c +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c | |||
@@ -986,13 +986,19 @@ static void dsi_set_tx_power_mode(int mode, struct msm_dsi_host *msm_host) | |||
986 | 986 | ||
987 | static void dsi_wait4video_done(struct msm_dsi_host *msm_host) | 987 | static void dsi_wait4video_done(struct msm_dsi_host *msm_host) |
988 | { | 988 | { |
989 | u32 ret = 0; | ||
990 | struct device *dev = &msm_host->pdev->dev; | ||
991 | |||
989 | dsi_intr_ctrl(msm_host, DSI_IRQ_MASK_VIDEO_DONE, 1); | 992 | dsi_intr_ctrl(msm_host, DSI_IRQ_MASK_VIDEO_DONE, 1); |
990 | 993 | ||
991 | reinit_completion(&msm_host->video_comp); | 994 | reinit_completion(&msm_host->video_comp); |
992 | 995 | ||
993 | wait_for_completion_timeout(&msm_host->video_comp, | 996 | ret = wait_for_completion_timeout(&msm_host->video_comp, |
994 | msecs_to_jiffies(70)); | 997 | msecs_to_jiffies(70)); |
995 | 998 | ||
999 | if (ret <= 0) | ||
1000 | dev_err(dev, "wait for video done timed out\n"); | ||
1001 | |||
996 | dsi_intr_ctrl(msm_host, DSI_IRQ_MASK_VIDEO_DONE, 0); | 1002 | dsi_intr_ctrl(msm_host, DSI_IRQ_MASK_VIDEO_DONE, 0); |
997 | } | 1003 | } |
998 | 1004 | ||