Patch Detail
Show a patch.
GET /api/1.1/patches/22544/?format=api
{ "id": 22544, "url": "https://patchwork.libcamera.org/api/1.1/patches/22544/?format=api", "web_url": "https://patchwork.libcamera.org/patch/22544/", "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": "<20250113135108.13924-10-mzamazal@redhat.com>", "date": "2025-01-13T13:51:06", "name": "[v4,9/9] libcamera: software_isp: Apply CCM in debayering", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "620677051d6e87d0803956af25c502458e0eebc8", "submitter": { "id": 177, "url": "https://patchwork.libcamera.org/api/1.1/people/177/?format=api", "name": "Milan Zamazal", "email": "mzamazal@redhat.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/22544/mbox/", "series": [ { "id": 4946, "url": "https://patchwork.libcamera.org/api/1.1/series/4946/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4946", "date": "2025-01-13T13:50:57", "name": "Software ISP support for CCM", "version": 4, "mbox": "https://patchwork.libcamera.org/series/4946/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/22544/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/22544/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 92FB5C32F6\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 13 Jan 2025 13:51:51 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 3C46C6854D;\n\tMon, 13 Jan 2025 14:51:51 +0100 (CET)", "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 3AB096854D\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 13 Jan 2025 14:51:49 +0100 (CET)", "from mx-prod-mc-01.mail-002.prod.us-west-2.aws.redhat.com\n\t(ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63])\n\tby relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3,\n\tcipher=TLS_AES_256_GCM_SHA384) id us-mta-338-kBeBo49ROji8S0GkGW_CbQ-1;\n\tMon, 13 Jan 2025 08:51:45 -0500", "from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com\n\t(mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com\n\t[10.30.177.17])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\tkey-exchange X25519 server-signature RSA-PSS (2048 bits)\n\tserver-digest SHA256) (No client certificate requested)\n\tby mx-prod-mc-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix)\n\twith ESMTPS id 227001955F34; Mon, 13 Jan 2025 13:51:44 +0000 (UTC)", "from mzamazal-thinkpadp1gen3.tpbc.com (unknown [10.45.224.6])\n\tby mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix)\n\twith ESMTP id A5A081956056; Mon, 13 Jan 2025 13:51:41 +0000 (UTC)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"UeFlyWop\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1736776308;\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=+80hUaXe28lFpYqnRQmjEjCtat61TO5MEGgCc+xrcJI=;\n\tb=UeFlyWopAryw4OiNo8+LZVGkKFoc382jo04tyZc9sLnOa77Cc/gGjYr8ipYzJ9tjZXKnI7\n\tu/E2Bj0Qm6src3P/E2UGpEEiBfQ/fxJxxR3u0UX8Dx6u13nf5909O5Hga5DpM2pUvAHjbK\n\tfernLPUtMIVAs2plNaoR/BC51DRVdEQ=", "X-MC-Unique": "kBeBo49ROji8S0GkGW_CbQ-1", "X-Mimecast-MFC-AGG-ID": "kBeBo49ROji8S0GkGW_CbQ", "From": "Milan Zamazal <mzamazal@redhat.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Milan Zamazal <mzamazal@redhat.com>,\n\tRobert Mader <robert.mader@collabora.com>,\n\tHans de Goede <hdegoede@redhat.com>,\n\tLaurent Pinchart <laurent.pinchart@ideasonboard.com>,\n\tKieran Bingham <kieran.bingham@ideasonboard.com>", "Subject": "[PATCH v4 9/9] libcamera: software_isp: Apply CCM in debayering", "Date": "Mon, 13 Jan 2025 14:51:06 +0100", "Message-ID": "<20250113135108.13924-10-mzamazal@redhat.com>", "In-Reply-To": "<20250113135108.13924-1-mzamazal@redhat.com>", "References": "<20250113135108.13924-1-mzamazal@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 3.0 on 10.30.177.17", "X-Mimecast-Spam-Score": "0", "X-Mimecast-MFC-PROC-ID": "EG_kMNQWHMs-ou4wgtEWOODpuJiSpeqBOWieuhaai88_1736776304", "X-Mimecast-Originator": "redhat.com", "Content-Transfer-Encoding": "8bit", "content-type": "text/plain; charset=\"US-ASCII\"; x-default=true", "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": "This patch applies color correction matrix (CCM) in debayering if the\nCCM is specified. Not using CCM must still be supported for performance\nreasons.\n\nThe CCM is applied as follows:\n\n [r1 r2 r3]\n [r g b] * [g1 g2 g3]\n [b1 b2 b3]\n\nThe CCM matrix (the right side of the multiplication) is constant during\nsingle frame processing, while the input pixel (the left side) changes.\nBecause each of the color channels is only 8-bit in software ISP, we can\nmake 9 lookup tables with 256 input values for multiplications of each\nof the r_i, g_i, b_i values. This way we don't have to multiply each\npixel, we can use table lookups and additions instead. Gamma (which is\nnon-linear and thus cannot be a part of the 9 lookup tables values) is\napplied on the final values rounded to integers using another lookup\ntable.\n\nWe use int16_t to store the precomputed multiplications. This seems to\nbe noticeably (>10%) faster than `float' for the price of slightly less\naccuracy and it covers the range of values that sane CCMs produce. The\nselection and structure of data is performance critical, for example\nusing bytes would add significant (>10%) speedup but would be too short\nto cover the value range.\n\nThe color lookup tables are changed to unions to be able to serve as\nboth simple lookup tables or CCM lookup tables. This is arguable but it\nkeeps the code easier to understand if nothing else. It is important to\nmake the unions on the whole lookup tables rather than their values\nbecause the latter might cause a noticeable slowdown on simple lookup\ndue to the sheer fact that the table is not compact. Using unions makes\nthe initialization of the tables dubious because it is done before the\ndecision about using CCM is made but the initial values are dubious\nanyway. Using std::variant instead of unions would be even more\ndifficult, consider e.g. mmap allocation and the followup type-casting\nof the debayer params.\n\nThe tables are copied (as before), which is not elegant but also not a\nbig problem. There are patches posted that use shared buffers for\nparameters passing in software ISP (see software ISP TODO #5) and they\ncan be adjusted for the new parameter format.\n\nColor gains from white balance are supposed not to be a part of the\nspecified CCM. They are applied on it using matrix multiplication,\nwhich is simple and in correspondence with future additions in the form\nof matrix multiplication, like saturation adjustment.\n\nWith this patch, the reported per-frame slowdown when applying CCM is\nabout 45% on Debix Model A and about 75% on TI AM69 SK.\n\nUsing std::clamp in debayering adds some performance penalty (a few\npercent). The clamping is necessary to eliminate out of range values\npossibly produced by the CCM. If it could be avoided by adjusting the\nprecomputed tables some way then performance could be improved a bit.\n\nSigned-off-by: Milan Zamazal <mzamazal@redhat.com>\n---\n .../internal/software_isp/debayer_params.h | 20 +++++-\n src/ipa/simple/algorithms/lut.cpp | 63 ++++++++++++++-----\n src/ipa/simple/algorithms/lut.h | 1 +\n src/libcamera/software_isp/debayer.cpp | 50 +++++++++++++++\n src/libcamera/software_isp/debayer_cpu.cpp | 34 +++++++---\n src/libcamera/software_isp/debayer_cpu.h | 7 ++-\n src/libcamera/software_isp/software_isp.cpp | 6 +-\n 7 files changed, 147 insertions(+), 34 deletions(-)", "diff": "diff --git a/include/libcamera/internal/software_isp/debayer_params.h b/include/libcamera/internal/software_isp/debayer_params.h\nindex 7d8fdd48..f8731eae 100644\n--- a/include/libcamera/internal/software_isp/debayer_params.h\n+++ b/include/libcamera/internal/software_isp/debayer_params.h\n@@ -18,11 +18,25 @@ namespace libcamera {\n struct DebayerParams {\n \tstatic constexpr unsigned int kRGBLookupSize = 256;\n \n+\tstruct CcmColumn {\n+\t\tint16_t r;\n+\t\tint16_t g;\n+\t\tint16_t b;\n+\t};\n+\n \tusing ColorLookupTable = std::array<uint8_t, kRGBLookupSize>;\n+\tusing CcmLookupTable = std::array<CcmColumn, kRGBLookupSize>;\n+\tusing GammaLookupTable = std::array<uint8_t, kRGBLookupSize>;\n+\n+\tunion LookupTable {\n+\t\tColorLookupTable simple;\n+\t\tCcmLookupTable ccm;\n+\t};\n \n-\tColorLookupTable red;\n-\tColorLookupTable green;\n-\tColorLookupTable blue;\n+\tLookupTable red;\n+\tLookupTable green;\n+\tLookupTable blue;\n+\tGammaLookupTable gammaLut;\n };\n \n } /* namespace libcamera */\ndiff --git a/src/ipa/simple/algorithms/lut.cpp b/src/ipa/simple/algorithms/lut.cpp\nindex 243f0818..ddb2adf1 100644\n--- a/src/ipa/simple/algorithms/lut.cpp\n+++ b/src/ipa/simple/algorithms/lut.cpp\n@@ -80,6 +80,11 @@ void Lut::updateGammaTable(IPAContext &context)\n \tcontext.activeState.gamma.contrast = contrast;\n }\n \n+int16_t Lut::ccmValue(unsigned int i, float ccm) const\n+{\n+\treturn std::round(i * ccm);\n+}\n+\n void Lut::prepare(IPAContext &context,\n \t\t [[maybe_unused]] const uint32_t frame,\n \t\t [[maybe_unused]] IPAFrameContext &frameContext,\n@@ -91,28 +96,52 @@ void Lut::prepare(IPAContext &context,\n \t * observed, it's not permanently prone to minor fluctuations or\n \t * rounding errors.\n \t */\n-\tif (context.activeState.gamma.blackLevel != context.activeState.blc.level ||\n-\t context.activeState.gamma.contrast != context.activeState.knobs.contrast)\n+\tconst bool gammaUpdateNeeded =\n+\t\tcontext.activeState.gamma.blackLevel != context.activeState.blc.level ||\n+\t\tcontext.activeState.gamma.contrast != context.activeState.knobs.contrast;\n+\tif (gammaUpdateNeeded)\n \t\tupdateGammaTable(context);\n \n \tauto &gains = context.activeState.awb.gains;\n \tauto &gammaTable = context.activeState.gamma.gammaTable;\n \tconst unsigned int gammaTableSize = gammaTable.size();\n-\n-\tfor (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++) {\n-\t\tconst double div = static_cast<double>(DebayerParams::kRGBLookupSize) /\n-\t\t\t\t gammaTableSize;\n-\t\t/* Apply gamma after gain! */\n-\t\tunsigned int idx;\n-\t\tidx = std::min({ static_cast<unsigned int>(i * gains.red / div),\n-\t\t\t\t gammaTableSize - 1 });\n-\t\tparams->red[i] = gammaTable[idx];\n-\t\tidx = std::min({ static_cast<unsigned int>(i * gains.green / div),\n-\t\t\t\t gammaTableSize - 1 });\n-\t\tparams->green[i] = gammaTable[idx];\n-\t\tidx = std::min({ static_cast<unsigned int>(i * gains.blue / div),\n-\t\t\t\t gammaTableSize - 1 });\n-\t\tparams->blue[i] = gammaTable[idx];\n+\tconst double div = static_cast<double>(DebayerParams::kRGBLookupSize) /\n+\t\t\t gammaTableSize;\n+\n+\tif (!context.activeState.ccm.enabled) {\n+\t\tfor (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++) {\n+\t\t\t/* Apply gamma after gain! */\n+\t\t\tunsigned int idx;\n+\t\t\tidx = std::min({ static_cast<unsigned int>(i * gains.red / div),\n+\t\t\t\t\t gammaTableSize - 1 });\n+\t\t\tparams->red.simple[i] = gammaTable[idx];\n+\t\t\tidx = std::min({ static_cast<unsigned int>(i * gains.green / div),\n+\t\t\t\t\t gammaTableSize - 1 });\n+\t\t\tparams->green.simple[i] = gammaTable[idx];\n+\t\t\tidx = std::min({ static_cast<unsigned int>(i * gains.blue / div),\n+\t\t\t\t\t gammaTableSize - 1 });\n+\t\t\tparams->blue.simple[i] = gammaTable[idx];\n+\t\t}\n+\t} else if (context.activeState.ccm.changed || gammaUpdateNeeded) {\n+\t\tMatrix<double, 3, 3> gainCcm = { { gains.red, 0, 0,\n+\t\t\t\t\t\t 0, gains.green, 0,\n+\t\t\t\t\t\t 0, 0, gains.blue } };\n+\t\tauto ccm = gainCcm * context.activeState.ccm.ccm;\n+\t\tauto &red = params->red.ccm;\n+\t\tauto &green = params->green.ccm;\n+\t\tauto &blue = params->blue.ccm;\n+\t\tfor (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++) {\n+\t\t\tred[i].r = ccmValue(i, ccm[0][0]);\n+\t\t\tred[i].g = ccmValue(i, ccm[1][0]);\n+\t\t\tred[i].b = ccmValue(i, ccm[2][0]);\n+\t\t\tgreen[i].r = ccmValue(i, ccm[0][1]);\n+\t\t\tgreen[i].g = ccmValue(i, ccm[1][1]);\n+\t\t\tgreen[i].b = ccmValue(i, ccm[2][1]);\n+\t\t\tblue[i].r = ccmValue(i, ccm[0][2]);\n+\t\t\tblue[i].g = ccmValue(i, ccm[1][2]);\n+\t\t\tblue[i].b = ccmValue(i, ccm[2][2]);\n+\t\t\tparams->gammaLut[i] = gammaTable[i / div];\n+\t\t}\n \t}\n }\n \ndiff --git a/src/ipa/simple/algorithms/lut.h b/src/ipa/simple/algorithms/lut.h\nindex 889f864b..77324800 100644\n--- a/src/ipa/simple/algorithms/lut.h\n+++ b/src/ipa/simple/algorithms/lut.h\n@@ -33,6 +33,7 @@ public:\n \n private:\n \tvoid updateGammaTable(IPAContext &context);\n+\tint16_t ccmValue(unsigned int i, float ccm) const;\n };\n \n } /* namespace ipa::soft::algorithms */\ndiff --git a/src/libcamera/software_isp/debayer.cpp b/src/libcamera/software_isp/debayer.cpp\nindex 45fe6960..559de3e6 100644\n--- a/src/libcamera/software_isp/debayer.cpp\n+++ b/src/libcamera/software_isp/debayer.cpp\n@@ -23,11 +23,56 @@ namespace libcamera {\n * \\brief Size of a color lookup table\n */\n \n+/**\n+ * \\struct DebayerParams::CcmColumn\n+ * \\brief Type of a single column of a color correction matrix\n+ */\n+\n+/**\n+ * \\var DebayerParams::CcmColumn::r\n+ * \\brief Red (first) component of a CCM column\n+ */\n+\n+/**\n+ * \\var DebayerParams::CcmColumn::g\n+ * \\brief Green (second) component of a CCM column\n+ */\n+\n+/**\n+ * \\var DebayerParams::CcmColumn::b\n+ * \\brief Blue (third) component of a CCM column\n+ */\n+\n /**\n * \\typedef DebayerParams::ColorLookupTable\n+ * \\brief Type of the simple lookup tables for red, green, blue values\n+ */\n+\n+/**\n+ * \\typedef DebayerParams::CcmLookupTable\n+ * \\brief Type of the CCM lookup tables for red, green, blue values\n+ */\n+\n+/**\n+ * \\typedef DebayerParams::GammaLookupTable\n+ * \\brief Type of the gamma lookup tables for CCM\n+ */\n+\n+/**\n+ * \\union DebayerParams::LookupTable\n * \\brief Type of the lookup tables for red, green, blue values\n */\n \n+/**\n+ * \\var DebayerParams::LookupTable::simple\n+ * \\brief Simple lookup table for red, green, blue values\n+ */\n+\n+/**\n+ * \\var DebayerParams::LookupTable::ccm\n+ * \\brief CCM lookup table for red, green, blue values\n+ */\n+\n /**\n * \\var DebayerParams::red\n * \\brief Lookup table for red color, mapping input values to output values\n@@ -43,6 +88,11 @@ namespace libcamera {\n * \\brief Lookup table for blue color, mapping input values to output values\n */\n \n+/**\n+ * \\var DebayerParams::gammaLut\n+ * \\brief Gamma lookup table used with color correction matrix\n+ */\n+\n /**\n * \\class Debayer\n * \\brief Base debayering class\ndiff --git a/src/libcamera/software_isp/debayer_cpu.cpp b/src/libcamera/software_isp/debayer_cpu.cpp\nindex 3c6597f9..38715b72 100644\n--- a/src/libcamera/software_isp/debayer_cpu.cpp\n+++ b/src/libcamera/software_isp/debayer_cpu.cpp\n@@ -11,6 +11,7 @@\n \n #include \"debayer_cpu.h\"\n \n+#include <algorithm>\n #include <stdlib.h>\n #include <sys/ioctl.h>\n #include <time.h>\n@@ -51,8 +52,12 @@ DebayerCpu::DebayerCpu(std::unique_ptr<SwStatsCpu> stats)\n \tenableInputMemcpy_ = true;\n \n \t/* Initialize color lookup tables */\n-\tfor (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++)\n-\t\tred_[i] = green_[i] = blue_[i] = i;\n+\tfor (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++) {\n+\t\tred_.simple[i] = green_.simple[i] = blue_.simple[i] = i;\n+\t\tred_.ccm[i].r = red_.ccm[i].g = red_.ccm[i].b = 0;\n+\t\tgreen_.ccm[i].r = green_.ccm[i].g = green_.ccm[i].b = 0;\n+\t\tblue_.ccm[i].r = blue_.ccm[i].g = blue_.ccm[i].b = 0;\n+\t}\n }\n \n DebayerCpu::~DebayerCpu() = default;\n@@ -62,12 +67,24 @@ DebayerCpu::~DebayerCpu() = default;\n \tconst pixel_t *curr = (const pixel_t *)src[1] + xShift_; \\\n \tconst pixel_t *next = (const pixel_t *)src[2] + xShift_;\n \n-#define STORE_PIXEL(b, g, r) \\\n-\t*dst++ = blue_[b]; \\\n-\t*dst++ = green_[g]; \\\n-\t*dst++ = red_[r]; \\\n-\tif constexpr (addAlphaByte) \\\n-\t\t*dst++ = 255; \\\n+#define GAMMA(value) \\\n+\t*dst++ = gammaLut_[std::clamp(value, 0, static_cast<int>(gammaLut_.size()) - 1)]\n+\n+#define STORE_PIXEL(b_, g_, r_) \\\n+\tif constexpr (ccmEnabled) { \\\n+\t\tDebayerParams::CcmColumn &blue = blue_.ccm[b_]; \\\n+\t\tDebayerParams::CcmColumn &green = green_.ccm[g_]; \\\n+\t\tDebayerParams::CcmColumn &red = red_.ccm[r_]; \\\n+\t\tGAMMA(blue.r + blue.g + blue.b); \\\n+\t\tGAMMA(green.r + green.g + green.b); \\\n+\t\tGAMMA(red.r + red.g + red.b); \\\n+\t} else { \\\n+\t\t*dst++ = blue_.simple[b_]; \\\n+\t\t*dst++ = green_.simple[g_]; \\\n+\t\t*dst++ = red_.simple[r_]; \\\n+\t} \\\n+\tif constexpr (addAlphaByte) \\\n+\t\t*dst++ = 255; \\\n \tx++;\n \n /*\n@@ -752,6 +769,7 @@ void DebayerCpu::process(uint32_t frame, FrameBuffer *input, FrameBuffer *output\n \tgreen_ = params.green;\n \tred_ = swapRedBlueGains_ ? params.blue : params.red;\n \tblue_ = swapRedBlueGains_ ? params.red : params.blue;\n+\tgammaLut_ = params.gammaLut;\n \n \t/* Copy metadata from the input buffer */\n \tFrameMetadata &metadata = output->_d()->metadata();\ndiff --git a/src/libcamera/software_isp/debayer_cpu.h b/src/libcamera/software_isp/debayer_cpu.h\nindex b2ec8f1b..35c51e4f 100644\n--- a/src/libcamera/software_isp/debayer_cpu.h\n+++ b/src/libcamera/software_isp/debayer_cpu.h\n@@ -137,9 +137,10 @@ private:\n \t/* Max. supported Bayer pattern height is 4, debayering this requires 5 lines */\n \tstatic constexpr unsigned int kMaxLineBuffers = 5;\n \n-\tDebayerParams::ColorLookupTable red_;\n-\tDebayerParams::ColorLookupTable green_;\n-\tDebayerParams::ColorLookupTable blue_;\n+\tDebayerParams::LookupTable red_;\n+\tDebayerParams::LookupTable green_;\n+\tDebayerParams::LookupTable blue_;\n+\tDebayerParams::GammaLookupTable gammaLut_;\n \tdebayerFn debayer0_;\n \tdebayerFn debayer1_;\n \tdebayerFn debayer2_;\ndiff --git a/src/libcamera/software_isp/software_isp.cpp b/src/libcamera/software_isp/software_isp.cpp\nindex 6a07de85..40e11b9e 100644\n--- a/src/libcamera/software_isp/software_isp.cpp\n+++ b/src/libcamera/software_isp/software_isp.cpp\n@@ -82,9 +82,9 @@ SoftwareIsp::SoftwareIsp(PipelineHandler *pipe, const CameraSensor *sensor,\n \tfor (unsigned int i = 0; i < 256; i++)\n \t\tgammaTable[i] = UINT8_MAX * std::pow(i / 256.0, 0.5);\n \tfor (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++) {\n-\t\tdebayerParams_.red[i] = gammaTable[i];\n-\t\tdebayerParams_.green[i] = gammaTable[i];\n-\t\tdebayerParams_.blue[i] = gammaTable[i];\n+\t\tdebayerParams_.red.simple[i] = gammaTable[i];\n+\t\tdebayerParams_.green.simple[i] = gammaTable[i];\n+\t\tdebayerParams_.blue.simple[i] = gammaTable[i];\n \t}\n \n \tif (!dmaHeap_.isValid()) {\n", "prefixes": [ "v4", "9/9" ] }