Hi Laurent, On Tue, May 30, 2023 at 05:14:43AM +0300, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Thu, May 25, 2023 at 12:15:45PM +0300, Sakari Ailus wrote: > > Also v4l2_async_nf_add_fwnode() may be used to add an async sub-device > > descriptor to a notifier. Document this. > > > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > --- > > Documentation/driver-api/media/v4l2-subdev.rst | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/driver-api/media/v4l2-subdev.rst b/Documentation/driver-api/media/v4l2-subdev.rst > > index 602dadaa81d86..ce8e9d0a332bc 100644 > > --- a/Documentation/driver-api/media/v4l2-subdev.rst > > +++ b/Documentation/driver-api/media/v4l2-subdev.rst > > @@ -204,9 +204,9 @@ that the bridge device needs for its operation. Several functions are available > > to add subdevice descriptors to a notifier, depending on the type of device and > > the needs of the driver. > > > > -:c:func:`v4l2_async_nf_add_fwnode_remote` and > > -:c:func:`v4l2_async_nf_add_i2c` are for bridge and ISP drivers for > > -registering their async sub-devices with the notifier. > > +:c:func:`v4l2_async_nf_add_fwnode`, :c:func:`v4l2_async_nf_add_fwnode_remote` > > +:c:and func:`v4l2_async_nf_add_i2c` are for registering their async sub-devices > > The 'and' isn't in the right place. With this fixed, > > Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> Thanks. Same below. Joe's word wrap sometimes does little tricks... > > +:c:with the notifier. > > > > :c:func:`v4l2_async_register_subdev_sensor` is a helper function for > > sensor drivers registering their own async sub-device, but it also registers a -- Regards, Sakari Ailus
