Hi Jacopo,
Thanks for your work.
On 2018-08-24 12:24:21 +0200, Jacopo Mondi wrote:
> Add configurable capture format to propagate_parallel() function.
> The capture format is the image format set on the VIN nodes.
>
> Signed-off-by: Jacopo Mondi <jacopo@xxxxxxxxxx>
> ---
> scripts/boards.sh | 2 ++
> scripts/vin-tests.sh | 2 +-
> 2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/boards.sh b/scripts/boards.sh
> index 7eb3a27..b379af6 100644
> --- a/scripts/boards.sh
> +++ b/scripts/boards.sh
> @@ -28,6 +28,7 @@ case $info in
> # FIXME: This is a hack and not the correct mbus format
> # for V3M, but results in an image.
> parallelformat="YUYV8_1X16"
> + parallel_captureformat="RGB565"
I'm sorry I don't see the value of this change, am I missing something?
There is no functional change but I assume you use this for something
but until I figure out what I will leave this change hanging.
> ;;
> "Renesas Draak board based on r8a77995")
> gen="gen3"
> @@ -35,6 +36,7 @@ case $info in
> parallelname="adv7612 0-004c"
> # FIXME: This is a hackfor D3, but results in an image.
> parallelformat="YUYV8_1X16"
> + parallel_captureformat="RGB565"
> ;;
> "Koelsch")
> gen="gen2"
> diff --git a/scripts/vin-tests.sh b/scripts/vin-tests.sh
> index 0c5b29a..e7b7a48 100644
> --- a/scripts/vin-tests.sh
> +++ b/scripts/vin-tests.sh
> @@ -111,5 +111,5 @@ mc_propagate_parallel() {
> echo "format: $format size: $size/$vinsize field: $field/$vinfield vdev: $vdev"
>
> $mediactl -d $mdev -V "$cam [fmt:$format/$size field:$field]"
> - yavta -f RGB565 -s $vinsize --field $vinfield $vdev
> + yavta -f $parallel_captureformat -s $vinsize --field $vinfield $vdev
> }
> --
> 2.7.4
>
--
Regards,
Niklas Söderlund
