From patchwork Thu Nov 6 16:42:24 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stefan Klug X-Patchwork-Id: 24980 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 78AC1BDE4C for ; Thu, 6 Nov 2025 16:42:46 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 90E4760A8B; Thu, 6 Nov 2025 17:42:45 +0100 (CET) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="a0wPU7P1"; 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 D5F30606E6 for ; Thu, 6 Nov 2025 17:42:43 +0100 (CET) Received: from ideasonboard.com (unknown [IPv6:2a00:6020:448c:6c00:1d00:3bae:f27:7601]) by perceval.ideasonboard.com (Postfix) with UTF8SMTPSA id A816F6AF; Thu, 6 Nov 2025 17:40:48 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1762447248; bh=s94Ms0z5emjQLXxcBnHiZe+je3imskF5ZlWQm9wmn2w=; h=From:To:Cc:Subject:Date:From; b=a0wPU7P1JxaG/pwQPCgqon7+3aZkrSV6bytI1Imw0YkIfbQXWI3d/ySf0Sazqbt1b CRO31vktlJEw4/Jdz/+pz3isO7pQQJdFsLgXgziCTpSxVRm2ZeJmAEXzq7USqffLaI MleDPSab9oVVXph8TDz/idLZ/oUdBMK4qHcpeYKU= From: Stefan Klug To: libcamera-devel@lists.libcamera.org Cc: Stefan Klug Subject: [PATCH v2 0/3] libipa: agc: Fix constraints yTarget handling and add PWL support Date: Thu, 6 Nov 2025 17:42:24 +0100 Message-ID: <20251106164239.460738-1-stefan.klug@ideasonboard.com> X-Mailer: git-send-email 2.51.0 MIME-Version: 1.0 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" Hi all, This series introduces support for specifying the relativeLuminanceTarget based on the current lux level by using a PWL for the mapping. The typical use case is to ensure that images in a bright environment are regulated a tad brighter than images taken in a dark environment. This is a very subjective setting and should therefore be left to the user/camera designer. Mainlining this feature was already tried in [1] but got rejected at that time because there was no direct user of it. Meanwhile the lux algorithm got upstreamed and we have users that actually miss that feature. So I start another try of upstreaming it (patch 2). During implementation it turned out that the yTarget of AGC constraints never worked at all. So patches 3 fixes that and also add PWL support for the constraints which is sensible to do if we support it for the relativeLuminanceTarget. Changes in v2: - Dropped patch 1 that implemented Pwl::swap() as the intended use can already be achieved using the autogenerated move assignment operator and move constructor together with std::move() - Dropped the addition of a initializer_list based constructor to Pwl as this can be achieved in a bit more verbose way already. - Added patch 1 that extends the yaml PWL reading code to support plain values. This also simplified the subsequent patches. Best regards, Stefan Stefan Klug (3): ipa: libipa: pwl: Allow to parse a plain yaml value as single point PWL ipa: libipa: agc_mean_luminance: Change luminance target to piecewise linear function ipa: libipa: agc_mean_luminance: Fix yTarget handling in constraints src/ipa/libipa/agc_mean_luminance.cpp | 92 +++++++++++++++++++++------ src/ipa/libipa/agc_mean_luminance.h | 16 +++-- src/ipa/libipa/pwl.cpp | 9 +++ src/ipa/rkisp1/algorithms/agc.cpp | 1 + src/ipa/rkisp1/algorithms/wdr.cpp | 3 +- 5 files changed, 97 insertions(+), 24 deletions(-)