Hi Sakari, On Friday 07 Apr 2017 13:45:09 Sakari Ailus wrote: > 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. fwnode is young, we should try to fix it instead of propagating issues :-) -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html