[libcamera-devel] [PATCH v2] libcamera: device_enumerator: extend documentation of DeviceMatch
Niklas Söderlund
niklas.soderlund at ragnatech.se
Tue Jan 22 13:58:08 CET 2019
Hi Laurent,
On 2019-01-21 01:06:43 +0200, Laurent Pinchart wrote:
> Hi Niklas,
>
> Thank you for the patch.
>
> On Sun, Jan 20, 2019 at 03:04:35PM +0100, Niklas Söderlund wrote:
> > Extend the document of the intended usage of DeviceMatch need for
>
> s/document/documentation/
>
> > information to correctly function to find a uniquely identifiable media
>
> I'm not sure what you meant by the "need for information to correctly
> function".
How about,
Extend the documentation of the intended usage of DeviceMatch. The
DeviceMatch needs enough information to be able to uniquely identify a
specific media device.
>
> > device.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> > ---
> > * Changes since v1
> > - Rephrased most of it after review comments from Laurent, Thanks!
> >
> > src/libcamera/device_enumerator.cpp | 12 ++++++++++++
> > 1 file changed, 12 insertions(+)
> >
> > diff --git a/src/libcamera/device_enumerator.cpp b/src/libcamera/device_enumerator.cpp
> > index 55c510e3b79a415b..d41e1e580b3451ab 100644
> > --- a/src/libcamera/device_enumerator.cpp
> > +++ b/src/libcamera/device_enumerator.cpp
> > @@ -52,6 +52,18 @@ namespace libcamera {
> > *
> > * The description is meant to be filled by pipeline managers and passed to a
> > * device enumerator to find matching media devices.
> > + *
> > + * A DeviceMatch is created with a specific Linux device driver in mind,
> > + * therefore the name of the driver is a required property. One or more Entity
> > + * names can be added as as match criteria.
>
> s/as as/as/
>
> > + *
> > + * Pipeline handlers are recommended to add entities to DeviceMatch as
> > + * appropriare to ensure that the media device they need can be uniquely
> > + * identified. This is useful when the corresponding kernel driver can produce
>
> s/ / /
>
> > + * different graphs, for instance as a result of different driver versions or
> > + * hardware configurations, and not all those graphs are suitable for a pipeline
> > + * handler.
> > + *
>
> s/^ \*\n//
>
> With these fixed,
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>
> > */
> >
> > /**
>
> --
> Regards,
>
> Laurent Pinchart
--
Regards,
Niklas Söderlund
More information about the libcamera-devel
mailing list