SleepTracing: don't spam logs if no lock is held

facchinm · facchinm · commit c18ecc16486a · 2020-12-11T10:29:04.000+01:00
diff --git a/platform/source/mbed_power_mgmt.c b/platform/source/mbed_power_mgmt.c
@@ -138,7 +138,7 @@ static sleep_statistic_t *sleep_tracker_add(const char *const filename)
 
 static void sleep_tracker_print_stats(void)
 {
-    mbed_error_printf("Sleep locks held:\r\n");
+    bool once = true;
     for (int i = 0; i < STATISTIC_COUNT; ++i) {
         if (sleep_stats[i].count == 0) {
             continue;
@@ -148,6 +148,11 @@ static void sleep_tracker_print_stats(void)
             return;
         }
 
+        if (once) {
+            mbed_error_printf("Sleep locks held:\r\n");
+            once = false;
+        }
+
         mbed_error_printf("[id: %s, count: %u]\r\n", sleep_stats[i].identifier,
                           sleep_stats[i].count);
     }

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ static sleep_statistic_t sleep_tracker_add(const char const filename)`
`138`	`138`
`139`	`139`	`static void sleep_tracker_print_stats(void)`
`140`	`140`	`{`
`141`		`- mbed_error_printf("Sleep locks held:\r\n");`
	`141`	`+ bool once = true;`
`142`	`142`	`for (int i = 0; i < STATISTIC_COUNT; ++i) {`
`143`	`143`	`if (sleep_stats[i].count == 0) {`
`144`	`144`	`continue;`
`@@ -148,6 +148,11 @@ static void sleep_tracker_print_stats(void)`
`148`	`148`	`return;`
`149`	`149`	`}`
`150`	`150`
	`151`	`+ if (once) {`
	`152`	`+ mbed_error_printf("Sleep locks held:\r\n");`
	`153`	`+ once = false;`
	`154`	`+ }`
	`155`	`+`
`151`	`156`	`mbed_error_printf("[id: %s, count: %u]\r\n", sleep_stats[i].identifier,`
`152`	`157`	`sleep_stats[i].count);`
`153`	`158`	`}`