Hi Sakari, Thank you for the patch. On Mon, Sep 05, 2022 at 03:37:12PM +0300, Sakari Ailus wrote: > struct v4l2_fwnode_endpoint was zeroed previously apart from the endpoint > information itself when the endpoint properties were parsed. Now this > hasn't been the case for a few years so remove the comment. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > --- > include/media/v4l2-fwnode.h | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/include/media/v4l2-fwnode.h b/include/media/v4l2-fwnode.h > index 15e4ab6722232..394d798f3dfa4 100644 > --- a/include/media/v4l2-fwnode.h > +++ b/include/media/v4l2-fwnode.h > @@ -45,10 +45,6 @@ struct v4l2_async_subdev; > */ > struct v4l2_fwnode_endpoint { > struct fwnode_endpoint base; > - /* > - * Fields below this line will be zeroed by > - * v4l2_fwnode_endpoint_parse() > - */ > enum v4l2_mbus_type bus_type; > struct { > struct v4l2_mbus_config_parallel parallel; -- Regards, Laurent Pinchart