From patchwork Mon Jul 6 08:01:33 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 27213 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 CD571C330A for ; Mon, 6 Jul 2026 08:02:18 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 1952F66041; Mon, 6 Jul 2026 10:02:18 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="aoQorZNs"; dkim-atps=neutral Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 97C7B66024 for ; Mon, 6 Jul 2026 10:02:06 +0200 (CEST) Received: from [192.168.125.177] (mob-109-113-60-81.net.vodafone.it [109.113.60.81]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 4DB56293E; Mon, 6 Jul 2026 10:01:17 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1783324878; bh=oHPPuVkGX5pi4HusVH9TgGsRpDNtBgpyaSHuyM2mitM=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=aoQorZNs3JTIXciJ96t51PtIVWQcE7yFM1zuRWvNPMneNIBMQwKqO1QI2+/RN+ECC qjuGhT3dcqDhaXCC77f6GpC8qS18Qt91qoppcChWujawjpsWr4NmgeDxR1GxVJhWAN HBexJT1nKwajE0hQF/EUH/JhwMnDVzw0pNycfpyA= From: Jacopo Mondi Date: Mon, 06 Jul 2026 10:01:33 +0200 Subject: [PATCH v3 13/35] ipa: libipa: lsc_polynomial: Do not inline functions MIME-Version: 1.0 Message-Id: <20260706-libipa-algorithms-v3-13-968757b038bb@ideasonboard.com> References: <20260706-libipa-algorithms-v3-0-968757b038bb@ideasonboard.com> In-Reply-To: <20260706-libipa-algorithms-v3-0-968757b038bb@ideasonboard.com> To: libcamera-devel@lists.libcamera.org, Stefan Klug Cc: Jacopo Mondi X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=3967; i=jacopo.mondi@ideasonboard.com; h=from:subject:message-id; bh=oHPPuVkGX5pi4HusVH9TgGsRpDNtBgpyaSHuyM2mitM=; b=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBqS2Dwvr/mU9Qeis5xYR9p6ugc+3XnAT0mB+qDy EKDdJ6SwdGJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCaktg8AAKCRByNAaPFqFW PGRJD/9EgTFOVj6S0LWr6BkFIOkXf9JUOeKT2PHLNtxA39ACufG6eijIaUxuAFGXQC6NfzOPUrY XiLHtMj5JDltS7/zK5qTy/Z1WYno956UpJu8xKQKUjXHoq+TFPHu+wnfpWMIoOiBqtYWfG1nnsN U+htL0iUK5b3UAif5TRALKC32F1pFyUHeCzBDvFoNN0g0y1auwjHs4WQW36VFexwzUeQkthlU70 Q7wyCvDGChiuGdciNupGLwVRv7ktwBOxGmoMlXD64+CVNh+sJnONpFOJjewziJdUnJOsJPwfSfG qyySsa8RpKajhYwgW4SmFZeordqJLOVrrdSe0hwTeptI0c0StxVAOi6TgsHe1i3L8mabkIx+Ixa y9eXkBDAizlec+IyZaws+H3S6YEnYCbhDaYngE3BOdFq8DFl1mfVPo4TVdOT7/gdYcgrNqkcoem qiaR//Tt5t4hzLHO8VB5rHxxEBhr5Ns6DsTUTCPoyj5ClRqLCwRZY5PEikw0DTcHwRCjQW14Tgq b5oKrq9wBzzdhZf0alScCShHLisnwmUAIvhwh1eyUEGC+Jy4RgKyUELLN98UWzGoFK9pQ8XEB+8 Jm8wJsJC95ceMM+f8iTcMl8zmtmWiN1XvcQiHGYHBj4S3Y85sJeK/kFxOMcbF6MPDCBSDvzSXEj YNN3Pn4iORP0ujw== X-Developer-Key: i=jacopo.mondi@ideasonboard.com; a=openpgp; fpr=72392EDC88144A65C701EA9BA5826A2587AD026B 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" There is no reason to inline the Polynomial class implementation. Move functions implementation to the corresponding .cpp file. While at it remove a rougue empty line between class members definitions. Signed-off-by: Jacopo Mondi --- src/ipa/libipa/lsc_polynomial.cpp | 34 +++++++++++++++++++++++++++++++--- src/ipa/libipa/lsc_polynomial.h | 36 +++--------------------------------- 2 files changed, 34 insertions(+), 36 deletions(-) diff --git a/src/ipa/libipa/lsc_polynomial.cpp b/src/ipa/libipa/lsc_polynomial.cpp index d8981bd8a26a..a845d8b1783b 100644 --- a/src/ipa/libipa/lsc_polynomial.cpp +++ b/src/ipa/libipa/lsc_polynomial.cpp @@ -47,7 +47,6 @@ namespace lsc { */ /** - * \fn Polynomial::sampleAtNormalizedPixelPos(double x, double y) * \brief Sample the polynomial at the given normalized pixel position * * This functions samples the polynomial at the given pixel position divided by @@ -57,9 +56,20 @@ namespace lsc { * \param y y position in normalized coordinates * \return The sampled value */ +double Polynomial::sampleAtNormalizedPixelPos(double x, double y) const +{ + double dx = x - cnx_; + double dy = y - cny_; + double r = sqrt(dx * dx + dy * dy); + double res = 1.0; + + for (unsigned int i = 0; i < coefficients_.size(); i++) + res += coefficients_[i] * std::pow(r, (i + 1) * 2); + + return res; +} /** - * \fn Polynomial::getM() * \brief Get the value m as described in the dng specification * * Returns m according to dng spec. m represents the Euclidean distance @@ -68,9 +78,17 @@ namespace lsc { * * \return The sampled value */ +double Polynomial::getM() const +{ + double cpx = imageSize_.width * cx_; + double cpy = imageSize_.height * cy_; + double mx = std::max(cpx, std::fabs(imageSize_.width - cpx)); + double my = std::max(cpy, std::fabs(imageSize_.height - cpy)); + + return sqrt(mx * mx + my * my); +} /** - * \fn Polynomial::setReferenceImageSize(const Size &size) * \brief Set the reference image size * * Set the reference image size that is used for subsequent calls to getM() and @@ -78,6 +96,16 @@ namespace lsc { * * \param size The size of the reference image */ +void Polynomial::setReferenceImageSize(const Size &size) +{ + assert(!size.isNull()); + imageSize_ = size; + + /* Calculate normalized centers */ + double m = getM(); + cnx_ = (size.width * cx_) / m; + cny_ = (size.height * cy_) / m; +} } /* namespace lsc */ diff --git a/src/ipa/libipa/lsc_polynomial.h b/src/ipa/libipa/lsc_polynomial.h index 3be33edfe7d8..2caf46d2d759 100644 --- a/src/ipa/libipa/lsc_polynomial.h +++ b/src/ipa/libipa/lsc_polynomial.h @@ -37,38 +37,9 @@ public: { } - double sampleAtNormalizedPixelPos(double x, double y) const - { - double dx = x - cnx_; - double dy = y - cny_; - double r = sqrt(dx * dx + dy * dy); - double res = 1.0; - for (unsigned int i = 0; i < coefficients_.size(); i++) { - res += coefficients_[i] * std::pow(r, (i + 1) * 2); - } - return res; - } - - double getM() const - { - double cpx = imageSize_.width * cx_; - double cpy = imageSize_.height * cy_; - double mx = std::max(cpx, std::fabs(imageSize_.width - cpx)); - double my = std::max(cpy, std::fabs(imageSize_.height - cpy)); - - return sqrt(mx * mx + my * my); - } - - void setReferenceImageSize(const Size &size) - { - assert(!size.isNull()); - imageSize_ = size; - - /* Calculate normalized centers */ - double m = getM(); - cnx_ = (size.width * cx_) / m; - cny_ = (size.height * cy_) / m; - } + double sampleAtNormalizedPixelPos(double x, double y) const; + double getM() const; + void setReferenceImageSize(const Size &size); private: double cx_; @@ -76,7 +47,6 @@ private: double cnx_; double cny_; std::array coefficients_; - Size imageSize_; };