This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: platform: xilinx: use for_each_endpoint_of_node()
Author:  Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Date:    Tue Jul 30 00:35:10 2024 +0000

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...@renesas.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/platform/xilinx/xilinx-vipp.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

---

diff --git a/drivers/media/platform/xilinx/xilinx-vipp.c 
b/drivers/media/platform/xilinx/xilinx-vipp.c
index 996684a73038..bfe48cc0ab52 100644
--- a/drivers/media/platform/xilinx/xilinx-vipp.c
+++ b/drivers/media/platform/xilinx/xilinx-vipp.c
@@ -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);

Reply via email to