rg_display: Track the number of spi transfers

This is a test, I don't think it's worth adding to the main branch. But maybe...
ducalex · Mar 15, 2024 · c742cae · c742cae
1 parent e4ee580
commit c742cae
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 2 deletions.
diff --git a/components/retro-go/rg_display.c b/components/retro-go/rg_display.c
@@ -86,6 +86,7 @@ static inline void spi_queue_transaction(const void *data, size_t length, uint32
     {
         RG_PANIC("display");
     }
+    counters.spiTransfers++;
 }
 
 IRAM_ATTR

diff --git a/components/retro-go/rg_display.h b/components/retro-go/rg_display.h
@@ -59,6 +59,7 @@ typedef struct
     int32_t partFrames;
     int64_t blockTime;
     int64_t busyTime;
+    int64_t spiTransfers;
 } rg_display_counters_t;
 
 typedef struct

diff --git a/components/retro-go/rg_system.c b/components/retro-go/rg_system.c
@@ -39,6 +39,7 @@ typedef struct
 {
     int32_t totalFrames, fullFrames, partFrames, ticks;
     int64_t busyTime, updateTime;
+    int64_t spiTransfers;
 } counters_t;
 
 typedef struct
@@ -140,6 +141,7 @@ static void update_statistics(void)
     counters.busyTime = statistics.busyTime;
     counters.ticks = statistics.ticks;
     counters.updateTime = statistics.lastTick;
+    counters.spiTransfers = display.spiTransfers;
 
     // We prefer to use the tick time for more accurate FPS
     // but if we're not ticking, we need to use current time
@@ -164,6 +166,7 @@ static void update_statistics(void)
         statistics.skippedFPS = (ticks - frames) / totalTimeSecs;
         statistics.fullFPS = fullFrames / totalTimeSecs;
         statistics.partialFPS = partFrames / totalTimeSecs;
+        statistics.spiTransfers = counters.spiTransfers - previous.spiTransfers;
     }
     statistics.uptime = rg_system_timer() / 1000000;
 
@@ -194,7 +197,7 @@ static void system_monitor_task(void *arg)
         }
 
         // Try to avoid complex conversions that could allocate, prefer rounding/ceiling if necessary.
-        rg_system_log(RG_LOG_DEBUG, NULL, "STACK:%d, HEAP:%d+%d (%d+%d), BUSY:%d%%, FPS:%d (%d+%d+%d), BATT:%d\n",
+        rg_system_log(RG_LOG_DEBUG, NULL, "STACK:%d, HEAP:%d+%d (%d+%d), BUSY:%d%%, FPS:%d (%d+%d+%d), BATT:%d SPI:%d\n",
             statistics.freeStackMain,
             statistics.freeMemoryInt / 1024,
             statistics.freeMemoryExt / 1024,
@@ -205,7 +208,8 @@ static void system_monitor_task(void *arg)
             (int)roundf(statistics.skippedFPS),
             (int)roundf(statistics.partialFPS),
             (int)roundf(statistics.fullFPS),
-            (int)roundf((battery.volts * 1000) ?: battery.level));
+            (int)roundf((battery.volts * 1000) ?: battery.level),
+            statistics.spiTransfers);
 
         // Auto frameskip
         if (statistics.ticks > app.tickRate * 2)

diff --git a/components/retro-go/rg_system.h b/components/retro-go/rg_system.h
@@ -196,6 +196,7 @@ typedef struct
     int64_t lastTick;
     int ticks;
     int uptime;
+    int spiTransfers;
     int totalMemoryInt;
     int totalMemoryExt;
     int freeMemoryInt;