diff options
author | Hou Qi <qi.hou@nxp.com> | 2024-07-25 18:26:22 +0900 |
---|---|---|
committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2024-07-26 10:15:31 +0100 |
commit | c9152bad5ce905f5a31dbd05b40195f02c0cc2a9 (patch) | |
tree | 4e3817de68e47aca4f9b57eab4662871714841d2 /test/controls/control_value.cpp | |
parent | 9c403207632cd5fa4cb22be83e3f56ac5425b7fe (diff) |
gstreamer: Fix width and height range handling
This changes is fixing critical error message
"gst_value_set_int_range_step: assertion 'start < end' failed" observed
when building GStreamer caps from a stream configuration whose size
range holds a single size.
GStreamer range step definition requires distinct min and max values
definitions, otherwise above error message is output.
libcamera SizeRange instance may return a single size leading to
identical min and max values. Add a conditional check where the min and
max of the range are distinct during iterating the supported sizes for
each pixelformat.
To prevent appending structures that are already expressed with this
update, gst_caps_merge_structure() is used in place of
gst_caps_append_structure().
Signed-off-by: Hou Qi <qi.hou@nxp.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Acked-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'test/controls/control_value.cpp')
0 files changed, 0 insertions, 0 deletions