From patchwork Mon Jul 20 14:59:32 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 27406 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 A6847C330A for ; Mon, 20 Jul 2026 15:00:05 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id D458367E90; Mon, 20 Jul 2026 17:00:04 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="eyNM48PK"; 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 1981767E61 for ; Mon, 20 Jul 2026 16:59:49 +0200 (CEST) Received: from [192.168.1.106] (mob-5-90-63-152.net.vodafone.it [5.90.63.152]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 366D51D17; Mon, 20 Jul 2026 16:58:50 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1784559530; bh=+BLekKS2khj7QSEHhQsHfSrQMgwm055rzcXMP/0cFSk=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=eyNM48PKRwWL867hYSmVv2WRPrrLKRO7TMCRhzFQeWCZO1bRMj4f3eJA0wmddu7uB +oCmAQ0ZHxzWqplFiu/CtQKiusdTgt8Tns2Z/aaExllfcA+n5AotcLz2tdezQjQ5pY rOZupXpn4+lD1TMXtfCf8PvOQg/xPMDUC90w26E0= From: Jacopo Mondi Date: Mon, 20 Jul 2026 16:59:32 +0200 Subject: [PATCH v6 14/31] ipa: libipa: lsc_polynomial: Do not inline functions MIME-Version: 1.0 Message-Id: <20260720-libipa-algorithms-v6-14-ececb73f97cb@ideasonboard.com> References: <20260720-libipa-algorithms-v6-0-ececb73f97cb@ideasonboard.com> In-Reply-To: <20260720-libipa-algorithms-v6-0-ececb73f97cb@ideasonboard.com> To: libcamera-devel@lists.libcamera.org, Stefan Klug , Milan Zamazal Cc: Jacopo Mondi , Kieran Bingham X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=4087; i=jacopo.mondi@ideasonboard.com; h=from:subject:message-id; bh=+BLekKS2khj7QSEHhQsHfSrQMgwm055rzcXMP/0cFSk=; b=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBqXjfWAcgqvVKPJhQnKbu+c6WO0AEjEvPtdpXCz qDwybG1xmyJAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCal431gAKCRByNAaPFqFW PKxcD/0dWVz6TzE/qZUJnx2UWjQVMrcRL8nzAXy4wCR1WwQj6ySLSgHdRvhXA/kZH8VzAu1bgUw 4/mImkDsr6kt007Wi5G9BSQSTApsi30lXIEDk9tOV+lDpP8n6WOh+kGL2fyMaTjeGF+7U6V3wZU PxDka77AXMT2R77PO+qaDOXaMWYbY8we4zAZlGadEPCEu6zaVxKBj3AnyDIo/qKKqcut8B4qHls e5kFKKk9yUtHXW5pXByQX52f/n+VfsKXGRG/+VrUSGYET4MHZgkq1zOoBAWG/XuHwQ21cpCXPWQ KTsiO7UrdXklgVrYeGho0mNv56EWDUhQcaxwZgLW1zFZl11C/b2ig3YPRq8ZnE4YkhOpvLg7Dib HkOVXNcXo2B14c4Vtp2ACJNgGF8S6Pb+zUVbL5RRtdGFPX0Ezz0A+uIfUyXzonD4lUJeef7qZ27 n5zcZ5ixk40mZN/CFep0sCBE2bLMLIWfPWUY1DoMht4pc5vdG7Af1Y8NTu4SVRhUVTECtgng9GM YaC9tWXSxFyv2lPIVrT5XTLYVTGBbD5NM1wKBnsILEbA7fZ9ngQChR9X8wMRWqx6TnKJjIrQy1o 9vge8iMh9vE2COr5V44yR43Unxi2wNSvCcL1qSFUvW8zFg9ymApQP1hDa0VapqNeKR3mBCjXyPj dClWf8VALMVezuQ== 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 Reviewed-by: Kieran Bingham Reviewed-by: Stefan Klug --- 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 b93063bf8bc7..fb6f959e5a09 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_; };