diff options
author | Lothar Waßmann <LW@KARO-electronics.de> | 2016-07-12 09:30:01 -0400 |
---|---|---|
committer | Philipp Zabel <p.zabel@pengutronix.de> | 2016-07-12 12:26:06 -0400 |
commit | 76ecd9c9fb24b014a6f33fbb1287ede3be12158b (patch) | |
tree | daa8f92a7ebaf951a6a0c1953bf21eb6a17b9342 /drivers/gpu/drm/imx/parallel-display.c | |
parent | b575b31387f64f63019c88a0ddbf1e0cc0cd8cd9 (diff) |
drm/imx: parallel-display: check return code from of_get_drm_display_mode()
of_get_drm_display_mode() may fail. Check its return code and bail out
on error.
Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Diffstat (limited to 'drivers/gpu/drm/imx/parallel-display.c')
-rw-r--r-- | drivers/gpu/drm/imx/parallel-display.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c index 7b7bb3100831..4abac21c5c9b 100644 --- a/drivers/gpu/drm/imx/parallel-display.c +++ b/drivers/gpu/drm/imx/parallel-display.c | |||
@@ -73,10 +73,16 @@ static int imx_pd_connector_get_modes(struct drm_connector *connector) | |||
73 | 73 | ||
74 | if (np) { | 74 | if (np) { |
75 | struct drm_display_mode *mode = drm_mode_create(connector->dev); | 75 | struct drm_display_mode *mode = drm_mode_create(connector->dev); |
76 | int ret; | ||
76 | 77 | ||
77 | if (!mode) | 78 | if (!mode) |
78 | return -EINVAL; | 79 | return -EINVAL; |
79 | of_get_drm_display_mode(np, &imxpd->mode, OF_USE_NATIVE_MODE); | 80 | |
81 | ret = of_get_drm_display_mode(np, &imxpd->mode, | ||
82 | OF_USE_NATIVE_MODE); | ||
83 | if (ret) | ||
84 | return ret; | ||
85 | |||
80 | drm_mode_copy(mode, &imxpd->mode); | 86 | drm_mode_copy(mode, &imxpd->mode); |
81 | mode->type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED, | 87 | mode->type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED, |
82 | drm_mode_probed_add(connector, mode); | 88 | drm_mode_probed_add(connector, mode); |