Hi Kieran,
Thank you for the patch.
On Wednesday 08 Feb 2017 14:04:00 Kieran Bingham wrote:
> From: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>
>
> Test both the input cropping size and position
>
> Signed-off-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>
> ---
> tests/vsp-unit-test-0021.sh | 39 ++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 39 insertions(+)
> create mode 100755 tests/vsp-unit-test-0021.sh
>
> diff --git a/tests/vsp-unit-test-0021.sh b/tests/vsp-unit-test-0021.sh
> new file mode 100755
> index 000000000000..d00dd0dece97
> --- /dev/null
> +++ b/tests/vsp-unit-test-0021.sh
> @@ -0,0 +1,39 @@
> +#!/bin/sh
> +
> +#
> +# Test RPF crop in RGB and YUV mode. Use a RPF -> WPF pipeline with a fixed
> ARGB32
> +# format on the input and capture output frames in all RGB formats
> supported
> +# by the WPF.
> +#
> +
> +source vsp-lib.sh
> +
> +features="rpf.0 wpf.0"
> +crops="(0,0)/512x384 (32,32)/512x384 (32,64)/512x384 (64,32)/512x384"
> +
> +
A single blank line should do.
> +test_rpf_cropping() {
> + test_start "RPF crop from $crop"
> +
> + pipe_configure rpf-wpf 0 0
> + format_configure crop-rpf-wpf 0 0 RGB24 1024x768 ARGB32 $crop
> +
> + vsp_runner rpf.0 &
> + vsp_runner wpf.0
> +
> + local result=$(compare_frames crop=${crop})
> +
> + test_complete $result
> +}
> +
> +test_main() {
> + local format
You don't use this variable.
With this fixed,
Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> + local crop
> +
> + for crop in $crops ; do
> + test_rpf_cropping $crop
> + done
> +}
> +
> +test_init $0 "$features"
> +test_run
--
Regards,
Laurent Pinchart
