diff mbox

DRM: i.MX: parallel display: Support probe deferral for finding DRM panel

Message ID 1424660991-7926-1-git-send-email-Ying.Liu@freescale.com (mailing list archive)
State New, archived
Headers show

Commit Message

Liu Ying Feb. 23, 2015, 3:09 a.m. UTC
Signed-off-by: Liu Ying <Ying.Liu@freescale.com>
---
 drivers/gpu/drm/imx/parallel-display.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Philipp Zabel Feb. 23, 2015, 4:16 p.m. UTC | #1
Hi,

Am Montag, den 23.02.2015, 11:09 +0800 schrieb Liu Ying:
> Signed-off-by: Liu Ying <Ying.Liu@freescale.com>

thank you for the patch. I've applied it to my branch.

> ---
>  drivers/gpu/drm/imx/parallel-display.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c
> index 5e83e00..900dda6 100644
> --- a/drivers/gpu/drm/imx/parallel-display.c
> +++ b/drivers/gpu/drm/imx/parallel-display.c
> @@ -236,8 +236,11 @@ static int imx_pd_bind(struct device *dev, struct device *master, void *data)
>  	}
>  
>  	panel_node = of_parse_phandle(np, "fsl,panel", 0);

This property is not documented anywhere and I'd like to get rid of this
in the future. This should be replaced with of graph endpoint links.

regards
Philipp
diff mbox

Patch

diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c
index 5e83e00..900dda6 100644
--- a/drivers/gpu/drm/imx/parallel-display.c
+++ b/drivers/gpu/drm/imx/parallel-display.c
@@ -236,8 +236,11 @@  static int imx_pd_bind(struct device *dev, struct device *master, void *data)
 	}
 
 	panel_node = of_parse_phandle(np, "fsl,panel", 0);
-	if (panel_node)
+	if (panel_node) {
 		imxpd->panel = of_drm_find_panel(panel_node);
+		if (!imxpd->panel)
+			return -EPROBE_DEFER;
+	}
 
 	imxpd->dev = dev;