Skip to content

igt_chamelium: Increase timeout for CaptureVideo

Neel Desai igt-dev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest1 patch 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1lib/igt_chamelium: Increase timeout for CaptureVideolib/igt_chamelium.c+31 -21 reply

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [igt-dev] [PATCH i-g-t 0/1] igt_chamelium: Increase timeout for CaptureVideo

For some chamelium boards, we observed that the chamelium_rpc() call with method_name as CaptureVideo timed out because the display resolution read by the FPGA using the VideoDumper did not match the resolution in the RX chip. The resolution read by the FPGA from the memory was incorrect after 10 seconds and needed more time to stabilize to the correct value.

So, instead of checking the chamelium->env.fault_occured in the chamelium_rpc() funtion, we defer the check in the parent function chamelium_capture(). We iterate upto 10 times if chamelium_rpc() with method name as CaptureVideo fails thus effectively increasing the timeout to 100 seconds instead if 10 seconds. If after 10 iterations, chamelium->env.fault_occured is still set, we return an error.

This looks like an issue from Google's end in the FPGA code. This patch introduces a workaround till we get the issue resolved from Google's end.

Neel Desai (1):
lib/igt_chamelium: Increase timeout for CaptureVideo

 lib/igt_chamelium.c | 33 +++++++++++++++++++++++++++++++--
 1 file changed, 31 insertions(+), 2 deletions(-)

--
2.17.1

_______________________________________________
igt-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/igt-dev