Hi Sakari, On Tuesday, 19 September 2017 15:11:32 EEST Sakari Ailus wrote: > On Tue, Sep 19, 2017 at 02:35:01PM +0300, Laurent Pinchart wrote: > > On Friday, 15 September 2017 17:17:04 EEST Sakari Ailus wrote: > >> Add two functions for parsing devices graph endpoints: > >> v4l2_async_notifier_parse_fwnode_endpoints and > >> v4l2_async_notifier_parse_fwnode_endpoints_by_port. The former iterates > >> over all endpoints whereas the latter only iterates over the endpoints > >> in a given port. > >> > >> The former is mostly useful for existing drivers that currently > >> implement the iteration over all the endpoints themselves whereas the > >> latter is especially intended for devices with both sinks and sources: > >> async sub-devices for external devices connected to the device's sources > >> will have already been set up, or they are part of the master device. > > > > Did you mean s/or they/as they/ ? > > No. There are two options here: either the sub-devices a sub-device is > connected to (through a graph endpoint) is instantiated through the async > framework *or* through the master device driver. But not by both of them at > the same time. The message is then contradicting itself: "async sub-devices for external devices connected to the device's sources will have already been set up, or they are part of the master device." They refers to "async sub-devices". If they're part of the master device, they're not async sub-devices. -- Regards, Laurent Pinchart