Hi Sakari, Thanks for your patch. On 2017-12-21 14:45:46 +0200, Sakari Ailus wrote: > Use fwnode_graph_for_each_endpoint iterator for better readability. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > --- > drivers/media/v4l2-core/v4l2-fwnode.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/media/v4l2-core/v4l2-fwnode.c b/drivers/media/v4l2-core/v4l2-fwnode.c > index 3cc8b6b69b41..c33d519281a2 100644 > --- a/drivers/media/v4l2-core/v4l2-fwnode.c > +++ b/drivers/media/v4l2-core/v4l2-fwnode.c > @@ -472,8 +472,7 @@ static int __v4l2_async_notifier_parse_fwnode_endpoints( > if (WARN_ON(asd_struct_size < sizeof(struct v4l2_async_subdev))) > return -EINVAL; > > - for (fwnode = NULL; (fwnode = fwnode_graph_get_next_endpoint( > - dev_fwnode(dev), fwnode)); ) { > + fwnode_graph_for_each_endpoint(dev_fwnode(dev), fwnode) { > struct fwnode_handle *dev_fwnode; > bool is_available; > > -- > 2.11.0 > > -- Regards, Niklas Söderlund