Re: [PATCH] V4L2: add documentation for V4L2 clock helpers and asynchronous probing

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Guennadi,

On Monday 24 June 2013 10:53:35 Guennadi Liakhovetski wrote:
> On Mon, 17 Jun 2013, Laurent Pinchart wrote:
> 
> [snip]
> 
> > > +drivers. 
> > > +
> > > +Bridge drivers in turn have to register a notifier object with an array
> > > +of subdevice descriptors, that the bridge device needs for its
> > > +operation. This
> >
> > s/descriptors,/descriptors/
> > 
> > > +is performed using the v4l2_async_notifier_register() call. To
> > > +unregister the notifier the driver has to call
> > > +v4l2_async_notifier_unregister(). The
> > > +former of the two functions takes two arguments: a pointer to struct
> > > +v4l2_device and a pointer to struct v4l2_async_notifier. The latter
> > > +contains a pointer to an array of pointers to subdevice descriptors of
> > > +type struct v4l2_async_subdev type.
> > 
> > Isn't it the other way around ?
> 
> I don't think I see anything above, that needs to be swapped. What exactly
> do you mean?

My bad. I got confused by "the latter" (which should be "the later" ;-)) and 
thought it was referring to the later of the two functions, given that the 
previous sentence starts with "the former of the two functions".

Maybe s/The format of the two functions/The first function/ and s/The 
latter/The second structure/ ?

-- 
Regards,

Laurent Pinchart