Hi Laurent, On Fri, Apr 07, 2017 at 01:04:47PM +0300, Laurent Pinchart wrote: > > @@ -58,6 +60,9 @@ struct v4l2_async_subdev { > > const struct device_node *node; > > } of; > > struct { > > + struct fwnode_handle *fwn; > > Shouldn't this be const ? I thought the same, but a lot of functions that operate on fwnode_handle take a non-const argument. I attempted changing that, but it starts a cascade of unavoidable changes elsewhere. That's not very well suitable for this patchset. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx