Cover Letter Detail
Show a cover letter.
GET /api/covers/24629/?format=api
{ "id": 24629, "url": "https://patchwork.libcamera.org/api/covers/24629/?format=api", "web_url": "https://patchwork.libcamera.org/cover/24629/", "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": "<20251014-v4l2-params-v6-0-caf5fa69eb29@ideasonboard.com>", "date": "2025-10-14T10:55:35", "name": "[v6,0/5] ipa: libipa: Introduce V4L2Params", "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/24629/mbox/", "series": [ { "id": 5499, "url": "https://patchwork.libcamera.org/api/series/5499/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=5499", "date": "2025-10-14T10:55:35", "name": "ipa: libipa: Introduce V4L2Params", "version": 6, "mbox": "https://patchwork.libcamera.org/series/5499/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/24629/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 48B56BE080\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 14 Oct 2025 10:55:53 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 59035605DF;\n\tTue, 14 Oct 2025 12:55:52 +0200 (CEST)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id B8D0060443\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 14 Oct 2025 12:55:50 +0200 (CEST)", "from [192.168.1.182] (93-46-82-201.ip106.fastwebnet.it\n\t[93.46.82.201])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 05612741;\n\tTue, 14 Oct 2025 12:54:11 +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=\"uMXkFc6e\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1760439252;\n\tbh=Wo+AtGw+ImxV9/zVn9C8NiN4t07hDCNOfuEANt6VCDw=;\n\th=From:Subject:Date:To:Cc:From;\n\tb=uMXkFc6e5qA9qKmUcxqdiL9kXa7JMe8yqe7LAtoCeACbovQkwpH29A3vYBQgGvSXh\n\tau9wVkMGQ6QaSZkdsFp6X+Z0y5d/UkZ5grIF2QD/y3Gs1DCT1lffNotSXhYMYXuc6l\n\tM4qk8KxZZ63GyOv4A+i2eLmlZVe7+JjfvrfvPFEc=", "From": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Subject": "[PATCH v6 0/5] ipa: libipa: Introduce V4L2Params", "Date": "Tue, 14 Oct 2025 12:55:35 +0200", "Message-Id": "<20251014-v4l2-params-v6-0-caf5fa69eb29@ideasonboard.com>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "X-B4-Tracking": "v=1; b=H4sIACcs7mgC/3XMQW7DIBCF4atErEvFDIOBrHqPqgtsxg1SYyKor\n\tFaR716cRRtX8fJ/0vuuonJJXMXxcBWF51RTnlp0TwcxnML0zjLF1gIVGuXQy5k+UF5CCecqI2E\n\tIxnu2Loj2uBQe09dNe31rfUr1M5fvGz7Duj52ZpBKalLW6rHv/Di+pMih5qnPocTnIZ/Fys34R\n\t3jotgQ2wnZEFjh2pM0Ooe8I/EfoRjADkHU4QM87BP0SoBRsCWqEG7Enaz0qs0eYe8JuCbMSsSc\n\tDwYH27gGxLMsPGt2YMrkBAAA=", "X-Change-ID": "20250829-v4l2-params-d42aa599e78a", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>, \n\tLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, \n\tKieran Bingham <kieran.bingham@ideasonboard.com>, \n\tAntoine Bouyer <antoine.bouyer@nxp.com>, \n\tLaurent Pinchart <laurent.pinchart@ideasonboard.com>, \n\tDaniel Scally <dan.scally@ideasonboard.com>", "X-Mailer": "b4 0.14.2", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=4491;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=Wo+AtGw+ImxV9/zVn9C8NiN4t07hDCNOfuEANt6VCDw=;\n\tb=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBo7iw0jKsNu/Ra4kklwD0/T6Z4lD0c7lhpqW9n7\n\t8j1QV0y6SiJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCaO4sNAAKCRByNAaPFqFW\n\tPI2+EACbFFrHM2lBuseZa1EqvhImja9GBwSP6XhR58tTrS3pYNgbh8SpmIiYA4ebTLuI1EY8YaC\n\tC6ERzodp11jzQTjRx2qEJzadrXjNZB+s3qREp3+Qm7E0TaSeQlVlBkjLM1aetN0tbmKbymaDknd\n\tBPOTiFxZiZHBIbNhWRUAvm8rGhFtgqAH6ARpWM06nM0cp21zsrN8pBX8wNdP93iUV7RC3X0c7Ld\n\tL8w27vMVyJoSfGCdDvly5w5TjsliOAoWgJawSsmJ/vxuVfcJTjG/r+rGw1VXujOiXMBO3mdkRa9\n\tFaMvryjZ1dDoWXFzQdXD0T1ea4h96GiCHaKPx3FRCW6z5gYoXlvrWPr8Yl1mP7k5T74aQJ5ttJf\n\t5qu17h9mYAIau6x5fgSQ06iYrjGc5WZEoxTap72BpUn18MCot09bScPCXlegBIjp0k9+zZxGYRa\n\thxevRUzTIGK3Tapm6oklJbi/W3StYybLfY44QRfwGNtKqTwBkTwpgPQyhQMyX6YcYJG+5DLEjLD\n\tPezoovf2GdyELrSWnpMq90IqAgUgjQgS+djBQ2Eg/ExKWW64J44hXE6yGgw6Q39XiFX+HyQFUGp\n\toeGEp/6uEr5jCPjpLF0TpTO1AojaOcCz9LqAQqJBJE9P0VQrdL3H7znSoWkeLXiEFBC4QjMU21/\n\tfScHr7zBr5ZFmTA==", "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": "With the introduction of v4l2-isp.h in the Linux kernel currently in\nprogress:\nhttps://lore.kernel.org/all/20251007-extensible-parameters-validation-v6-0-5f719d9f39e5@ideasonboard.com/\n\nthe MaliC55 ISP driver at version in v11 has been ported to use a\nparameters format compatible with the one defined by v4l2 extensible\nparameters:\nhttps://lore.kernel.org/all/20250714-c55-v11-0-bc20e460e42a@ideasonboard.com/\n\nThis series\n1) Imports v4l2-isp in libcamera\n2) Update the Mali C55 header and IPA to compile with v11 of the patch\n series that introduces MaliC55 support in Linux\n3) Introduces V4L2Params by generalizing RkISP1Params\n4) Ports the Mali C55 IPA to use the newly introduced V4L2Params\n\nTested on rkisp1 and rzv2h\n\nSigned-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>\n---\nChanges in v6:\n- Apply Dan's comments on documentation\n- Assert with Fatal if we run out of space\n- Link to v5: https://lore.kernel.org/r/20251007-v4l2-params-v5-0-8db451a81398@ideasonboard.com\n\nChanges in v5:\n- Update to v6 of v4l2 isp\n- Import linux/stddef.h instead of re-defining __counted_by()\n- Properly memset the whole mali-c55 buffer in the transitory patch\n before MaliC55Params introduction\n\nChanges in v4:\n- Mark RkISP1ParamBlock final\n- Fix code examples as suggested by Barnabas\n\nChanges in v3:\n- Fix Mali-C55 and re-work RkISP1\n The Mali and RkISP1 IPA have a different requirements:\n\n\tstruct mali_c55_params_awb_gains {\n\t\tstruct v4l2_params_block_header header;\n\t\t__u16 gain00;\n\t\t__u16 gain01;\n\t\t__u16 gain10;\n\t\t__u16 gain11;\n\t};\n\n\tstruct rkisp1_ext_params_awb_gain_config {\n\t\tstruct rkisp1_ext_params_block_header header;\n\t\tstruct rkisp1_cif_isp_awb_gain_config config;\n\t} __attribute__((aligned(8)));\n\n The RkISP1 IPA needs to support legacy and extensible parameter\n formats, and wants to operate on 'struct rkisp1_cif_isp_' basically\n discarding the header.\n\n The Mali IPA (and all other IPA compatible with v4l2-isp) operates on\n the whole structure and do not need to discard the header.\n\n Rework V4L2ParamsBlock to return a Span<> matching the whole structure\n (header included) to support Mali correctly.\n\n This means the RkISP1ParamsBlock derived class has to override all the\n dereference operators to discard the header and allow the IPA to work\n with both the legacy and extensible formats.\n\n Tested with Mali C55, RkISP1 in extensible and legacy modes.\n\n- Link to v2: https://lore.kernel.org/r/20250916-v4l2-params-v2-0-764471ed6435@ideasonboard.com\n\nChanges in v2:\n- Address Barnabas' reviwe that allows to remove the block() overload in\n MaliC55\n- Link to v1: https://lore.kernel.org/r/20250829-v4l2-params-v1-0-340773fb69ff@ideasonboard.com\n\n---\nJacopo Mondi (5):\n include: linux: Add stddef.h\n include: linux: Add v4l2-isp.h\n ipa: Update Mali C55 header file and adjust IPA\n ipa: libipa: Introduce V4L2Params\n ipa: mali-c55: Introduce MaliC55Params\n\n include/linux/mali-c55-config.h | 161 +++-----------------\n include/linux/media/v4l2-isp.h | 102 +++++++++++++\n include/linux/stddef.h | 75 ++++++++++\n src/ipa/libipa/meson.build | 2 +\n src/ipa/libipa/v4l2_params.cpp | 254 ++++++++++++++++++++++++++++++++\n src/ipa/libipa/v4l2_params.h | 152 +++++++++++++++++++\n src/ipa/mali-c55/algorithms/agc.cpp | 87 +++++------\n src/ipa/mali-c55/algorithms/agc.h | 14 +-\n src/ipa/mali-c55/algorithms/algorithm.h | 2 +-\n src/ipa/mali-c55/algorithms/awb.cpp | 64 ++++----\n src/ipa/mali-c55/algorithms/awb.h | 10 +-\n src/ipa/mali-c55/algorithms/blc.cpp | 20 +--\n src/ipa/mali-c55/algorithms/blc.h | 3 +-\n src/ipa/mali-c55/algorithms/lsc.cpp | 58 +++-----\n src/ipa/mali-c55/algorithms/lsc.h | 8 +-\n src/ipa/mali-c55/mali-c55.cpp | 19 +--\n src/ipa/mali-c55/module.h | 3 +-\n src/ipa/mali-c55/params.h | 77 ++++++++++\n src/ipa/rkisp1/params.cpp | 93 +-----------\n src/ipa/rkisp1/params.h | 175 ++++++++++++----------\n src/ipa/rkisp1/rkisp1.cpp | 2 +-\n utils/update-kernel-headers.sh | 2 +\n 22 files changed, 906 insertions(+), 477 deletions(-)\n---\nbase-commit: 05bfebed2657cc1d032c9796efd9041bfbdc881c\nchange-id: 20250829-v4l2-params-d42aa599e78a\n\nBest regards," }