Re: [PATCH v2 1/9] of: property: add of_graph_get_next_port()

2024-08-24 Thread Sakari Ailus
Hi Laurent, Morimoto-san, On Sun, Aug 11, 2024 at 08:03:16PM +0300, Laurent Pinchart wrote: > Hi Morimoto-san, > > (CC'ing Sakari) > > Thank you for the patch. > > On Fri, Aug 09, 2024 at 04:22:22AM +, Kuninori Morimoto wrote: > > We have endpoint base functions > > - of_graph_get_next_

Re: [PATCH v2 1/9] of: property: add of_graph_get_next_port()

2024-08-19 Thread Kuninori Morimoto
Hi Laurent Thank you for the review > Having multiple "ports" nodes in a device node is not something I've > ever seen before. There may be use cases, but how widespread are they ? > I would prefer handling this in driver code instead of creating a helper > function if the use case is rare. In

Re: [PATCH v2 1/9] of: property: add of_graph_get_next_port()

2024-08-11 Thread Laurent Pinchart
Hi Morimoto-san, (CC'ing Sakari) Thank you for the patch. On Fri, Aug 09, 2024 at 04:22:22AM +, Kuninori Morimoto wrote: > We have endpoint base functions > - of_graph_get_next_device_endpoint() > - of_graph_get_device_endpoint_count() > - for_each_of_graph_device_endpoint(

[PATCH v2 1/9] of: property: add of_graph_get_next_port()

2024-08-08 Thread Kuninori Morimoto
We have endpoint base functions - of_graph_get_next_device_endpoint() - of_graph_get_device_endpoint_count() - for_each_of_graph_device_endpoint() Here, for_each_of_graph_device_endpoint() loop finds each endpoints ports { port@0 { (1)