[PATCH v2 0/5] libcamera: dma_buffer_allocator: Add support for using udmabuf to alloc dma-buffers

[Hans de Goede]

Hi,

On 5/31/24 5:02 PM, Bryan O'Donoghue wrote:
> On 30/05/2024 18:15, Hans de Goede wrote:
>> Hi All,
>>
>> Here is v2 of my patch-series to add /dev/udmabuf dma-buffer allocation
>> support. This is based on:
>> https://patchwork.libcamera.org/patch/18922/
>>
>> Changes in v2:
>> - New patch: Add linux/udmabuf.h to libcamera's local kernel headers
>> - New patch: Rename DmaHeap class to DmaBufAllocator
>> - libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf :
>>    - Reword the commit message
>>    - Add a new DmaBufAllocator::DmaBufAllocatorFlag::UDmaBuf type for udmabuf
>>    - Drop unnecessary size != size check
>>    - Reword log messages to be more like the DMA heap alloc path
>>    - Move UniqueFD(ret) up so as to not leak the fd on errors
>> -New patch: software_isp: Allow using dma-buffers from /dev/udmabuf
>>
>> I have also pushed these patches to:
>> https://gitlab.freedesktop.org/camera/libcamera-softisp/-/commits/udmabuf-v2
>>
>> for CI, CI is happy with it except for the lint task which does not like
>> the kernel headers sync, which seems to be a false positive.
>>
>> Regards,
>>
>> Hans
> 
> For the series
> 
> Tested-by: Bryan O'Donoghue <bryan.odonoghue at linaro.org> # Lenovo-x13s

Thank you for testing.

As for the small conflict with Milan's series I guess we'll just
wait and see which series gets applied to the master branch first
and then the other one can be rebased on top.

Regards,

Hans

>