summaryrefslogtreecommitdiff
path: root/test/py
diff options
context:
space:
mode:
authorTomi Valkeinen <tomi.valkeinen@ideasonboard.com>2022-05-30 17:27:17 +0300
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2022-06-04 02:21:05 +0300
commit7ef83e02284715db95e3f4cd61744290baff2840 (patch)
tree2954b0a38a8e7004eab84297ebc5f98145ea36fa /test/py
parentb4bb5ce629294e5a0b186c6824ffe231bd86c3c7 (diff)
py: Merge read_event() and get_ready_requests()
We always call CameraManager.read_event() and CameraManager.get_ready_requests(), so to simplify the use merge the read_event() into the get_ready_requests(). This has the side effect that get_ready_requests() will now block if there is no event ready. If we ever need to call get_ready_requests() in a polling manner we will need a new function which behaves differently. However, afaics the only sensible way to manage the event loop is to use select/poll on the eventfd and then call get_ready_requests() once, which is the use case what the current merged function supports. Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'test/py')
-rwxr-xr-xtest/py/unittests.py4
1 files changed, 0 insertions, 4 deletions
diff --git a/test/py/unittests.py b/test/py/unittests.py
index 33b35a0a..9adc4337 100755
--- a/test/py/unittests.py
+++ b/test/py/unittests.py
@@ -210,8 +210,6 @@ class SimpleCaptureMethods(CameraTesterBase):
reqs = []
while True:
- cm.read_event()
-
ready_reqs = cm.get_ready_requests()
reqs += ready_reqs
@@ -283,8 +281,6 @@ class SimpleCaptureMethods(CameraTesterBase):
while running:
events = sel.select()
for key, _ in events:
- cm.read_event()
-
ready_reqs = cm.get_ready_requests()
reqs += ready_reqs