From 916e5c5ddf02c7d0346e830413ae4c7dbca0c7a4 Mon Sep 17 00:00:00 2001
From: Lightos1 <124387232+Lightos1@users.noreply.github.com>
Date: Fri, 8 May 2026 18:10:32 +0200
Subject: [PATCH] add stable max index checks

---
 .../common/include/hocclk/clock_manager.h     |  4 +-
 .../hoc-clk/sysmodule/src/clock_manager.cpp   | 45 ++++++++++++++-----
 2 files changed, 37 insertions(+), 12 deletions(-)

diff --git a/Source/hoc-clk/common/include/hocclk/clock_manager.h b/Source/hoc-clk/common/include/hocclk/clock_manager.h
index bc167a41..717a6668 100644
--- a/Source/hoc-clk/common/include/hocclk/clock_manager.h
+++ b/Source/hoc-clk/common/include/hocclk/clock_manager.h
@@ -36,7 +36,7 @@ typedef struct {
         #define HocClkModuleStable_EnumMax 5
         #define HocClkThermalSensorStable_EnumMax 11
         #define HocClkPowerSensorStable_EnumMax 2
-        #define HocClkRamLoadStable_EnumMax 10
+        #define HocClkPartLoadStable_EnumMax 10
         #define HocClkVoltageStable_EnumMax 7
 
         u32 freqs[HocClkModuleStable_EnumMax];
@@ -44,7 +44,7 @@ typedef struct {
         u32 overrideFreqs[HocClkModuleStable_EnumMax];
         s32 temps[HocClkThermalSensorStable_EnumMax];
         u32 power[HocClkPowerSensorStable_EnumMax];
-        u32 partLoad[HocClkRamLoadStable_EnumMax];
+        u32 partLoad[HocClkPartLoadStable_EnumMax];
         u32 voltages[HocClkVoltageStable_EnumMax];
     } stable;
 
diff --git a/Source/hoc-clk/sysmodule/src/clock_manager.cpp b/Source/hoc-clk/sysmodule/src/clock_manager.cpp
index a94cf97c..25a256c2 100644
--- a/Source/hoc-clk/sysmodule/src/clock_manager.cpp
+++ b/Source/hoc-clk/sysmodule/src/clock_manager.cpp
@@ -469,7 +469,10 @@ namespace clockManager {
             if (hz != 0 && hz != gContext.freqs[module]) {
                 fileUtils::LogLine("[mgr] %s clock change: %u.%u MHz", board::GetModuleName((HocClkModule)module, true), hz / 1000000, hz / 100000 - hz / 1000000 * 10);
                 gContext.freqs[module] = hz;
-                gContext.stable.freqs[module] = hz;
+
+                if (module < HocClkModuleStable_EnumMax) {
+                    gContext.stable.freqs[module] = hz;
+                }
                 hasChanged = true;
             }
 
@@ -479,7 +482,10 @@ namespace clockManager {
                     fileUtils::LogLine("[mgr] %s override change: %u.%u MHz", board::GetModuleName((HocClkModule)module, true), hz / 1000000, hz / 100000 - hz / 1000000 * 10);
                 }
                 gContext.overrideFreqs[module] = hz;
-                gContext.stable.overrideFreqs[module] = hz;
+
+                if (module < HocClkModuleStable_EnumMax) {
+                    gContext.stable.overrideFreqs[module] = hz;
+                }
                 hasChanged = true;
             }
         }
@@ -495,7 +501,10 @@ namespace clockManager {
                 fileUtils::LogLine("[mgr] %s temp: %u.%u °C", board::GetThermalSensorName((HocClkThermalSensor)sensor, true), millis / 1000, (millis - millis / 1000 * 1000) / 100);
             }
             gContext.temps[sensor] = millis;
-            gContext.stable.temps[sensor] = millis;
+
+            if (sensor < HocClkThermalSensorStable_EnumMax) {
+                gContext.stable.temps[sensor] = millis;
+            }
         }
 
         // power stats do not and should not force a refresh, hasChanged untouched
@@ -507,7 +516,10 @@ namespace clockManager {
                 fileUtils::LogLine("[mgr] Power %s: %d mW", board::GetPowerSensorName((HocClkPowerSensor)sensor, false), mw);
             }
             gContext.power[sensor] = mw;
-            gContext.stable.power[sensor] = mw;
+
+            if (sensor < HocClkPowerSensorStable_EnumMax) {
+                gContext.stable.power[sensor] = mw;
+            }
         }
 
         // real freqs do not and should not force a refresh, hasChanged untouched
@@ -519,18 +531,27 @@ namespace clockManager {
                 fileUtils::LogLine("[mgr] %s real freq: %u.%u MHz", board::GetModuleName((HocClkModule)module, true), realHz / 1000000, realHz / 100000 - realHz / 1000000 * 10);
             }
             gContext.realFreqs[module] = realHz;
-            gContext.stable.realFreqs[module] = realHz;
+
+            if (module < HocClkModuleStable_EnumMax) {
+                gContext.stable.realFreqs[module] = realHz;
+            }
         }
 
         // ram load do not and should not force a refresh, hasChanged untouched
         for (unsigned int loadSource = 0; loadSource < HocClkPartLoad_EnumMax; loadSource++) {
             gContext.partLoad[loadSource] = board::GetPartLoad((HocClkPartLoad)loadSource);
-            gContext.stable.partLoad[loadSource] = board::GetPartLoad((HocClkPartLoad)loadSource);
+
+            if (loadSource < HocClkPartLoadStable_EnumMax) {
+                gContext.stable.partLoad[loadSource] = board::GetPartLoad((HocClkPartLoad)loadSource);
+            }
         }
 
         for (unsigned int voltageSource = 0; voltageSource < HocClkVoltage_EnumMax; voltageSource++) {
             gContext.voltages[voltageSource] = board::GetVoltage((HocClkVoltage)voltageSource);
-            gContext.stable.voltages[voltageSource] = board::GetVoltage((HocClkVoltage)voltageSource);
+
+            if (voltageSource < HocClkVoltageStable_EnumMax) {
+                gContext.stable.voltages[voltageSource] = board::GetVoltage((HocClkVoltage)voltageSource);
+            }
         }
 
         if (ConfigIntervalTimeout(HocClkConfigValue_CsvWriteIntervalMs, ns, &gLastCsvWriteNs)) {
@@ -570,9 +591,13 @@ namespace clockManager {
             gContext.freqs[module] = 0;
             gContext.realFreqs[module] = 0;
             gContext.overrideFreqs[module] = 0;
-            gContext.stable.freqs[module] = 0;
-            gContext.stable.realFreqs[module] = 0;
-            gContext.stable.overrideFreqs[module] = 0;
+
+            if (module < HocClkModuleStable_EnumMax) {
+                gContext.stable.freqs[module] = 0;
+                gContext.stable.realFreqs[module] = 0;
+                gContext.stable.overrideFreqs[module] = 0;
+            }
+
             RefreshFreqTableRow((HocClkModule)module);
         }