diff options
author | Naushir Patuck <naush@raspberrypi.com> | 2020-11-24 17:59:34 +0000 |
---|---|---|
committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2020-11-26 10:31:15 +0000 |
commit | c286b00aa1eb9331c3bab2b35905dcf2e4ccde66 (patch) | |
tree | 30c79ff8934ec51551d59e3bd96cb32f5f994e0b /src/qcam/dng_writer.h | |
parent | 770df5ca4eab84ec1d8a683d1ace04d7d00001c6 (diff) |
pipeline: raspberrypi: Fix erroneous bayer buffer requeue on buffer matching
With the recent change in the bayer/embedded buffer matching code,
a condition would make the bayer buffer be requeued back to the device,
even though it could potentially be queued for matching. This would
cause unnecessary frame drops as sync would be lost.
The fix is to ensure the bayer buffer only gets requeued if the embedded
data buffer queue is not empty, i.e. the buffer truly is orphaned.
Additionally, we do this test before deciding to flush any of the two
queues of all their buffers.
Fixes: 909882b (pipeline: raspberrypi: Rework bayer/embedded data buffer matching)
Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
Tested-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Tested-by: David Plowman <david.plowman@raspberrypi.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'src/qcam/dng_writer.h')
0 files changed, 0 insertions, 0 deletions