Message ID | 20210317192831.359014-6-nfraprado@collabora.com |
---|---|
State | Accepted |
Headers | show |
Series |
|
Related | show |
Hey Nícolas, Thank you for the patch. Reviewed-by: Sebastian Fricke <sebastian.fricke@posteo.net> On 17.03.2021 16:28, Nícolas F. R. A. Prado wrote: >The number of frames passed to the -C flag of cam should come right >after it, without a space, otherwise the value is discarded. >The log below already showed the correct usage, but the command in the >code-block was wrong, so fix it. > >Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> >--- > Documentation/guides/pipeline-handler.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/Documentation/guides/pipeline-handler.rst b/Documentation/guides/pipeline-handler.rst >index bcce86793ccd..8ef83fea9837 100644 >--- a/Documentation/guides/pipeline-handler.rst >+++ b/Documentation/guides/pipeline-handler.rst >@@ -1433,7 +1433,7 @@ capture through the pipeline through both of the cam and qcam utilities. > .. code-block:: shell > > ninja -C build >- ./build/src/cam/cam -c vivid -C 5 >+ ./build/src/cam/cam -c vivid -C5 I think this could be even more descriptive by writing: ``` ./build/src/cam/cam --camera=vivid --capture=5 ``` But that is probably just my own preference. > > To test that the pipeline handler can detect a device, and capture input. > >-- >2.30.2 > >_______________________________________________ >libcamera-devel mailing list >libcamera-devel@lists.libcamera.org >https://lists.libcamera.org/listinfo/libcamera-devel
Hi Nicolas, Thank you for the patch. On Thu, Mar 18, 2021 at 06:21:20AM +0100, Sebastian Fricke wrote: > Hey Nícolas, > > Thank you for the patch. > > Reviewed-by: Sebastian Fricke <sebastian.fricke@posteo.net> > > On 17.03.2021 16:28, Nícolas F. R. A. Prado wrote: > > The number of frames passed to the -C flag of cam should come right > > after it, without a space, otherwise the value is discarded. > > The log below already showed the correct usage, but the command in the > > code-block was wrong, so fix it. > > > > Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> > > --- > > Documentation/guides/pipeline-handler.rst | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/Documentation/guides/pipeline-handler.rst b/Documentation/guides/pipeline-handler.rst > > index bcce86793ccd..8ef83fea9837 100644 > > --- a/Documentation/guides/pipeline-handler.rst > > +++ b/Documentation/guides/pipeline-handler.rst > > @@ -1433,7 +1433,7 @@ capture through the pipeline through both of the cam and qcam utilities. > > .. code-block:: shell > > > > ninja -C build > > - ./build/src/cam/cam -c vivid -C 5 > > + ./build/src/cam/cam -c vivid -C5 > > I think this could be even more descriptive by writing: > ``` > ./build/src/cam/cam --camera=vivid --capture=5 > ``` > But that is probably just my own preference. That's nice too. I don't have a strong preference. I'll push patch 1/5 to 4/5 already, and let you (Nicolas) decide what option you prefer for this. Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > > To test that the pipeline handler can detect a device, and capture input. > >
Em 2021-03-18 21:02, Laurent Pinchart escreveu: > Hi Nicolas, > > Thank you for the patch. > > On Thu, Mar 18, 2021 at 06:21:20AM +0100, Sebastian Fricke wrote: > > Hey Nícolas, > > > > Thank you for the patch. > > > > Reviewed-by: Sebastian Fricke <sebastian.fricke@posteo.net> > > > > On 17.03.2021 16:28, Nícolas F. R. A. Prado wrote: > > > The number of frames passed to the -C flag of cam should come right > > > after it, without a space, otherwise the value is discarded. > > > The log below already showed the correct usage, but the command in the > > > code-block was wrong, so fix it. > > > > > > Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> > > > --- > > > Documentation/guides/pipeline-handler.rst | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/Documentation/guides/pipeline-handler.rst b/Documentation/guides/pipeline-handler.rst > > > index bcce86793ccd..8ef83fea9837 100644 > > > --- a/Documentation/guides/pipeline-handler.rst > > > +++ b/Documentation/guides/pipeline-handler.rst > > > @@ -1433,7 +1433,7 @@ capture through the pipeline through both of the cam and qcam utilities. > > > .. code-block:: shell > > > > > > ninja -C build > > > - ./build/src/cam/cam -c vivid -C 5 > > > + ./build/src/cam/cam -c vivid -C5 > > > > I think this could be even more descriptive by writing: > > ``` > > ./build/src/cam/cam --camera=vivid --capture=5 > > ``` > > But that is probably just my own preference. > > That's nice too. I don't have a strong preference. I'll push patch 1/5 > to 4/5 already, and let you (Nicolas) decide what option you prefer for > this. I agree with Sebastian that using `--capture=` is more descriptive, but that usage is obvious from the `--help`, while the lack of space after `-C` is not. So I'd keep the `-C` usage (moreover, people are lazy and will use the shorter forms, so it should be clear how to do so :)). Thanks, Nícolas > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > > > > > To test that the pipeline handler can detect a device, and capture input. > > > > > -- > Regards, > > Laurent Pinchart
diff --git a/Documentation/guides/pipeline-handler.rst b/Documentation/guides/pipeline-handler.rst index bcce86793ccd..8ef83fea9837 100644 --- a/Documentation/guides/pipeline-handler.rst +++ b/Documentation/guides/pipeline-handler.rst @@ -1433,7 +1433,7 @@ capture through the pipeline through both of the cam and qcam utilities. .. code-block:: shell ninja -C build - ./build/src/cam/cam -c vivid -C 5 + ./build/src/cam/cam -c vivid -C5 To test that the pipeline handler can detect a device, and capture input.
The number of frames passed to the -C flag of cam should come right after it, without a space, otherwise the value is discarded. The log below already showed the correct usage, but the command in the code-block was wrong, so fix it. Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> --- Documentation/guides/pipeline-handler.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)