[PATCH v5 0/5] libcamera: start frame events changes
Stanislaw Gruszka
stanislaw.gruszka at linux.intel.com
Tue Feb 25 17:41:11 CET 2025
v4 -> v5:
- split patch2 into 3 separte patches
- drop frameStartEnabled(), return error from start
- use for (const Entity &entity : entities_) insted of iterator
- reset delayed controls before enabling events
- drop R-b , T-b tags since code changes substantially since got them
v3 -> v4:
- constify frameStartEnabled()
- make supportsFrameStartEvent() more readable
- rename eventEmitter to startFrameEmitter
- update patch2 changelog to make it more clear
v2 -> v3:
- add helpers for v4l2_device
- initialize eventEmitter_ in init(), when no proper device is found
use nullptr (this fix crash seen by Kieran as eventEmitetter was
not nullptr in v2)
v1 -> v2:
- make eventEmitter_ subdevice part of SimpleCameraData
- add debug log when found event emitter device
- nullify eventEmitter_ on stop
- remove direct sensor_->setControls()
- add delayedCtrls_->reset() on start
Laurent Pinchart (1):
pipeline: simple: Create DelayedControls instance once only
Stanislaw Gruszka (4):
libcamera: v4l2_device: add frame start event helpers
pipeline: simple: Use proper device for frame start events
pipeline: simple: Connect and disconnect start frame events
pipeline: simple: Do not apply controls twice
include/libcamera/internal/v4l2_device.h | 1 +
src/libcamera/pipeline/simple/simple.cpp | 58 ++++++++++++++++++------
src/libcamera/v4l2_device.cpp | 19 ++++++++
3 files changed, 65 insertions(+), 13 deletions(-)
--
2.43.0
More information about the libcamera-devel
mailing list