Hi Morimoto-san, Thank you for the patch. On Tue, May 28, 2024 at 11:55:59PM +0000, Kuninori Morimoto wrote: > We already have of_graph_get_remote_port(), Let's use it. > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > --- > drivers/video/fbdev/omap2/omapfb/dss/dss-of.c | 15 +-------------- > 1 file changed, 1 insertion(+), 14 deletions(-) > > diff --git a/drivers/video/fbdev/omap2/omapfb/dss/dss-of.c b/drivers/video/fbdev/omap2/omapfb/dss/dss-of.c > index 14965a3fd05b7..4040e247e026e 100644 > --- a/drivers/video/fbdev/omap2/omapfb/dss/dss-of.c > +++ b/drivers/video/fbdev/omap2/omapfb/dss/dss-of.c > @@ -117,19 +117,6 @@ u32 dss_of_port_get_port_number(struct device_node *port) > return reg; > } > > -static struct device_node *omapdss_of_get_remote_port(const struct device_node *node) > -{ > - struct device_node *np; > - > - np = of_graph_get_remote_endpoint(node); > - if (!np) > - return NULL; > - > - np = of_get_next_parent(np); > - > - return np; > -} > - > struct omap_dss_device * > omapdss_of_find_source_for_first_ep(struct device_node *node) > { > @@ -141,7 +128,7 @@ omapdss_of_find_source_for_first_ep(struct device_node *node) > if (!ep) > return ERR_PTR(-EINVAL); > > - src_port = omapdss_of_get_remote_port(ep); > + src_port = of_graph_get_remote_port(ep); > if (!src_port) { > of_node_put(ep); > return ERR_PTR(-EINVAL); -- Regards, Laurent Pinchart