commit aba6fcac54aba04c8ce03d8aea7cdf30922f34d8
author Ed Tanous <etanous@nvidia.com> Mon Sep 29 13:53:20 2025 -0700
committer Ed Tanous <etanous@nvidia.com> Mon Sep 29 14:13:13 2025 -0700
tree d6f0c50292524a3b9738339a9f95c5af543f7d1d
parent 082ae0bb8b10533e668b466a0d624f3f2953525c

Fix tidy build

This appears to be something tidy is wrong about.  The suggestion of
adding math to the struct initializers appears to not compile.

Move the calculation of hysteresisTrigger and hysteresisPublish into the
constructor body itself to avoid the warning.

Change-Id: I833fd12966c69c0e081692d6d40ba0cf1805ead1
Signed-off-by: Ed Tanous <etanous@nvidia.com>

src/nvidia-gpu/NvidiaGpuPowerSensor.cpp

diff --git a/src/nvidia-gpu/NvidiaGpuPowerSensor.cpp b/src/nvidia-gpu/NvidiaGpuPowerSensor.cpp
index 756abe6..997ce3d 100644
--- a/src/nvidia-gpu/NvidiaGpuPowerSensor.cpp
+++ b/src/nvidia-gpu/NvidiaGpuPowerSensor.cpp
@@ -32,9 +32,6 @@
 
 using namespace std::literals;
 
-// GPU Power Sensor Averaging Interval in seconds, 0 implies default
-constexpr uint8_t gpuPowerAveragingIntervalInSec{0};
-
 static constexpr double gpuPowerSensorMaxReading = 5000;
 static constexpr double gpuPowerSensorMinReading =
     std::numeric_limits<uint32_t>::min();
@@ -49,7 +46,7 @@
            "power", false, true, gpuPowerSensorMaxReading,
            gpuPowerSensorMinReading, conn),
     eid(eid), sensorId{sensorId},
-    averagingInterval{gpuPowerAveragingIntervalInSec},
+
     mctpRequester(mctpRequester), objectServer(objectServer)
 
 {

src/nvidia-gpu/NvidiaGpuPowerSensor.hpp

diff --git a/src/nvidia-gpu/NvidiaGpuPowerSensor.hpp b/src/nvidia-gpu/NvidiaGpuPowerSensor.hpp
index 89b0ab4..7933f75 100644
--- a/src/nvidia-gpu/NvidiaGpuPowerSensor.hpp
+++ b/src/nvidia-gpu/NvidiaGpuPowerSensor.hpp
@@ -45,7 +45,7 @@
 
     uint8_t sensorId;
 
-    uint8_t averagingInterval;
+    uint8_t averagingInterval = 0;
 
     std::shared_ptr<sdbusplus::asio::connection> conn;