[libcamera-devel,0/2] libcamera: Misc buffer fixes
mbox series

Message ID 20191216124456.1049508-1-niklas.soderlund@ragnatech.se
Headers show
Series
  • libcamera: Misc buffer fixes
Related show

Message

Niklas Söderlund Dec. 16, 2019, 12:44 p.m. UTC 
Hi,

This was previously part of the rather large buffer rework series but is 
now broken out to a separate series as they have value on their own.

Both patches have already been reviewed and have tags but I wanted to 
post them one more time to the list stating my intention to merge them 
ahead of the buffer series if no complains are raised. So if you don't 
think this shall be merged ahead of the buffer series now is the time to 
speak up :-)

Niklas Söderlund (2):
  libcamera: pipelines: Align bookkeeping in queueRequest()
  libcamera: Remove buffer index from logging

 src/cam/capture.cpp                      |  3 +-
 src/libcamera/include/pipeline_handler.h |  4 ++-
 src/libcamera/pipeline/ipu3/ipu3.cpp     |  6 ++--
 src/libcamera/pipeline/rkisp1/rkisp1.cpp |  7 ++---
 src/libcamera/pipeline/uvcvideo.cpp      |  6 ++--
 src/libcamera/pipeline/vimc.cpp          |  6 ++--
 src/libcamera/pipeline_handler.cpp       | 35 +++++++++++++++++-------
 src/libcamera/v4l2_videodevice.cpp       |  3 +-
 src/qcam/main_window.cpp                 |  3 +-
 test/v4l2_videodevice/buffer_sharing.cpp |  6 ++--
 test/v4l2_videodevice/capture_async.cpp  |  2 +-
 test/v4l2_videodevice/v4l2_m2mdevice.cpp |  4 +--
 12 files changed, 45 insertions(+), 40 deletions(-)

Comments

Niklas Söderlund Dec. 17, 2019, 10:20 p.m. UTC | #1 
I have now pushed these two patches to master.

On 2019-12-16 13:44:54 +0100, Niklas Söderlund wrote:
> Hi,
> 
> This was previously part of the rather large buffer rework series but is 
> now broken out to a separate series as they have value on their own.
> 
> Both patches have already been reviewed and have tags but I wanted to 
> post them one more time to the list stating my intention to merge them 
> ahead of the buffer series if no complains are raised. So if you don't 
> think this shall be merged ahead of the buffer series now is the time to 
> speak up :-)
> 
> Niklas Söderlund (2):
>   libcamera: pipelines: Align bookkeeping in queueRequest()
>   libcamera: Remove buffer index from logging
> 
>  src/cam/capture.cpp                      |  3 +-
>  src/libcamera/include/pipeline_handler.h |  4 ++-
>  src/libcamera/pipeline/ipu3/ipu3.cpp     |  6 ++--
>  src/libcamera/pipeline/rkisp1/rkisp1.cpp |  7 ++---
>  src/libcamera/pipeline/uvcvideo.cpp      |  6 ++--
>  src/libcamera/pipeline/vimc.cpp          |  6 ++--
>  src/libcamera/pipeline_handler.cpp       | 35 +++++++++++++++++-------
>  src/libcamera/v4l2_videodevice.cpp       |  3 +-
>  src/qcam/main_window.cpp                 |  3 +-
>  test/v4l2_videodevice/buffer_sharing.cpp |  6 ++--
>  test/v4l2_videodevice/capture_async.cpp  |  2 +-
>  test/v4l2_videodevice/v4l2_m2mdevice.cpp |  4 +--
>  12 files changed, 45 insertions(+), 40 deletions(-)
> 
> -- 
> 2.24.0
>