Patch Detail
Show a patch.
GET /api/1.1/patches/27279/?format=api
{ "id": 27279, "url": "https://patchwork.libcamera.org/api/1.1/patches/27279/?format=api", "web_url": "https://patchwork.libcamera.org/patch/27279/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/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": "<20260708-libipa-algorithms-v4-14-f0e99f035294@ideasonboard.com>", "date": "2026-07-08T09:25:01", "name": "[v4,14/35] ipa: libipa: lsc: Move Interpolator<lsc::Components> to lsc_base", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "d179d9f4051e8b314356d21436d6416943aca887", "submitter": { "id": 143, "url": "https://patchwork.libcamera.org/api/1.1/people/143/?format=api", "name": "Jacopo Mondi", "email": "jacopo.mondi@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/27279/mbox/", "series": [ { "id": 6046, "url": "https://patchwork.libcamera.org/api/1.1/series/6046/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=6046", "date": "2026-07-08T09:24:47", "name": "ipa: libipa: Introduce libipa algorithms", "version": 4, "mbox": "https://patchwork.libcamera.org/series/6046/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/27279/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/27279/checks/", "tags": {}, "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 4429DC3329\n\tfor <parsemail@patchwork.libcamera.org>;\n\tWed, 8 Jul 2026 09:25:32 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id F3986660C3;\n\tWed, 8 Jul 2026 11:25:23 +0200 (CEST)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 60E036607E\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed, 8 Jul 2026 11:25:09 +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 B772F28CB;\n\tWed, 8 Jul 2026 11:24:19 +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=\"ghAdIkFq\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1783502659;\n\tbh=FI6u9HiVyZE3mGOS+nR17xOPN1xl6wtQAM1eP5szdPw=;\n\th=From:Date:Subject:References:In-Reply-To:To:Cc:From;\n\tb=ghAdIkFq1nzSSpvEJX57tCiyj04ECMad7+t4NebjPqWE0toFyRRjV1r9U3Fy2WlR/\n\tw/tRsweUV+aF/aPtdI/NKYOUWP123gISEMF6v6y+MI1W2Ru4s3VeHLiNxf/QufYI5z\n\tMnctH9q+jqtZuPSxnnhpic+gOUQmnAEYQm6xodic=", "From": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Date": "Wed, 08 Jul 2026 11:25:01 +0200", "Subject": "[PATCH v4 14/35] ipa: libipa: lsc: Move\n\tInterpolator<lsc::Components> to lsc_base", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "Message-Id": "<20260708-libipa-algorithms-v4-14-f0e99f035294@ideasonboard.com>", "References": "<20260708-libipa-algorithms-v4-0-f0e99f035294@ideasonboard.com>", "In-Reply-To": "<20260708-libipa-algorithms-v4-0-f0e99f035294@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org, \n\tStefan Klug <stefan.klug@ideasonboard.com>", "Cc": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>, \n\tKieran Bingham <kieran.bingham@ideasonboard.com>", "X-Mailer": "b4 0.14.3", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=3329;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=FI6u9HiVyZE3mGOS+nR17xOPN1xl6wtQAM1eP5szdPw=;\n\tb=kA0DAAoBcjQGjxahVjwByyZiAGpOF22ilSxmaZQgSqc4otvR1kiuD2c+i8f/p4LKxfZXfgYpa\n\tokCMwQAAQoAHRYhBLXEPUiAmiCKj1g4wHI0Bo8WoVY8BQJqThdtAAoJEHI0Bo8WoVY8s0kP/jzw\n\tCcAyOb93c1Qo1iPtB7o+qoNJkw+wH22hOA/g89PkqLCUcLlBjMukNDymwF5YFmI4oaKpvyQVbRX\n\tYwh3w2sQxuGpbNANxo7TbwR4AYww4r8Ydn9hwkr7OHgws7AgZQclF7VGdHrdKy3OQEAsJ4X9t/M\n\tMEwOs/kc1JjAUr2qvt0Rs5duHWJ/tJgyEGb9XGg2NUEi7aZ1184YUvsmzV25bXlcY7tWc9EH4fz\n\tGqyYiC7jXJAUwwQW2VD1irdlBbGML6dJradRmCJNCQRdWPY7tutKZaVEd8bXkeGFRfxtVewAhmS\n\tBtYNQ7rrCK4bYp2oOAx4tVk5keIj7pk3h/KEZl7GgBF0PB7HN8UjrloX6A3LMKryrHYTH8Ju2U2\n\tA0VKa6ubw9K1j+hlzYlgiPQqc1bxuY5dgeSvD3totRIHkOXAOp68SsfpbzH0TFvU18vFX/Y8KHz\n\tDsvGTDw8oVev4lmG55w3Y3HHeOtaffDTCXmaSHvi6fVesiD9/NfexmncmvR+TKr73aKgogiAYBs\n\tqtQE0U9itUD++vaiGXRRwXnM83M756sSgo9yLVpekLVu//FuFOTJk6aFrdSENxu7aeMx1v8t92B\n\tzUuqnLxxTnypCV75MgU5i9CruGaCVT/dKBzXXvegTJQ5b/neDUF7FozzHvbKvxYecIJMLXdUwiI\n\tA97sU", "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": "Move the Interpolator<lsc::Components>: specialization from the RkISP1\nIPA to the lsc_base header.\n\nSigned-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>\nReviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n---\n src/ipa/libipa/lsc_base.cpp | 15 +++++++++++++++\n src/ipa/libipa/lsc_base.h | 21 +++++++++++++++++++++\n src/ipa/rkisp1/algorithms/lsc.cpp | 27 ---------------------------\n 3 files changed, 36 insertions(+), 27 deletions(-)", "diff": "diff --git a/src/ipa/libipa/lsc_base.cpp b/src/ipa/libipa/lsc_base.cpp\nindex d05075b988d7..945020794b57 100644\n--- a/src/ipa/libipa/lsc_base.cpp\n+++ b/src/ipa/libipa/lsc_base.cpp\n@@ -49,6 +49,21 @@ namespace lsc {\n \n } /* namespace lsc */\n \n+#ifndef __DOXYGEN__\n+template<>\n+void Interpolator<lsc::Components>::\n+\tinterpolate(const lsc::Components &a,\n+\t\t const lsc::Components &b,\n+\t\t lsc::Components &dest,\n+\t\t double lambda)\n+{\n+\tinterpolateVector(a.r, b.r, dest.r, lambda);\n+\tinterpolateVector(a.gr, b.gr, dest.gr, lambda);\n+\tinterpolateVector(a.gb, b.gb, dest.gb, lambda);\n+\tinterpolateVector(a.b, b.b, dest.b, lambda);\n+}\n+#endif\n+\n /**\n * \\class LscImplementation\n * \\brief Pure virtual base class for LSC algorithm implementations\ndiff --git a/src/ipa/libipa/lsc_base.h b/src/ipa/libipa/lsc_base.h\nindex 93f819d3c2af..51cc35888209 100644\n--- a/src/ipa/libipa/lsc_base.h\n+++ b/src/ipa/libipa/lsc_base.h\n@@ -18,6 +18,8 @@\n \n #include \"libcamera/internal/value_node.h\"\n \n+#include \"interpolator.h\"\n+\n namespace libcamera {\n \n namespace ipa {\n@@ -35,6 +37,25 @@ using ComponentsMap = std::map<unsigned int, Components>;\n \n } /* namespace lsc */\n \n+#ifndef __DOXYGEN__\n+template<typename T>\n+void interpolateVector(const std::vector<T> &a, const std::vector<T> &b,\n+\t\t std::vector<T> &dest, double lambda)\n+{\n+\tASSERT(a.size() == b.size());\n+\tdest.resize(a.size());\n+\tfor (size_t i = 0; i < a.size(); i++)\n+\t\tdest[i] = a[i] * (1.0 - lambda) + b[i] * lambda;\n+}\n+\n+template<>\n+void Interpolator<lsc::Components>::\n+\tinterpolate(const lsc::Components &a,\n+\t\t const lsc::Components &b,\n+\t\t lsc::Components &dest,\n+\t\t double lambda);\n+#endif /* __DOXYGEN__ */\n+\n class LscImplementation\n {\n public:\ndiff --git a/src/ipa/rkisp1/algorithms/lsc.cpp b/src/ipa/rkisp1/algorithms/lsc.cpp\nindex 84d2b1d3bdac..8d92ee89cdaf 100644\n--- a/src/ipa/rkisp1/algorithms/lsc.cpp\n+++ b/src/ipa/rkisp1/algorithms/lsc.cpp\n@@ -25,33 +25,6 @@\n \n namespace libcamera {\n \n-namespace ipa {\n-\n-template<typename T>\n-void interpolateVector(const std::vector<T> &a, const std::vector<T> &b,\n-\t\t std::vector<T> &dest, double lambda)\n-{\n-\tASSERT(a.size() == b.size());\n-\tdest.resize(a.size());\n-\tfor (size_t i = 0; i < a.size(); i++)\n-\t\tdest[i] = a[i] * (1.0 - lambda) + b[i] * lambda;\n-}\n-\n-template<>\n-void Interpolator<lsc::Components>::\n-\tinterpolate(const lsc::Components &a,\n-\t\t const lsc::Components &b,\n-\t\t lsc::Components &dest,\n-\t\t double lambda)\n-{\n-\tinterpolateVector(a.r, b.r, dest.r, lambda);\n-\tinterpolateVector(a.gr, b.gr, dest.gr, lambda);\n-\tinterpolateVector(a.gb, b.gb, dest.gb, lambda);\n-\tinterpolateVector(a.b, b.b, dest.b, lambda);\n-}\n-\n-} /* namespace ipa */\n-\n namespace ipa::rkisp1::algorithms {\n \n \n", "prefixes": [ "v4", "14/35" ] }