[PATCH v2 0/2] Add DmaSyncer
Kieran Bingham
kieran.bingham at ideasonboard.com
Wed Nov 13 11:00:20 CET 2024
Quoting Harvey Yang (2024-11-13 05:54:31)
> Hi folks,
>
> This series of patches follows the discussion with Kieran in patch [1],
Aha, ok so I was a bit confused here. I didn't recall suggesting anytime
we should make this into an object, but what you mean is that it follows
"Roberts" implementation - not 'Mine'.
I don't mind a helper object here ... Just clarifying (to my self) that
this was not produced from something I said ;-)
Moving syncBufferForCPU to a helper in dma_buf_allocator.h does indeed
seem like a good idea to promote re-usabilty though! So lets follow the
reviews on the patches ;-)
--
Kieran
> which adds a helper function and a helper class to make synchronizing
> DMA buffers easier.
>
> The second patch updates debayer_cpu to utilize the new helper class.
> The following mtkisp7 pipeline handler will also depend on the helper
> class.
>
> I put the function in DmaBufAllocator and the helper class in the same
> file for now. Let me know if I should put them elsewhere.
>
> This passes gitlab pipeline:
> https://gitlab.freedesktop.org/chenghaoyang/libcamera/-/pipelines/1309296
>
> v2:
> - Fixed ioctl return value check when being interrupted.
>
> BR,
> Harvey
>
> [1]: [PATCH v4] libcamera: debayer_cpu: Sync DMABUFs
>
>
> Harvey Yang (2):
> DmaBufAllocator: Add Dma Buffer synchronization function & helper
> class
> debayer_cpu: Replace syncing DMABUFs with DmaSyncer
>
> .../libcamera/internal/dma_buf_allocator.h | 33 +++++++
> src/libcamera/dma_buf_allocator.cpp | 90 +++++++++++++++++++
> src/libcamera/software_isp/debayer_cpu.cpp | 29 ++----
> 3 files changed, 131 insertions(+), 21 deletions(-)
>
> --
> 2.47.0.277.g8800431eea-goog
>
More information about the libcamera-devel
mailing list