Patch Detail
Show a patch.
GET /api/patches/16763/?format=api
{ "id": 16763, "url": "https://patchwork.libcamera.org/api/patches/16763/?format=api", "web_url": "https://patchwork.libcamera.org/patch/16763/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/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": "<20220725134639.4572-11-naush@raspberrypi.com>", "date": "2022-07-25T13:46:34", "name": "[libcamera-devel,10/15] DNI: ipa: raspberrypi: Code refactoring to match style guidelines", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "54ebf5d3b1ac06a9b5e1c0e08b9aba9c748f47c6", "submitter": { "id": 34, "url": "https://patchwork.libcamera.org/api/people/34/?format=api", "name": "Naushir Patuck", "email": "naush@raspberrypi.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/16763/mbox/", "series": [ { "id": 3323, "url": "https://patchwork.libcamera.org/api/series/3323/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=3323", "date": "2022-07-25T13:46:24", "name": "Raspberry Pi IPA code refactor", "version": 1, "mbox": "https://patchwork.libcamera.org/series/3323/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/16763/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/16763/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 AAF73C3275\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 25 Jul 2022 13:46:59 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 5FC1B63325;\n\tMon, 25 Jul 2022 15:46:59 +0200 (CEST)", "from mail-wm1-x330.google.com (mail-wm1-x330.google.com\n\t[IPv6:2a00:1450:4864:20::330])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C66E26331B\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 25 Jul 2022 15:46:51 +0200 (CEST)", "by mail-wm1-x330.google.com with SMTP id\n\tj29-20020a05600c1c1d00b003a2fdafdefbso6395927wms.2\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 25 Jul 2022 06:46:51 -0700 (PDT)", "from naush-laptop.localdomain ([93.93.133.154])\n\tby smtp.gmail.com with ESMTPSA id\n\ta20-20020a05600c225400b003a32167b8d4sm18054320wmm.13.2022.07.25.06.46.50\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tMon, 25 Jul 2022 06:46:50 -0700 (PDT)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1658756819;\n\tbh=CCyGgB+YxhvG4xAkPWe5sx9cDycCqcY3/2JbgIMiuo4=;\n\th=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=O8jVfxKdtUVwuZ79Ec80JygAOpIIO7Oc1mtDlpNPdTQgVoPAbfYUvBtwkYUXn4OG0\n\tq6iXRDowxn0f64vpt04Tpx0Y7Gpar0svFZcY09C/MNvom8dNwz72GMFOfUxUMnli16\n\tgUZByMS4r7DbGZz6wkniDkSNFCVEL4y9ZNHmKCsRY1h0yYPjXJZlK9sYdLYYQHoiew\n\t/DAv4k46nlzVY5jl85R8jTlQI9BdnnrTwrf9vMIM9YnVNJBtJIjWKfDPdA50/2r9xF\n\ttsDtIKJTj+5x386OJ18YJapH3TmJJckwDpuSIQFx0x+3l/CnAvabegEMEi/gf3otJS\n\tVt8y7URFPB9HA==", "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references\n\t:mime-version:content-transfer-encoding;\n\tbh=CS8z/a4Vg/9QpxoeDhdG8m9mp8Ky5pivjVp9+vV5WRA=;\n\tb=SryIVCPr5ZzljWn17ZfvYfHZ4wFX9FBj121jyh9qKJB6CrQZ8p9MVGa+Tocg0zzqY9\n\t/vqfc/dgMLZfiXc3/imfIyp/xUTjsNomqb1dMczpPLGD+JxGIImKjN5PxrU4tINEH8VY\n\tijEmhZt6ky9sCx4F8jMcD4AjGdpAOHoS3z7qUSRcgJ9guRVjndCsAT77oIV14LnqqKc6\n\tYgE8w4dud2bRVcY3EjrB67zBQc3t8zP3BB/VTpFSVONADQW2V9tWz8C+XT66Na8pOYzG\n\t1rBecu06n59poXmRy5gGYY5YTsSwAe+iEf4aDIgbXuVmk5v6j1Ix9eg7p3JI91mMh5U6\n\tEK7w==" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=raspberrypi.com\n\theader.i=@raspberrypi.com\n\theader.b=\"SryIVCPr\"; dkim-atps=neutral", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references:mime-version:content-transfer-encoding;\n\tbh=CS8z/a4Vg/9QpxoeDhdG8m9mp8Ky5pivjVp9+vV5WRA=;\n\tb=XxzI7as2uWcCsrruf7Jos29FXTZHBbgOhvmZMSxXbIWw87/1RxV1DM8gwtaZpoz9V4\n\tXytwTxb1oxgnKz8kYGvzyyf4lBqdB3AIPmwCpnmNHhCJPZ7Adi7HrqNSa33DVrNQN9Tg\n\tySA81AW0IeqUSznDYJ/07hshoSHIDmt+xZJreEcpkgXYevK9tiuzkEN7igTcP1agK1lp\n\tBfQiY0ph8jUh8XOj+NJ4TC59VuZwXbkDXI3V6i/YM2Mt9kuow2gCqHET6WAy7HUtvNvm\n\tBdoAuN8X5HPxvsu3PbmTvi1hrBk/lnS8gpV5iihR2L8ji/xU6FoIBv+lHiJr04N4EU8C\n\tgmYQ==", "X-Gm-Message-State": "AJIora86fxLZHicgzK3Jm610QO1N+mb7lElpVucbc2J0Vzt8+yWF/CkT\n\tdxeS4dpcgcfz/Gv1rP4Q+F0N2evcE41tdw==", "X-Google-Smtp-Source": "AGRyM1v9AFOkK+3CSsrGWVLMvQYZfCKqoT2HKvLi0C2yCVfFifm87Zq6Y5bESuyGRKxTbERIAEK1lA==", "X-Received": "by 2002:a7b:c442:0:b0:3a3:bdb:e84c with SMTP id\n\tl2-20020a7bc442000000b003a30bdbe84cmr8463244wmi.101.1658756811140; \n\tMon, 25 Jul 2022 06:46:51 -0700 (PDT)", "To": "libcamera-devel@lists.libcamera.org", "Date": "Mon, 25 Jul 2022 14:46:34 +0100", "Message-Id": "<20220725134639.4572-11-naush@raspberrypi.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20220725134639.4572-1-naush@raspberrypi.com>", "References": "<20220725134639.4572-1-naush@raspberrypi.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH 10/15] DNI: ipa: raspberrypi: Code\n\trefactoring to match style guidelines", "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>", "From": "Naushir Patuck via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>", "Reply-To": "Naushir Patuck <naush@raspberrypi.com>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Refactor the source files src/ipa/raspberrypi/controller/rps/[n|s]* to match the\nrecommended formatting guidelines for the libcamera project. The vast majority\nof changes in this commit comprise of switching from snake_case to CamelCase,\nand starting class member functions with a lower case character.\n\nSigned-off-by: Naushir Patuck <naush@raspberrypi.com>\n---\n src/ipa/raspberrypi/controller/rpi/noise.cpp | 34 ++++++++---------\n src/ipa/raspberrypi/controller/rpi/noise.hpp | 14 +++----\n src/ipa/raspberrypi/controller/rpi/sdn.cpp | 32 ++++++++--------\n src/ipa/raspberrypi/controller/rpi/sdn.hpp | 10 ++---\n .../raspberrypi/controller/rpi/sharpen.cpp | 38 +++++++++----------\n .../raspberrypi/controller/rpi/sharpen.hpp | 14 +++----\n 6 files changed, 72 insertions(+), 70 deletions(-)", "diff": "diff --git a/src/ipa/raspberrypi/controller/rpi/noise.cpp b/src/ipa/raspberrypi/controller/rpi/noise.cpp\nindex 63cad639f313..d6e4df4192f2 100644\n--- a/src/ipa/raspberrypi/controller/rpi/noise.cpp\n+++ b/src/ipa/raspberrypi/controller/rpi/noise.cpp\n@@ -22,45 +22,45 @@ LOG_DEFINE_CATEGORY(RPiNoise)\n #define NAME \"rpi.noise\"\n \n Noise::Noise(Controller *controller)\n-\t: Algorithm(controller), mode_factor_(1.0)\n+\t: Algorithm(controller), modeFactor_(1.0)\n {\n }\n \n-char const *Noise::Name() const\n+char const *Noise::name() const\n {\n \treturn NAME;\n }\n \n-void Noise::SwitchMode(CameraMode const &camera_mode,\n+void Noise::switchMode(CameraMode const &cameraMode,\n \t\t [[maybe_unused]] Metadata *metadata)\n {\n \t// For example, we would expect a 2x2 binned mode to have a \"noise\n \t// factor\" of sqrt(2x2) = 2. (can't be less than one, right?)\n-\tmode_factor_ = std::max(1.0, camera_mode.noise_factor);\n+\tmodeFactor_ = std::max(1.0, cameraMode.noiseFactor);\n }\n \n-void Noise::Read(boost::property_tree::ptree const ¶ms)\n+void Noise::read(boost::property_tree::ptree const ¶ms)\n {\n-\treference_constant_ = params.get<double>(\"reference_constant\");\n-\treference_slope_ = params.get<double>(\"reference_slope\");\n+\treferenceConstant_ = params.get<double>(\"reference_constant\");\n+\treferenceSlope_ = params.get<double>(\"reference_slope\");\n }\n \n-void Noise::Prepare(Metadata *image_metadata)\n+void Noise::prepare(Metadata *imageMetadata)\n {\n-\tstruct DeviceStatus device_status;\n-\tdevice_status.analogue_gain = 1.0; // keep compiler calm\n-\tif (image_metadata->Get(\"device.status\", device_status) == 0) {\n+\tstruct DeviceStatus deviceStatus;\n+\tdeviceStatus.analogueGain = 1.0; // keep compiler calm\n+\tif (imageMetadata->get(\"device.status\", deviceStatus) == 0) {\n \t\t// There is a slight question as to exactly how the noise\n \t\t// profile, specifically the constant part of it, scales. For\n \t\t// now we assume it all scales the same, and we'll revisit this\n \t\t// if it proves substantially wrong. NOTE: we may also want to\n \t\t// make some adjustments based on the camera mode (such as\n \t\t// binning), if we knew how to discover it...\n-\t\tdouble factor = sqrt(device_status.analogue_gain) / mode_factor_;\n+\t\tdouble factor = sqrt(deviceStatus.analogueGain) / modeFactor_;\n \t\tstruct NoiseStatus status;\n-\t\tstatus.noise_constant = reference_constant_ * factor;\n-\t\tstatus.noise_slope = reference_slope_ * factor;\n-\t\timage_metadata->Set(\"noise.status\", status);\n+\t\tstatus.noise_constant = referenceConstant_ * factor;\n+\t\tstatus.noise_slope = referenceSlope_ * factor;\n+\t\timageMetadata->set(\"noise.status\", status);\n \t\tLOG(RPiNoise, Debug)\n \t\t\t<< \"constant \" << status.noise_constant\n \t\t\t<< \" slope \" << status.noise_slope;\n@@ -69,8 +69,8 @@ void Noise::Prepare(Metadata *image_metadata)\n }\n \n // Register algorithm with the system.\n-static Algorithm *Create(Controller *controller)\n+static Algorithm *create(Controller *controller)\n {\n \treturn new Noise(controller);\n }\n-static RegisterAlgorithm reg(NAME, &Create);\n+static RegisterAlgorithm reg(NAME, &create);\ndiff --git a/src/ipa/raspberrypi/controller/rpi/noise.hpp b/src/ipa/raspberrypi/controller/rpi/noise.hpp\nindex 1c9de5c87d08..ed6ffe910e27 100644\n--- a/src/ipa/raspberrypi/controller/rpi/noise.hpp\n+++ b/src/ipa/raspberrypi/controller/rpi/noise.hpp\n@@ -17,16 +17,16 @@ class Noise : public Algorithm\n {\n public:\n \tNoise(Controller *controller);\n-\tchar const *Name() const override;\n-\tvoid SwitchMode(CameraMode const &camera_mode, Metadata *metadata) override;\n-\tvoid Read(boost::property_tree::ptree const ¶ms) override;\n-\tvoid Prepare(Metadata *image_metadata) override;\n+\tchar const *name() const override;\n+\tvoid switchMode(CameraMode const &cameraMode, Metadata *metadata) override;\n+\tvoid read(boost::property_tree::ptree const ¶ms) override;\n+\tvoid prepare(Metadata *imageMetadata) override;\n \n private:\n \t// the noise profile for analogue gain of 1.0\n-\tdouble reference_constant_;\n-\tdouble reference_slope_;\n-\tdouble mode_factor_;\n+\tdouble referenceConstant_;\n+\tdouble referenceSlope_;\n+\tdouble modeFactor_;\n };\n \n } // namespace RPiController\ndiff --git a/src/ipa/raspberrypi/controller/rpi/sdn.cpp b/src/ipa/raspberrypi/controller/rpi/sdn.cpp\nindex 9384550983e7..8707b6d9cd9e 100644\n--- a/src/ipa/raspberrypi/controller/rpi/sdn.cpp\n+++ b/src/ipa/raspberrypi/controller/rpi/sdn.cpp\n@@ -27,49 +27,51 @@ Sdn::Sdn(Controller *controller)\n {\n }\n \n-char const *Sdn::Name() const\n+char const *Sdn::name() const\n {\n \treturn NAME;\n }\n \n-void Sdn::Read(boost::property_tree::ptree const ¶ms)\n+void Sdn::read(boost::property_tree::ptree const ¶ms)\n {\n \tdeviation_ = params.get<double>(\"deviation\", 3.2);\n \tstrength_ = params.get<double>(\"strength\", 0.75);\n }\n \n-void Sdn::Initialise() {}\n+void Sdn::initialise()\n+{\n+}\n \n-void Sdn::Prepare(Metadata *image_metadata)\n+void Sdn::prepare(Metadata *imageMetadata)\n {\n-\tstruct NoiseStatus noise_status = {};\n-\tnoise_status.noise_slope = 3.0; // in case no metadata\n-\tif (image_metadata->Get(\"noise.status\", noise_status) != 0)\n+\tstruct NoiseStatus noiseStatus = {};\n+\tnoiseStatus.noise_slope = 3.0; // in case no metadata\n+\tif (imageMetadata->get(\"noise.status\", noiseStatus) != 0)\n \t\tLOG(RPiSdn, Warning) << \"no noise profile found\";\n \tLOG(RPiSdn, Debug)\n-\t\t<< \"Noise profile: constant \" << noise_status.noise_constant\n-\t\t<< \" slope \" << noise_status.noise_slope;\n+\t\t<< \"Noise profile: constant \" << noiseStatus.noise_constant\n+\t\t<< \" slope \" << noiseStatus.noise_slope;\n \tstruct DenoiseStatus status;\n-\tstatus.noise_constant = noise_status.noise_constant * deviation_;\n-\tstatus.noise_slope = noise_status.noise_slope * deviation_;\n+\tstatus.noise_constant = noiseStatus.noise_constant * deviation_;\n+\tstatus.noise_slope = noiseStatus.noise_slope * deviation_;\n \tstatus.strength = strength_;\n \tstatus.mode = static_cast<std::underlying_type_t<DenoiseMode>>(mode_);\n-\timage_metadata->Set(\"denoise.status\", status);\n+\timageMetadata->set(\"denoise.status\", status);\n \tLOG(RPiSdn, Debug)\n \t\t<< \"programmed constant \" << status.noise_constant\n \t\t<< \" slope \" << status.noise_slope\n \t\t<< \" strength \" << status.strength;\n }\n \n-void Sdn::SetMode(DenoiseMode mode)\n+void Sdn::setMode(DenoiseMode mode)\n {\n \t// We only distinguish between off and all other modes.\n \tmode_ = mode;\n }\n \n // Register algorithm with the system.\n-static Algorithm *Create(Controller *controller)\n+static Algorithm *create(Controller *controller)\n {\n \treturn (Algorithm *)new Sdn(controller);\n }\n-static RegisterAlgorithm reg(NAME, &Create);\n+static RegisterAlgorithm reg(NAME, &create);\ndiff --git a/src/ipa/raspberrypi/controller/rpi/sdn.hpp b/src/ipa/raspberrypi/controller/rpi/sdn.hpp\nindex 2371ce04163f..d9b18f296635 100644\n--- a/src/ipa/raspberrypi/controller/rpi/sdn.hpp\n+++ b/src/ipa/raspberrypi/controller/rpi/sdn.hpp\n@@ -17,11 +17,11 @@ class Sdn : public DenoiseAlgorithm\n {\n public:\n \tSdn(Controller *controller = NULL);\n-\tchar const *Name() const override;\n-\tvoid Read(boost::property_tree::ptree const ¶ms) override;\n-\tvoid Initialise() override;\n-\tvoid Prepare(Metadata *image_metadata) override;\n-\tvoid SetMode(DenoiseMode mode) override;\n+\tchar const *name() const override;\n+\tvoid read(boost::property_tree::ptree const ¶ms) override;\n+\tvoid initialise() override;\n+\tvoid prepare(Metadata *imageMetadata) override;\n+\tvoid setMode(DenoiseMode mode) override;\n \n private:\n \tdouble deviation_;\ndiff --git a/src/ipa/raspberrypi/controller/rpi/sharpen.cpp b/src/ipa/raspberrypi/controller/rpi/sharpen.cpp\nindex 18825a43867b..775ed0fd2c46 100644\n--- a/src/ipa/raspberrypi/controller/rpi/sharpen.cpp\n+++ b/src/ipa/raspberrypi/controller/rpi/sharpen.cpp\n@@ -21,23 +21,23 @@ LOG_DEFINE_CATEGORY(RPiSharpen)\n #define NAME \"rpi.sharpen\"\n \n Sharpen::Sharpen(Controller *controller)\n-\t: SharpenAlgorithm(controller), user_strength_(1.0)\n+\t: SharpenAlgorithm(controller), userStrength_(1.0)\n {\n }\n \n-char const *Sharpen::Name() const\n+char const *Sharpen::name() const\n {\n \treturn NAME;\n }\n \n-void Sharpen::SwitchMode(CameraMode const &camera_mode,\n+void Sharpen::switchMode(CameraMode const &cameraMode,\n \t\t\t [[maybe_unused]] Metadata *metadata)\n {\n \t// can't be less than one, right?\n-\tmode_factor_ = std::max(1.0, camera_mode.noise_factor);\n+\tmodeFactor_ = std::max(1.0, cameraMode.noiseFactor);\n }\n \n-void Sharpen::Read(boost::property_tree::ptree const ¶ms)\n+void Sharpen::read(boost::property_tree::ptree const ¶ms)\n {\n \tthreshold_ = params.get<double>(\"threshold\", 1.0);\n \tstrength_ = params.get<double>(\"strength\", 1.0);\n@@ -48,38 +48,38 @@ void Sharpen::Read(boost::property_tree::ptree const ¶ms)\n \t\t<< \" limit \" << limit_;\n }\n \n-void Sharpen::SetStrength(double strength)\n+void Sharpen::setStrength(double strength)\n {\n \t// Note that this function is how an application sets the overall\n \t// sharpening \"strength\". We call this the \"user strength\" field\n \t// as there already is a strength_ field - being an internal gain\n \t// parameter that gets passed to the ISP control code. Negative\n \t// values are not allowed - coerce them to zero (no sharpening).\n-\tuser_strength_ = std::max(0.0, strength);\n+\tuserStrength_ = std::max(0.0, strength);\n }\n \n-void Sharpen::Prepare(Metadata *image_metadata)\n+void Sharpen::prepare(Metadata *imageMetadata)\n {\n \t// The user_strength_ affects the algorithm's internal gain directly, but\n \t// we adjust the limit and threshold less aggressively. Using a sqrt\n \t// function is an arbitrary but gentle way of accomplishing this.\n-\tdouble user_strength_sqrt = sqrt(user_strength_);\n+\tdouble userStrengthSqrt = sqrt(userStrength_);\n \tstruct SharpenStatus status;\n \t// Binned modes seem to need the sharpening toned down with this\n \t// pipeline, thus we use the mode_factor here. Also avoid\n-\t// divide-by-zero with the user_strength_sqrt.\n-\tstatus.threshold = threshold_ * mode_factor_ /\n-\t\t\t std::max(0.01, user_strength_sqrt);\n-\tstatus.strength = strength_ / mode_factor_ * user_strength_;\n-\tstatus.limit = limit_ / mode_factor_ * user_strength_sqrt;\n-\t// Finally, report any application-supplied parameters that were used.\n-\tstatus.user_strength = user_strength_;\n-\timage_metadata->Set(\"sharpen.status\", status);\n+\t// divide-by-zero with the userStrengthSqrt.\n+\tstatus.threshold = threshold_ * modeFactor_ /\n+\t\t\t std::max(0.01, userStrengthSqrt);\n+\tstatus.strength = strength_ / modeFactor_ * userStrength_;\n+\tstatus.limit = limit_ / modeFactor_ * userStrengthSqrt;\n+\t/* Finally, report any application-supplied parameters that were used. */\n+\tstatus.userStrength = userStrength_;\n+\timageMetadata->set(\"sharpen.status\", status);\n }\n \n // Register algorithm with the system.\n-static Algorithm *Create(Controller *controller)\n+static Algorithm *create(Controller *controller)\n {\n \treturn new Sharpen(controller);\n }\n-static RegisterAlgorithm reg(NAME, &Create);\n+static RegisterAlgorithm reg(NAME, &create);\ndiff --git a/src/ipa/raspberrypi/controller/rpi/sharpen.hpp b/src/ipa/raspberrypi/controller/rpi/sharpen.hpp\nindex 13a076a86895..ced917f3c42b 100644\n--- a/src/ipa/raspberrypi/controller/rpi/sharpen.hpp\n+++ b/src/ipa/raspberrypi/controller/rpi/sharpen.hpp\n@@ -17,18 +17,18 @@ class Sharpen : public SharpenAlgorithm\n {\n public:\n \tSharpen(Controller *controller);\n-\tchar const *Name() const override;\n-\tvoid SwitchMode(CameraMode const &camera_mode, Metadata *metadata) override;\n-\tvoid Read(boost::property_tree::ptree const ¶ms) override;\n-\tvoid SetStrength(double strength) override;\n-\tvoid Prepare(Metadata *image_metadata) override;\n+\tchar const *name() const override;\n+\tvoid switchMode(CameraMode const &cameraMode, Metadata *metadata) override;\n+\tvoid read(boost::property_tree::ptree const ¶ms) override;\n+\tvoid setStrength(double strength) override;\n+\tvoid prepare(Metadata *imageMetadata) override;\n \n private:\n \tdouble threshold_;\n \tdouble strength_;\n \tdouble limit_;\n-\tdouble mode_factor_;\n-\tdouble user_strength_;\n+\tdouble modeFactor_;\n+\tdouble userStrength_;\n };\n \n } // namespace RPiController\n", "prefixes": [ "libcamera-devel", "10/15" ] }