On Wed, Mar 31, 2021 at 11:12:38AM +0300, Dan Carpenter wrote: > 644 of_node_put(endpoint); > 645 > 646 lt->host_node = of_graph_get_remote_node(dev->of_node, 0, -1); > 647 if (!lt->host_node) { > 648 dev_err(lt->dev, "%s: Failed to get remote port\n", __func__); > 649 ret = -ENODEV; > 650 goto end; > 651 } > 652 > 653 port_node = of_graph_get_remote_node(dev->of_node, 1, -1); > 654 if (!port_node) { > 655 dev_err(lt->dev, "%s: Failed to get connector port\n", __func__); > 656 ret = -ENODEV; > 657 goto err_free_host_node; > 658 } > 659 > 660 lt->hdmi_port = of_drm_find_bridge(port_node); > 661 if (IS_ERR(lt->hdmi_port)) { This isn't an error pointer either. of_drm_find_bridge() returns NULL. regards, dan carpenter _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel