media: platform: xilinx: use for_each_endpoint_of_node()
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Tue, 30 Jul 2024 00:35:10 +0000 (00:35 +0000)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Fri, 9 Aug 2024 05:56:39 +0000 (07:56 +0200)
We already have for_each_endpoint_of_node(), don't use
of_graph_get_next_endpoint() directly. Replace it.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/platform/xilinx/xilinx-vipp.c

index 996684a..bfe48cc 100644 (file)
@@ -199,18 +199,13 @@ static int xvip_graph_build_dma(struct xvip_composite_device *xdev)
        struct media_pad *sink_pad;
        struct xvip_graph_entity *ent;
        struct v4l2_fwnode_link link;
-       struct device_node *ep = NULL;
+       struct device_node *ep;
        struct xvip_dma *dma;
        int ret = 0;
 
        dev_dbg(xdev->dev, "creating links for DMA engines\n");
 
-       while (1) {
-               /* Get the next endpoint and parse its link. */
-               ep = of_graph_get_next_endpoint(node, ep);
-               if (ep == NULL)
-                       break;
-
+       for_each_endpoint_of_node(node, ep) {
                dev_dbg(xdev->dev, "processing endpoint %pOF\n", ep);
 
                ret = v4l2_fwnode_parse_link(of_fwnode_handle(ep), &link);