[{"id":29750,"web_url":"https://patchwork.libcamera.org/comment/29750/","msgid":"<171741596445.205609.12824896539887250104@ping.linuxembedded.co.uk>","date":"2024-06-03T11:59:24","subject":"Re: [PATCH v3 0/3] libcamera: dma_buffer_allocator: Add support for\n\tusing udmabuf to alloc dma-buffers","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Hans de Goede (2024-06-03 12:12:56)\n> Hi All,\n> \n> Here is v3 of my patch-series to add /dev/udmabuf dma-buffer allocation\n> support. This is based on: https://patchwork.libcamera.org/patch/18922/\n> \n> Changes in v3:\n> - Rebase on top of latest master which includes Milan's \"[PATCH v6 0/5]\n>   Software ISP levels cleanup\" series and Laurent's \"[PATCH 0/3] libcamera:\n>   Update to the new upstream subdev routing API\" series\n> - Drop include/linux related patches (included in Laurent's merged series)\n> - Wrap comments and error message logging at 80 chars\n> - Style fixes to some comments\n> - Drop the unnecessary checking of the created dma-buf size\n> \n> Changes in v2:\n> - New patch: Add linux/udmabuf.h to libcamera's local kernel headers\n> - New patch: Rename DmaHeap class to DmaBufAllocator\n> - libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf :\n>   - Reword the commit message\n>   - Add a new DmaBufAllocator::DmaBufAllocatorFlag::UDmaBuf type for udmabuf\n>   - Drop unnecessary size != size check\n>   - Reword log messages to be more like the DMA heap alloc path\n>   - Move UniqueFD(ret) up so as to not leak the fd on errors\n> -New patch: software_isp: Allow using dma-buffers from /dev/udmabuf\n> \n> I did not push out a branch for this to libcamera-softisp because\n> of fdo gitlab maintenance. Note CI was happy with v2 and I see no reason\n> why v3 would be different.\n\nThanks,\n\nI think we should probably wait for FDO to finish their maintenance\nanyway, but this looks ready to merge indeed.\n\nI'll run my script that will kick this through CI and then merge when\nthat's completed.\n\nThough I think I might have to manually fix the patches as they apply\nwith the double '---' trailers. I think git-am currently doesn't know to\nskip those ?\n\n--\nKieran\n\n\n> \n> Regards,\n> \n> Hans\n> \n> \n> Hans de Goede (3):\n>   libcamera: Rename DmaHeap class to DmaBufAllocator\n>   libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf\n>   libcamera: software_isp: Allow using dma-buffers from /dev/udmabuf\n> \n>  .../libcamera/internal/dma_buf_allocator.h    |  42 ++++\n>  include/libcamera/internal/dma_heaps.h        |  38 ---\n>  include/libcamera/internal/meson.build        |   2 +-\n>  .../internal/software_isp/software_isp.h      |   4 +-\n>  src/libcamera/dma_buf_allocator.cpp           | 233 ++++++++++++++++++\n>  src/libcamera/dma_heaps.cpp                   | 165 -------------\n>  src/libcamera/meson.build                     |   2 +-\n>  src/libcamera/pipeline/rpi/vc4/vc4.cpp        |   4 +-\n>  src/libcamera/software_isp/software_isp.cpp   |   6 +-\n>  9 files changed, 285 insertions(+), 211 deletions(-)\n>  create mode 100644 include/libcamera/internal/dma_buf_allocator.h\n>  delete mode 100644 include/libcamera/internal/dma_heaps.h\n>  create mode 100644 src/libcamera/dma_buf_allocator.cpp\n>  delete mode 100644 src/libcamera/dma_heaps.cpp\n> \n> -- \n> 2.45.1\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id A84E1BD87C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon,  3 Jun 2024 11:59:30 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9CD2D634CA;\n\tMon,  3 Jun 2024 13:59:29 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id B7C20634C5\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon,  3 Jun 2024 13:59:27 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id A9081908;\n\tMon,  3 Jun 2024 13:59:20 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"TExT6Y0i\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1717415960;\n\tbh=ycqBVikMOGYhkznkhIdc76JIojA8HQN+rX0QW3Euy+U=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=TExT6Y0iIPizC5cRDGEA5neemKS+GEfsDvEgv3vB1eeGp395Q9LJCzLkEv7vYl6kI\n\tq2wVnPS06L5gnZRCuVw92ur0vw1ufD/uINVkVRagVgIlsbzok5qddN4z0GZt7BpJnm\n\t5pW7H++tdCzYLZowb7RkXRdM1zWqhesuQvx9AW8o=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20240603111259.54321-1-hdegoede@redhat.com>","References":"<20240603111259.54321-1-hdegoede@redhat.com>","Subject":"Re: [PATCH v3 0/3] libcamera: dma_buffer_allocator: Add support for\n\tusing udmabuf to alloc dma-buffers","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"Maxime Ripard <mripard@redhat.com>, Milan Zamazal <mzamazal@redhat.com>, \n\tHans de Goede <hdegoede@redhat.com>","To":"Hans de Goede <hdegoede@redhat.com>, libcamera-devel@lists.libcamera.org","Date":"Mon, 03 Jun 2024 12:59:24 +0100","Message-ID":"<171741596445.205609.12824896539887250104@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":29751,"web_url":"https://patchwork.libcamera.org/comment/29751/","msgid":"<4ea42651-f8b7-496b-98ae-409bf4a01033@redhat.com>","date":"2024-06-03T12:04:23","subject":"Re: [PATCH v3 0/3] libcamera: dma_buffer_allocator: Add support for\n\tusing udmabuf to alloc dma-buffers","submitter":{"id":102,"url":"https://patchwork.libcamera.org/api/people/102/","name":"Hans de Goede","email":"hdegoede@redhat.com"},"content":"Hi,\n\nOn 6/3/24 1:59 PM, Kieran Bingham wrote:\n> Quoting Hans de Goede (2024-06-03 12:12:56)\n>> Hi All,\n>>\n>> Here is v3 of my patch-series to add /dev/udmabuf dma-buffer allocation\n>> support. This is based on: https://patchwork.libcamera.org/patch/18922/\n>>\n>> Changes in v3:\n>> - Rebase on top of latest master which includes Milan's \"[PATCH v6 0/5]\n>>   Software ISP levels cleanup\" series and Laurent's \"[PATCH 0/3] libcamera:\n>>   Update to the new upstream subdev routing API\" series\n>> - Drop include/linux related patches (included in Laurent's merged series)\n>> - Wrap comments and error message logging at 80 chars\n>> - Style fixes to some comments\n>> - Drop the unnecessary checking of the created dma-buf size\n>>\n>> Changes in v2:\n>> - New patch: Add linux/udmabuf.h to libcamera's local kernel headers\n>> - New patch: Rename DmaHeap class to DmaBufAllocator\n>> - libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf :\n>>   - Reword the commit message\n>>   - Add a new DmaBufAllocator::DmaBufAllocatorFlag::UDmaBuf type for udmabuf\n>>   - Drop unnecessary size != size check\n>>   - Reword log messages to be more like the DMA heap alloc path\n>>   - Move UniqueFD(ret) up so as to not leak the fd on errors\n>> -New patch: software_isp: Allow using dma-buffers from /dev/udmabuf\n>>\n>> I did not push out a branch for this to libcamera-softisp because\n>> of fdo gitlab maintenance. Note CI was happy with v2 and I see no reason\n>> why v3 would be different.\n> \n> Thanks,\n> \n> I think we should probably wait for FDO to finish their maintenance\n> anyway, but this looks ready to merge indeed.\n> \n> I'll run my script that will kick this through CI and then merge when\n> that's completed.\n> \n> Though I think I might have to manually fix the patches as they apply\n> with the double '---' trailers. I think git-am currently doesn't know to\n> skip those ?\n\nThat is weird, by default git am should skip anything under the first\n'---' cut-line for the commit message and it does do this for me.\n\nMaybe you have some local .gitconfig setting influencing this ?\n\nRegards,\n\nHans\n\n\n\n>> Hans de Goede (3):\n>>   libcamera: Rename DmaHeap class to DmaBufAllocator\n>>   libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf\n>>   libcamera: software_isp: Allow using dma-buffers from /dev/udmabuf\n>>\n>>  .../libcamera/internal/dma_buf_allocator.h    |  42 ++++\n>>  include/libcamera/internal/dma_heaps.h        |  38 ---\n>>  include/libcamera/internal/meson.build        |   2 +-\n>>  .../internal/software_isp/software_isp.h      |   4 +-\n>>  src/libcamera/dma_buf_allocator.cpp           | 233 ++++++++++++++++++\n>>  src/libcamera/dma_heaps.cpp                   | 165 -------------\n>>  src/libcamera/meson.build                     |   2 +-\n>>  src/libcamera/pipeline/rpi/vc4/vc4.cpp        |   4 +-\n>>  src/libcamera/software_isp/software_isp.cpp   |   6 +-\n>>  9 files changed, 285 insertions(+), 211 deletions(-)\n>>  create mode 100644 include/libcamera/internal/dma_buf_allocator.h\n>>  delete mode 100644 include/libcamera/internal/dma_heaps.h\n>>  create mode 100644 src/libcamera/dma_buf_allocator.cpp\n>>  delete mode 100644 src/libcamera/dma_heaps.cpp\n>>\n>> -- \n>> 2.45.1\n>>\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id ABF6FBDE6B\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon,  3 Jun 2024 12:04:33 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id CF834634CA;\n\tMon,  3 Jun 2024 14:04:32 +0200 (CEST)","from us-smtp-delivery-124.mimecast.com\n\t(us-smtp-delivery-124.mimecast.com [170.10.133.124])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id EB601634C5\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon,  3 Jun 2024 14:04:30 +0200 (CEST)","from mail-lf1-f72.google.com (mail-lf1-f72.google.com\n\t[209.85.167.72]) by relay.mimecast.com with ESMTP with STARTTLS\n\t(version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n\tus-mta-16-FumhsllMNH-L98Oh09-pnQ-1; Mon, 03 Jun 2024 08:04:28 -0400","by mail-lf1-f72.google.com with SMTP id\n\t2adb3069b0e04-52b95e7decfso1237068e87.1\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 03 Jun 2024 05:04:28 -0700 (PDT)","from [10.40.98.157] ([78.108.130.194])\n\tby smtp.gmail.com with ESMTPSA id\n\t4fb4d7f45d1cf-57a690678bcsm822750a12.65.2024.06.03.05.04.25\n\t(version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n\tMon, 03 Jun 2024 05:04:25 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"MXvtT0AO\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1717416269;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\tto:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\tcontent-transfer-encoding:content-transfer-encoding:\n\tin-reply-to:in-reply-to:references:references;\n\tbh=dWr2g6ghP9Z9b2mg1mGSFNgzs5w5koxCF1hUgtw6sUU=;\n\tb=MXvtT0AO6Dqg1DakwLr42PwIrOxjQND3h7wYWBSrOohRZrUUXZukmiA0xjo7wk+r3P302W\n\tVAkSrGEYT153j2JEpWbdExi4zosnY4p0ykHlLl2x2cfLzxEdVY3kIao57x5VErtsFePYW2\n\tI+E57rYapR/8v0Nli3xNSgF5xk96LUs=","X-MC-Unique":"FumhsllMNH-L98Oh09-pnQ-1","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1717416267; x=1718021067;\n\th=content-transfer-encoding:in-reply-to:from:content-language\n\t:references:cc:to:subject:user-agent:mime-version:date:message-id\n\t:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to;\n\tbh=dWr2g6ghP9Z9b2mg1mGSFNgzs5w5koxCF1hUgtw6sUU=;\n\tb=K7nJyA08tqLSpkynyJTfjDQZOdYgQctO/uI5l2Ce1L+K7ZgNGy61YZuPrsxt06+BVT\n\t8WFuk4vc72L266j7OjpFEphi5G2JTzEO9dNdG1fRsp1/oSTTpJ3A9vCDRjYeuxdGLMjb\n\t98lCV8U7EkKh4lJFWe1KtzNar4X5jxfWwNwqplvJCgvrsmlNN1tDq+DyG1Sb23roLgwm\n\tlo4HXZAG/DLOr/AMzjRtMmQyDCxx+y0ztdqqyAJCQB+XBIg4FXPvMo4XVaRd1sWI55Ww\n\tshY/UEdMlIYkaE/8OIhmgRp4Qj7u+Mc6kducSYXcUUUJBwMtPjrPC4VrULRY4cID8FYZ\n\tEoFA==","X-Forwarded-Encrypted":"i=1;\n\tAJvYcCX4RkEU9cJe4aXa10Q8mfJVNYKi/+vJTe2Ow26zgorI193rF/X6VEmf427PBIaR/Hdf0+i4c0/3RDdnvK1hlMD/W5w5TnvRPHuUmT9mPZUUX4lmNw==","X-Gm-Message-State":"AOJu0YxylzlO9PMBFL+EtO3dD8UQBUiK/RFJ/ybBwO7Jx7JqnT9eafwz\n\txOReiU27TGvuJWKKeBSi4lQ6VDzm4SFozgjQxra+43BmcyKsT7UdUgbqBrc6v+fIyqwc2Akv4n5\n\tfDJes1VnxpVczKYv2F8GY/RGglpkdSfNrpoVWTp2ox7Bv+MCE0QoM55CWZpLpe3c+r1jFP1M=","X-Received":["by 2002:ac2:52a6:0:b0:52b:796e:66a5 with SMTP id\n\t2adb3069b0e04-52b896f4bc5mr5843517e87.66.1717416266862; \n\tMon, 03 Jun 2024 05:04:26 -0700 (PDT)","by 2002:ac2:52a6:0:b0:52b:796e:66a5 with SMTP id\n\t2adb3069b0e04-52b896f4bc5mr5843492e87.66.1717416266330; \n\tMon, 03 Jun 2024 05:04:26 -0700 (PDT)"],"X-Google-Smtp-Source":"AGHT+IHUstsE7l03e1NPR1jTiil3jOTGe4bzqL9Ys47reygxZ7ljAnBkOASf1VpTOyZn1B/kgmjfHw==","Message-ID":"<4ea42651-f8b7-496b-98ae-409bf4a01033@redhat.com>","Date":"Mon, 3 Jun 2024 14:04:23 +0200","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH v3 0/3] libcamera: dma_buffer_allocator: Add support for\n\tusing udmabuf to alloc dma-buffers","To":"Kieran Bingham <kieran.bingham@ideasonboard.com>,\n\tlibcamera-devel@lists.libcamera.org","Cc":"Maxime Ripard <mripard@redhat.com>, Milan Zamazal <mzamazal@redhat.com>","References":"<20240603111259.54321-1-hdegoede@redhat.com>\n\t<171741596445.205609.12824896539887250104@ping.linuxembedded.co.uk>","From":"Hans de Goede <hdegoede@redhat.com>","In-Reply-To":"<171741596445.205609.12824896539887250104@ping.linuxembedded.co.uk>","X-Mimecast-Spam-Score":"0","X-Mimecast-Originator":"redhat.com","Content-Language":"en-US","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"7bit","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":29752,"web_url":"https://patchwork.libcamera.org/comment/29752/","msgid":"<171741647258.2248009.3338510480545792819@ping.linuxembedded.co.uk>","date":"2024-06-03T12:07:52","subject":"Re: [PATCH v3 0/3] libcamera: dma_buffer_allocator: Add support for\n\tusing udmabuf to alloc dma-buffers","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Hans de Goede (2024-06-03 13:04:23)\n> Hi,\n> \n> On 6/3/24 1:59 PM, Kieran Bingham wrote:\n> > Quoting Hans de Goede (2024-06-03 12:12:56)\n> >> Hi All,\n> >>\n> >> Here is v3 of my patch-series to add /dev/udmabuf dma-buffer allocation\n> >> support. This is based on: https://patchwork.libcamera.org/patch/18922/\n> >>\n> >> Changes in v3:\n> >> - Rebase on top of latest master which includes Milan's \"[PATCH v6 0/5]\n> >>   Software ISP levels cleanup\" series and Laurent's \"[PATCH 0/3] libcamera:\n> >>   Update to the new upstream subdev routing API\" series\n> >> - Drop include/linux related patches (included in Laurent's merged series)\n> >> - Wrap comments and error message logging at 80 chars\n> >> - Style fixes to some comments\n> >> - Drop the unnecessary checking of the created dma-buf size\n> >>\n> >> Changes in v2:\n> >> - New patch: Add linux/udmabuf.h to libcamera's local kernel headers\n> >> - New patch: Rename DmaHeap class to DmaBufAllocator\n> >> - libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf :\n> >>   - Reword the commit message\n> >>   - Add a new DmaBufAllocator::DmaBufAllocatorFlag::UDmaBuf type for udmabuf\n> >>   - Drop unnecessary size != size check\n> >>   - Reword log messages to be more like the DMA heap alloc path\n> >>   - Move UniqueFD(ret) up so as to not leak the fd on errors\n> >> -New patch: software_isp: Allow using dma-buffers from /dev/udmabuf\n> >>\n> >> I did not push out a branch for this to libcamera-softisp because\n> >> of fdo gitlab maintenance. Note CI was happy with v2 and I see no reason\n> >> why v3 would be different.\n> > \n> > Thanks,\n> > \n> > I think we should probably wait for FDO to finish their maintenance\n> > anyway, but this looks ready to merge indeed.\n> > \n> > I'll run my script that will kick this through CI and then merge when\n> > that's completed.\n> > \n> > Though I think I might have to manually fix the patches as they apply\n> > with the double '---' trailers. I think git-am currently doesn't know to\n> > skip those ?\n> \n> That is weird, by default git am should skip anything under the first\n> '---' cut-line for the commit message and it does do this for me.\n> \n> Maybe you have some local .gitconfig setting influencing this ?\n\nI've just applied with 'git-pw series apply 4352 -s' and it did the\nright thing!\n\nSorry for the noise - I was probably mis-remembering something that I've\nhit in the past.\n\n--\nKieran\n\n\n> \n> Regards,\n> \n> Hans\n> \n> \n> \n> >> Hans de Goede (3):\n> >>   libcamera: Rename DmaHeap class to DmaBufAllocator\n> >>   libcamera: DmaBufAllocator: Support allocating from /dev/udmabuf\n> >>   libcamera: software_isp: Allow using dma-buffers from /dev/udmabuf\n> >>\n> >>  .../libcamera/internal/dma_buf_allocator.h    |  42 ++++\n> >>  include/libcamera/internal/dma_heaps.h        |  38 ---\n> >>  include/libcamera/internal/meson.build        |   2 +-\n> >>  .../internal/software_isp/software_isp.h      |   4 +-\n> >>  src/libcamera/dma_buf_allocator.cpp           | 233 ++++++++++++++++++\n> >>  src/libcamera/dma_heaps.cpp                   | 165 -------------\n> >>  src/libcamera/meson.build                     |   2 +-\n> >>  src/libcamera/pipeline/rpi/vc4/vc4.cpp        |   4 +-\n> >>  src/libcamera/software_isp/software_isp.cpp   |   6 +-\n> >>  9 files changed, 285 insertions(+), 211 deletions(-)\n> >>  create mode 100644 include/libcamera/internal/dma_buf_allocator.h\n> >>  delete mode 100644 include/libcamera/internal/dma_heaps.h\n> >>  create mode 100644 src/libcamera/dma_buf_allocator.cpp\n> >>  delete mode 100644 src/libcamera/dma_heaps.cpp\n> >>\n> >> -- \n> >> 2.45.1\n> >>\n> > \n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id A9062BD87C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon,  3 Jun 2024 12:07:57 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B083B634CD;\n\tMon,  3 Jun 2024 14:07:56 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 53003634B2\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon,  3 Jun 2024 14:07:55 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 34DC5A38;\n\tMon,  3 Jun 2024 14:07:48 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"f7fujn54\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1717416468;\n\tbh=TtgCei3qO7F56ywByupNO5ghLhyNgXBM39m3zH+BxFE=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=f7fujn54lDajTS0t4zVzyIyQVQgW7fw9gEHKFzZqFq9S1/H/sWUuO3ADUcPlLRGP0\n\tQpDTg/HzNd/VKL53jooM5F2wiycpbyOzdaL9OQV1vRTZAf7Yosd3sc2t5RNy56MY0k\n\tMJRjTjzdcqfkzKmQo/rcaSOeEHpzrBqS9Kdtf6v4=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<4ea42651-f8b7-496b-98ae-409bf4a01033@redhat.com>","References":"<20240603111259.54321-1-hdegoede@redhat.com>\n\t<171741596445.205609.12824896539887250104@ping.linuxembedded.co.uk>\n\t<4ea42651-f8b7-496b-98ae-409bf4a01033@redhat.com>","Subject":"Re: [PATCH v3 0/3] libcamera: dma_buffer_allocator: Add support for\n\tusing udmabuf to alloc dma-buffers","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"Maxime Ripard <mripard@redhat.com>, Milan Zamazal <mzamazal@redhat.com>","To":"Hans de Goede <hdegoede@redhat.com>, libcamera-devel@lists.libcamera.org","Date":"Mon, 03 Jun 2024 13:07:52 +0100","Message-ID":"<171741647258.2248009.3338510480545792819@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]