[libcamera-devel,0/3] test: Fix dequeue_watchdog test failure when running the whole test suite
mbox series

Message ID 20220407083719.21631-1-laurent.pinchart@ideasonboard.com
Headers show
Series
  • test: Fix dequeue_watchdog test failure when running the whole test suite
Related show

Message

Laurent Pinchart April 7, 2022, 8:37 a.m. UTC 
Hello,

This small series fails a failure of the dequeue_watchdog test due to
random configuration of the vimc pipeline. Please see individual patches
for details.

Laurent Pinchart (3):
  test: v4l2_videodevice: dequeue_watchdog: Log message on failures
  test: v4l2_videodevice: dequeue_watchdog: Check return value of
    streamOn
  test: v4l2_videodevice: Fix format configuration in the vimc pipeline

 test/v4l2_videodevice/dequeue_watchdog.cpp      | 10 ++++++++--
 test/v4l2_videodevice/v4l2_videodevice_test.cpp |  5 +++--
 2 files changed, 11 insertions(+), 4 deletions(-)

Comments

Kieran Bingham April 7, 2022, 1:15 p.m. UTC | #1 
Quoting Laurent Pinchart via libcamera-devel (2022-04-07 09:37:16)
> Hello,
> 
> This small series fails a failure of the dequeue_watchdog test due to
> random configuration of the vimc pipeline. Please see individual patches
> for details.
> 
> Laurent Pinchart (3):
>   test: v4l2_videodevice: dequeue_watchdog: Log message on failures
>   test: v4l2_videodevice: dequeue_watchdog: Check return value of
>     streamOn
>   test: v4l2_videodevice: Fix format configuration in the vimc pipeline
> 

I can see the current failure, and I can see that it is fixed when
applying this series so:

Tested-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

And the patches look fine so lets merge it:

Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>


>  test/v4l2_videodevice/dequeue_watchdog.cpp      | 10 ++++++++--
>  test/v4l2_videodevice/v4l2_videodevice_test.cpp |  5 +++--
>  2 files changed, 11 insertions(+), 4 deletions(-)
> 
> -- 
> Regards,
> 
> Laurent Pinchart
>