# Re: \[PATCH 9/9\] media: xilinx-tpg: use new of\_graph functions

- From: Tomi Valkeinen \<tomi.valkeinen@ideasonboard.com\>
- Date: 2024-08-08 10:05:43 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2024/08/8236329
- Thread: https://ratatoskr.run/linux-devicetree/2024/08/8236243/t.md

```text
On 06/08/2024 07:58, Kuninori Morimoto wrote:
> Now we can use new port related functions for port parsing. Use it.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>   drivers/media/platform/xilinx/xilinx-tpg.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/xilinx/xilinx-tpg.c b/drivers/media/platform/xilinx/xilinx-tpg.c
> index e05e528ffc6f..a25f216b2513 100644
> --- a/drivers/media/platform/xilinx/xilinx-tpg.c
> +++ b/drivers/media/platform/xilinx/xilinx-tpg.c
> @@ -13,6 +13,7 @@
>   #include <linux/gpio/consumer.h>
>   #include <linux/module.h>
>   #include <linux/of.h>
> +#include <linux/of_graph.h>
>   #include <linux/platform_device.h>
>   #include <linux/xilinx-v4l2-controls.h>
>   
> @@ -744,7 +745,7 @@ static int xtpg_parse_of(struct xtpg_device *xtpg)
>   		}
>   
>   		if (nports == 0) {
> -			endpoint = of_get_next_child(port, NULL);
> +			endpoint = of_graph_get_next_port_endpoint(port, NULL);
>   			if (endpoint)
>   				has_endpoint = true;
>   			of_node_put(endpoint);

Reviewed-by: Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com>

 Tomi

```