From patchwork Mon Sep 14 14:53:43 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Dan Scally X-Patchwork-Id: 28288 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id E43B3C3226 for ; Mon, 14 Sep 2026 14:54:05 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 2E555686B7; Mon, 14 Sep 2026 16:54:02 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="B8ZZUyq3"; dkim-atps=neutral Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 3F80568670 for ; Mon, 14 Sep 2026 16:53:56 +0200 (CEST) Received: from [127.0.1.1] (chfd-03-b2-v4wan-176392-cust229.vm15.cable.virginm.net [82.19.20.230]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 281731B39; Mon, 14 Sep 2026 16:52:16 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1789397536; bh=8qgoyU4sOLbW8aF7t6DynI+rE3RSdbbozLafWtXYOgM=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=B8ZZUyq3ZlZp7ASN/D0pUod84Hf8TnfJAja6p53LpRrOUrEbx5pCojmDZvgFdcp7x YzoBW+9JhOt+7WYRcwxmX/jz24LSqgiVf9M2tn+SvcfLJbq0W+R7Bg0mJi2W1mt0wt B8Lk8t1EtW3EvLbvUQ36iMcIQR9VTnJ9JxVdbQ28= From: Daniel Scally Date: Mon, 14 Sep 2026 15:53:43 +0100 Subject: [PATCH v4 03/10] ipu3: awb: Remove CCM settings from Awb algorithm MIME-Version: 1.0 Message-Id: <20260914-ipu3-libipa-rework-v4-3-ff8ee2d3dd24@ideasonboard.com> References: <20260914-ipu3-libipa-rework-v4-0-ff8ee2d3dd24@ideasonboard.com> In-Reply-To: <20260914-ipu3-libipa-rework-v4-0-ff8ee2d3dd24@ideasonboard.com> To: libcamera-devel@lists.libcamera.org Cc: Daniel Scally , Jacopo Mondi , Kieran Bingham , =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=1746; i=dan.scally@ideasonboard.com; h=from:subject:message-id; bh=8qgoyU4sOLbW8aF7t6DynI+rE3RSdbbozLafWtXYOgM=; b=owEBbQKS/ZANAwAKAchJV3psRXUyAcsmYgBqqAqANtQAVFqlgFNxFvt/w46OOEKdxAMu6TTHU 3qdd6A7IYyJAjMEAAEKAB0WIQQqyuwyDnZdb+mxmm/ISVd6bEV1MgUCaqgKgAAKCRDISVd6bEV1 Mk3GD/sHGuvyzrR38DNSro+osBRJKI/AjR/9wSsHUJLkJBSap/rDTrhS9aBtJwepeQJAvgpRBvC jtdRWp/E+mVKWX0++RZzMauVyHOFsPRiDdBzmqLxdJtZIgXCzdOPCGd2DlrE7jaY3EyBa9VLoFl ND6rAl7f2Kxn/PPyYPFtvu1kSiPfVwl1plwvvLTxxy/ZjBogpH4cyhAyVWJHt/qgkNGuGHi+fAs OYtMnfOksEuoi92nX20H1ewOB7bSxYCim8UoSVXRh1Y8ORlHSVFYtUevqZJzr6P4PDAPKw5APJA 5SELOQ+CjIt6uZNGtdnxMNM72zVI4yALr9tHsVTm8UVqKvbRfdapPJN05sBexfj6pOZFvaN/RW7 AO/rO/zKYX98anXJg58i8CF1hWOcTZsKilmvCjC3FY/aWB10yElKHMIt4SaAiKH4tRkdCiB60zl NTGEhVzcFYlY6T6UFBv4PCgMyrBPQ3hsQbuRcGuewXtu5HBEBexMitZyZAmhkT+jSDrYZcP/scD rmkCiMdU45+q6USDgg6mfyZp3HogZtRTE5yE0oumtdhMPQ3BuBWs/J9iCNcJBKwaHHVAOX2eQWZ 3XbY6DUTX/H04xz4ZXe08dI3PMgaJiTvRfUa1CEsvmqKM9EInmZ70XRQ7ST4BHbluIOiKq3MR/f uRL3Hw4jN3fswIw== X-Developer-Key: i=dan.scally@ideasonboard.com; a=openpgp; fpr=EEC699ACA1B7CB5D31330C0BBD501C2A3546CCF6 X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" The IPU3 AWB algorithm currently defines a CCM matrix that is just a no-op. Remove the settings entirely; they need really to go in to a dedicated CCM algorithm. Reviewed-by: Jacopo Mondi Reviewed-by: Kieran Bingham Reviewed-by: Barnabás Pőcze Signed-off-by: Daniel Scally --- Changes in v4: - None Changes in v3: - None Changes in v2: - None --- src/ipa/ipu3/algorithms/awb.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/src/ipa/ipu3/algorithms/awb.cpp b/src/ipa/ipu3/algorithms/awb.cpp index 6b4418aa6f3add0a41cae77000c1f5f67897111a..613bdbbf51cd127e03906571813260a6857067dc 100644 --- a/src/ipa/ipu3/algorithms/awb.cpp +++ b/src/ipa/ipu3/algorithms/awb.cpp @@ -112,13 +112,6 @@ static const struct ipu3_uapi_bnr_static_config imguCssBnrDefaults = { .opt_center_sqr = { 419904, 133956 }, }; -/* Default color correction matrix defined as an identity matrix */ -static const struct ipu3_uapi_ccm_mat_config imguCssCcmDefault = { - 8191, 0, 0, 0, - 0, 8191, 0, 0, - 0, 0, 8191, 0 -}; - /** * \class Awb * \brief A Grey world white balance correction algorithm @@ -293,12 +286,9 @@ void Awb::prepare(IPAContext &context, LOG(IPU3Awb, Debug) << "Color temperature estimated: " << asyncResults_.temperatureK; - /* The CCM matrix may change when color temperature will be used */ - params->acc_param.ccm = imguCssCcmDefault; params->use.acc_awb = 1; params->use.acc_bnr = 1; - params->use.acc_ccm = 1; } /* Generate an RGB vector with the average values for each zone */