Cover Letter Detail
Show a cover letter.
GET /api/covers/24562/?format=api
{ "id": 24562, "url": "https://patchwork.libcamera.org/api/covers/24562/?format=api", "web_url": "https://patchwork.libcamera.org/cover/24562/", "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-v2-0-54f3cd828ddd@ideasonboard.com>", "date": "2025-10-03T16:16:30", "name": "[v2,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/24562/mbox/", "series": [ { "id": 5478, "url": "https://patchwork.libcamera.org/api/series/5478/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=5478", "date": "2025-10-03T16:16:30", "name": "ipa: Allow IPA creation by name", "version": 2, "mbox": "https://patchwork.libcamera.org/series/5478/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/24562/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 82DA8BF415\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 3 Oct 2025 16:16:45 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 7854C6B5F9;\n\tFri, 3 Oct 2025 18:16:44 +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 1B3CB6B5AA\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 3 Oct 2025 18:16:43 +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 31904192C;\n\tFri, 3 Oct 2025 18:15:12 +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=\"f9myx0Ev\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1759508112;\n\tbh=aGp6cXvbVICmWsFXlIvFQo51BUGz8waKqM+aPyJEu3I=;\n\th=From:Subject:Date:To:Cc:From;\n\tb=f9myx0EvUyFmrv33u5jA3DdBXLvqJKNLaVbdfSn/AP/MgQXhN32CM8hE93hADTG5/\n\tygFqKS10BfI6bfUn6TyralSnj4Nh9FV/1ySGhTnXNww+IIEDcvVWhVtmLrob74t+aK\n\tASySTtQfO87OTQFMNsSrruMV75Lslfa0MeqGcPbo=", "From": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Subject": "[PATCH v2 0/2] ipa: Allow IPA creation by name", "Date": "Fri, 03 Oct 2025 18:16:30 +0200", "Message-Id": "<20251003-ipa-match-by-name-v2-0-54f3cd828ddd@ideasonboard.com>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "X-B4-Tracking": "v=1; b=H4sIAN/232gC/22NQQ6CMBBFr0K6dkw7RRBX3sOwmMIos6AlLSESw\n\tt2tJO5cvpf89zeVOAondSs2FXmRJMFnwFOhuoH8i0H6zAo1XozWCDIRjDR3A7gVPI0MV+TK2ZL\n\tQcqPybor8lPfRfLSZB0lziOtxsZiv/dXsn9piQIOuXd1UNTalwbv0TCl4Fyj25y6Mqt33/QOYg\n\tPefuQAAAA==", "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>, Kieran Bingham\n\t<kieran.bingham@ideasonboard.com>, =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?=\n\t<barnabas.pocze@ideasonboard.com>", "X-Mailer": "b4 0.14.2", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=2361;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=aGp6cXvbVICmWsFXlIvFQo51BUGz8waKqM+aPyJEu3I=;\n\tb=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBo3/bpEzyJGU+YH1/IeeliS4493nSG5jAXFIzIa\n\t8qVSIjXNluJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCaN/26QAKCRByNAaPFqFW\n\tPAvwEACFNhOxjldjFdOf1EWBBfLZb2fA2y2UJb8QPYGtNWe1hj0nB3/uQ1uNlfBdEYnRqhqebzF\n\toR+iaiVOycfm66q8JcR5ilwoOBuDOw/58kHzVESEdTwCeRtkHJ+jtFQyoFiaWG4kiXGre9P9CME\n\tVphEoxLwSP0+wvrv+M779mPxmPQD3KNK+lNZyejQWjUNrB7milfRyQ15/Z7OyjJbqEbxE6C32HL\n\t+EiJuP3OS1Y87Hbu5cEADiz4NbuUk0JoG4Y4/n44Vgk40eTm7cyuJvoHoyAqTufLqym/0H3YCFg\n\tXKGn1G0xHfZDaf1cYqlhx+SkZjpbbr1je0oJpVLMG0B+dvMQcf63/UYGdAmRZWaOzImTb8HBAzn\n\t6z8NTnBfcJtqHDxWayN9mVYWI7od+ClYBA1sJBEDmUWnXKbt92TMeNSYzwyBXPut+F97BrRra/J\n\tJvQfEaFvyv2+whJzo8sH2HUUj/3nKmAktsIT3XSe8kyFoNJcwBsPR4Ky5/rOkNUVGlolbDTGCrZ\n\tQ3AvA9dwsnf3XJoJ2nPpQXuv9N0mJ1Cnazg6R2lKbV8NwoNE/2A8q6wK7hpWYmA7nBOPlmheq+r\n\toyc+nKKXUNfMIgbwlUSsEJAIDZP+IFUfCxSSSohKUWYM1ZIq+Icq+1ohxyR+frHbxvrIjeQLrYb\n\taMmaNk1vUDJuUzQ==", "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---\nChanges in v2:\n- Changed Hans authorship to different email address\n- Removed stray blank line\n- Collect tags\n- Link to v1: https://lore.kernel.org/r/20251003-ipa-match-by-name-v1-0-07b796729412@ideasonboard.com\n\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 | 13 ++++++++++--\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, 53 insertions(+), 36 deletions(-)\n---\nbase-commit: f0f2aca56611bf9430eb6edc23e9bd55cf96a26d\nchange-id: 20251002-ipa-match-by-name-82e6b34a23e9\n\nBest regards," }