Hi Mauro, Thanks for the patch. On Wed, Sep 27, 2017 at 06:46:55PM -0300, Mauro Carvalho Chehab wrote: > Now that kernel-doc handles nested unions, better document the > bus union at struct v4l2_fwnode_endpoint. > > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> > --- > include/media/v4l2-fwnode.h | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/include/media/v4l2-fwnode.h b/include/media/v4l2-fwnode.h > index 7adec9851d9e..5f4716f967d0 100644 > --- a/include/media/v4l2-fwnode.h > +++ b/include/media/v4l2-fwnode.h > @@ -79,7 +79,18 @@ struct v4l2_fwnode_bus_mipi_csi1 { > * struct v4l2_fwnode_endpoint - the endpoint data structure > * @base: fwnode endpoint of the v4l2_fwnode > * @bus_type: bus type > - * @bus: bus configuration data structure > + * @bus: union with bus configuration data structure > + * @bus.parallel: pointer for &struct v4l2_fwnode_bus_parallel. > + * Used if the bus is parallel. > + * @bus.mipi_csi1: pointer for &struct v4l2_fwnode_bus_mipi_csi1. > + * Used if the bus is Mobile Industry Processor s/Mobile.*Interface/MIPI Alliance/ Same below. Please see: <URL:https://mipi.org/content/what-does-mipi-stand> With that, Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > + * Interface's Camera Serial Interface version 1 > + * (MIPI CSI1) or Standard Mobile Imaging Architecture's > + * Compact Camera Port 2 (SMIA CCP2). > + * @bus.mipi_csi2: pointer for &struct v4l2_fwnode_bus_mipi_csi2. > + * Used if the bus is Mobile Industry Processor > + * Interface's Camera Serial Interface version 2 > + * (MIPI CSI2). > * @link_frequencies: array of supported link frequencies > * @nr_of_link_frequencies: number of elements in link_frequenccies array > */ -- Regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html