[libcamera-devel] [PATCH v2 0/3] cam: options: Add support for repeatable options

[Jacopo Mondi]

Hi Niklas,

On Wed, Mar 27, 2019 at 01:21:04AM +0100, Niklas Söderlund wrote:
> Hi,
>
> This series extends the option parser for the cam tool to support
> repeatable options. This will be needed to extend the tool to support
> multiple streams.
>
> This was posted earlier as a RFC that included a fourth patch to make
> use of the new interface in the cam utility. That patch was incomplete
> and did not fully add multiple stream support to the cam tool and is
> therefor dropped in this non-RFC series which is intended to be merged
>
> All patches have review tags but as there was some debate in v1 I post
> v2 and let it stew for a day or two before pushing it in case someone
> have additional comment.
>

Don't hold this series for my comments on v1 as it seems they are not
addressable.

Thanks
  j

> Niklas Söderlund (3):
>   cam: options: Create separate enum for OptionValue types
>   cam: options: Add an array data type to OptionValue
>   cam: options: Add support for repeatable options
>
>  src/cam/options.cpp | 59 ++++++++++++++++++++++++++++++++++-----------
>  src/cam/options.h   | 23 +++++++++++++++---
>  2 files changed, 64 insertions(+), 18 deletions(-)
>
> --
> 2.21.0
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.libcamera.org/pipermail/libcamera-devel/attachments/20190327/be693ea9/attachment.sig>