[v5,0/5] libcamera: start frame events changes
mbox series

Message ID 20250225164116.414301-1-stanislaw.gruszka@linux.intel.com
Headers show
Series
  • libcamera: start frame events changes
Related show

Message

Stanislaw Gruszka Feb. 25, 2025, 4:41 p.m. UTC 
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