浏览代码

drm/sun4i: defer only if we didn't find our panel

Our code currently defers our probe on any error, even if we were not
expecting to have one at all.

Make sure we return -EPROBE_DEFER only when we were supposed to have a
panel, but it's not probed yet.

Also fix a typo while we're at it.

Fixes: 29e57fab97fc ("drm: sun4i: Add RGB output")
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Maxime Ripard 9 年之前
父节点
当前提交
0bbbb00bda
共有 1 个文件被更改,包括 9 次插入8 次删除
  1. 9 8
      drivers/gpu/drm/sun4i/sun4i_tcon.c

+ 9 - 8
drivers/gpu/drm/sun4i/sun4i_tcon.c

@@ -425,11 +425,11 @@ static struct drm_panel *sun4i_tcon_find_panel(struct device_node *node)
 
 
 	remote = of_graph_get_remote_port_parent(end_node);
 	remote = of_graph_get_remote_port_parent(end_node);
 	if (!remote) {
 	if (!remote) {
-		DRM_DEBUG_DRIVER("Enable to parse remote node\n");
+		DRM_DEBUG_DRIVER("Unable to parse remote node\n");
 		return ERR_PTR(-EINVAL);
 		return ERR_PTR(-EINVAL);
 	}
 	}
 
 
-	return of_drm_find_panel(remote);
+	return of_drm_find_panel(remote) ?: ERR_PTR(-EPROBE_DEFER);
 }
 }
 
 
 static int sun4i_tcon_bind(struct device *dev, struct device *master,
 static int sun4i_tcon_bind(struct device *dev, struct device *master,
@@ -522,12 +522,13 @@ static int sun4i_tcon_probe(struct platform_device *pdev)
 	 * Defer the probe.
 	 * Defer the probe.
 	 */
 	 */
 	panel = sun4i_tcon_find_panel(node);
 	panel = sun4i_tcon_find_panel(node);
-	if (IS_ERR(panel)) {
-		/*
-		 * If we don't have a panel endpoint, just go on
-		 */
-		if (PTR_ERR(panel) != -ENODEV)
-			return -EPROBE_DEFER;
+
+	/*
+	 * If we don't have a panel endpoint, just go on
+	 */
+	if (PTR_ERR(panel) == -EPROBE_DEFER) {
+		DRM_DEBUG_DRIVER("Still waiting for our panel. Deferring...\n");
+		return -EPROBE_DEFER;
 	}
 	}
 
 
 	return component_add(&pdev->dev, &sun4i_tcon_ops);
 	return component_add(&pdev->dev, &sun4i_tcon_ops);