Cover Letter Detail
Show a cover letter.
GET /api/covers/24554/?format=api
{ "id": 24554, "url": "https://patchwork.libcamera.org/api/covers/24554/?format=api", "web_url": "https://patchwork.libcamera.org/cover/24554/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20251003-ipa-match-by-name-v1-0-07b796729412@ideasonboard.com>", "date": "2025-10-03T09:27:28", "name": "[0/2] ipa: Allow IPA creation by name", "submitter": { "id": 143, "url": "https://patchwork.libcamera.org/api/people/143/?format=api", "name": "Jacopo Mondi", "email": "jacopo.mondi@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/24554/mbox/", "series": [ { "id": 5476, "url": "https://patchwork.libcamera.org/api/series/5476/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=5476", "date": "2025-10-03T09:27:28", "name": "ipa: Allow IPA creation by name", "version": 1, "mbox": "https://patchwork.libcamera.org/series/5476/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/24554/comments/", "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 54170C324C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 3 Oct 2025 09:27:43 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 0BDD56B5F0;\n\tFri, 3 Oct 2025 11:27:42 +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 D298F6B599\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 3 Oct 2025 11:27:39 +0200 (CEST)", "from [192.168.1.102] (93-61-96-190.ip145.fastwebnet.it\n\t[93.61.96.190])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id A0617C77;\n\tFri, 3 Oct 2025 11:26:08 +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=\"UNTjK8Ex\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1759483568;\n\tbh=E3taRu5No790VY1G8SZeCK0U0NlhGDkxcCr2skGsejc=;\n\th=From:Subject:Date:To:Cc:From;\n\tb=UNTjK8Exfbc2kxzibm1TELko/+bZ5B0FCwQtXnO5ecD4lgBb2Hcw0aOIUBQeDVKP+\n\tEJr8Wkfj9QgwlvzT4zJPJtYtjA/oAg8WfuSLLiM2zqHbYb0VJ4ZYgW6gjsAS5315mV\n\tEWnN+yFO24vMrpkrqNaqibt2vN1a9uYqbiCJtWbc=", "From": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Subject": "[PATCH 0/2] ipa: Allow IPA creation by name", "Date": "Fri, 03 Oct 2025 11:27:28 +0200", "Message-Id": "<20251003-ipa-match-by-name-v1-0-07b796729412@ideasonboard.com>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "X-B4-Tracking": "v=1; b=H4sIAACX32gC/x3MQQ5AQAxA0atI15qMDoKriEVRdGHIjAgRdzexf\n\tIv/HwjiVQI0yQNeTg26uYgsTWBY2M2COkYDGSoyYwh1Z1z5GBbsb3S8ClYkZW9zJis1xG73Mun\n\t1P9vufT91JnWmYwAAAA==", "X-Change-ID": "20251002-ipa-match-by-name-82e6b34a23e9", "To": "libcamera-devel@lists.libcamera.org, =?utf-8?q?Niklas_S=C3=B6derlund?=\n\t<niklas.soderlund+renesas@ragnatech.se>, Hans de Goede <hansg@kernel.org>", "Cc": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>, \n\tHans de Goede <hdegoede@redhat.com>", "X-Mailer": "b4 0.14.2", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=2138;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=E3taRu5No790VY1G8SZeCK0U0NlhGDkxcCr2skGsejc=;\n\tb=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBo35cKJcnPvdajMMh3VMqS5UxIEnoZatz3qoCzb\n\tC6nIZpT8CaJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCaN+XCgAKCRByNAaPFqFW\n\tPPaFD/9d4xhYXq0GKhSkA6j+PkUfDJzaaEh9JhHiEWhzS50cv1r4goZa7ySmSv+IXLppDc6H6jq\n\tmXHKWQOOWsOhG4+U+4rfl7iYzdIoqnq+8xrMxwuMC/nwFVStX0IZIcyFTTe3xOGcB5EcGrWaSMN\n\tLqjPQxCjQiqZHzqEdPny/W6KlmK+sSJosoIwNXzkHGtdt9jI7DW+LbDrEhGS2LEh0pc6Nz0nHQA\n\tLLNZRXWOxpnIUMDnCUOE5nA4L0WBPOHV66b3STqr4IEAtSrjoZP/rocTQhePbe1Lw1ZBdVEoEy4\n\t8FD1qnRPNTY40mRpL+zuuFo8lug/Wd/Bq/FBLLHc9TLMFI4WBBv5IbLfIxs4JpZfFwHi2AXPJuB\n\tHjJmlHSsf/Yx3JTLb55/01KldFbkpLVeqClZXDVT48BsooePDPfs6eK4fNGCIHC8iN5jx1Ff2gb\n\tgcXpAvP8vgkVqMYuiVbqHZ9XwyMYmFiwZjbKqo6n6YDSGH4XHgn8JnZOAgNGTpMAKiKLqj/WCT9\n\tK9eExrHmaJGs5Srt5PSPtSwg4txiloRPb4INSnp214rVdGj5SCFsNlZ9HjOWBY6l7By533L9d+t\n\tU4rUIN2wdrG1Juusz+ywFCohnO6nQXbC9DcRjgzvnnejL2g0snhTswDoEMvUwr5cHQ5BY+JOO07\n\tv8LLNhBCma7wKlA==", "X-Developer-Key": "i=jacopo.mondi@ideasonboard.com; a=openpgp;\n\tfpr=72392EDC88144A65C701EA9BA5826A2587AD026B", "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>" }, "content": "It is getting more and more common for different pipeline handlers to\nuse the same IPA modules whose name differs from the pipeline's one.\n\nThe first case has been the softISP, and now the in-review Renesas R-Car\nV4H pipeline handler will re-use the RkISP1 IPA.\n\nStarting from Hans' patch https://patchwork.libcamera.org/patch/23359/\n\"libcamera: ipa_manager: createIPA: Allow matching by IPA name instead\nof by pipeline\" I here took a slightly different approach and addressed\nthe review comments on Hans' v1 by creating an overload for\nIPAManager::createIPA() that allows pipelines to specify the IPA module\nname. If they don't provide a name, the pipeline handler's name is used.\n\nAs the next step, to further decouple IPAs from pipelines, remove the\npipelineName field from IPAModuleInfo that in all cases is identical to\nthe IPA name itself.\n\nNot tested on neither softISP or R-Car but I used the \"by name\" and \"by\npipe\" overloads on Mali and it worked.\n\nSigned-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>\n---\nHans de Goede (1):\n libcamera: ipa_manager: Create IPA by name\n\nJacopo Mondi (1):\n ipa: ipa_module: Remove pipelineName\n\n include/libcamera/internal/ipa_manager.h | 14 +++++++++++--\n include/libcamera/internal/ipa_module.h | 4 ++--\n include/libcamera/ipa/ipa_module_info.h | 1 -\n src/ipa/ipu3/ipu3.cpp | 1 -\n src/ipa/mali-c55/mali-c55.cpp | 1 -\n src/ipa/rkisp1/rkisp1.cpp | 1 -\n src/ipa/rpi/pisp/pisp.cpp | 1 -\n src/ipa/rpi/vc4/vc4.cpp | 1 -\n src/ipa/simple/soft_simple.cpp | 1 -\n src/ipa/vimc/vimc.cpp | 1 -\n src/libcamera/ipa_manager.cpp | 34 ++++++++++++++++++++++++++------\n src/libcamera/ipa_module.cpp | 27 +++++++++++--------------\n test/ipa/ipa_module_test.cpp | 3 ---\n 13 files changed, 54 insertions(+), 36 deletions(-)\n---\nbase-commit: f0f2aca56611bf9430eb6edc23e9bd55cf96a26d\nchange-id: 20251002-ipa-match-by-name-82e6b34a23e9\n\nBest regards," }