Commit 974652d7 authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Neil Armstrong

drm: bridge: thc63lvd1024: Print error message when DT parsing fails

Commit 00084f0c ("drm: bridge: thc63lvd1024: Switch to use
of_graph_get_remote_node()") simplified the thc63lvd1024 driver by
replacing hand-rolled code with a helper function. While doing so, it
created an error code path at probe time without any error message,
potentially causing probe issues that get annoying to debug. Fix it by
adding an error message.

Fixes: 00084f0c ("drm: bridge: thc63lvd1024: Switch to use of_graph_get_remote_node()")
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://lore.kernel.org/r/20240318160601.2813-1-laurent.pinchart+renesas@ideasonboard.comSigned-off-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240318160601.2813-1-laurent.pinchart+renesas@ideasonboard.com
parent 00084f0c
...@@ -125,8 +125,11 @@ static int thc63_parse_dt(struct thc63_dev *thc63) ...@@ -125,8 +125,11 @@ static int thc63_parse_dt(struct thc63_dev *thc63)
remote = of_graph_get_remote_node(thc63->dev->of_node, remote = of_graph_get_remote_node(thc63->dev->of_node,
THC63_RGB_OUT0, -1); THC63_RGB_OUT0, -1);
if (!remote) if (!remote) {
dev_err(thc63->dev, "No remote endpoint for port@%u\n",
THC63_RGB_OUT0);
return -ENODEV; return -ENODEV;
}
thc63->next = of_drm_find_bridge(remote); thc63->next = of_drm_find_bridge(remote);
of_node_put(remote); of_node_put(remote);
......
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