[v3,0/2] libcamera: Use span in FrameBuffer
mbox series

Message ID 20250904134641.29597-1-laurent.pinchart@ideasonboard.com
Headers show
Series
  • libcamera: Use span in FrameBuffer
Related show

Message

Laurent Pinchart Sept. 4, 2025, 1:46 p.m. UTC 
Hello,

This patch series is the third version of the standalone "[PATCH]
libcamera: framebuffer: Replace vector with span in constructor" patch.

Most of the patches in v2 have been merged already, or have been
superseded by other merged patches. Only patch 1/2 remains as a drive-by
improvement from v2, and of course patch 2/2 that performs to actual
conversion to Span.

With FrameBuffer::planes() returning a span, multiple callers have to be
modified through libcamera. While there is no actual change to test
applications thanks for previous refactoring that made application code
API-compatible with this change, the patch series still changes the
libcamera ABI and API, requiring recompilation and possibly code
changes.

Laurent Pinchart (2):
  pipelines: Use lambda functions to factor out buffer mapping code
  libcamera: framebuffer: Replace vector with span in constructor

 include/libcamera/framebuffer.h               |  4 +--
 include/libcamera/internal/framebuffer.h      |  3 +-
 .../mm/cros_frame_buffer_allocator.cpp        |  2 +-
 .../mm/generic_frame_buffer_allocator.cpp     |  2 +-
 src/libcamera/framebuffer.cpp                 | 10 +++----
 src/libcamera/pipeline/ipu3/ipu3.cpp          | 20 ++++++++-----
 src/libcamera/pipeline/mali-c55/mali-c55.cpp  | 30 +++++++++++--------
 src/libcamera/pipeline/rkisp1/rkisp1.cpp      | 26 ++++++++--------
 .../pipeline/rpi/common/pipeline_base.cpp     |  6 ++--
 src/libcamera/pipeline/vimc/vimc.cpp          |  5 +++-
 src/libcamera/pipeline/virtual/virtual.cpp    |  3 +-
 src/libcamera/v4l2_videodevice.cpp            |  6 ++--
 12 files changed, 68 insertions(+), 49 deletions(-)


base-commit: baea40a8a5dddbad3cf6398aa8ce248e4547c955
--
Regards,

Laurent Pinchart