What the developer wrote
Authored by kopecdav
chore(core): remove unused macro.
[no changelog]
What changed, and why it matters
This commit simply deletes one unused macro named PM_SUSPENDED_CHARGING_TIMEOUT_S from a header file in the Trezor firmware's power management code for the STM32U5 chip. A macro is a named constant used by programmers; 'unused' means no code referenced it anymore. There is no functional change, no bug fix, and no security relevance visible in the change.
No security action needed. This is a routine code-cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff removes a single line ‘#define PM_SUSPENDED_CHARGING_TIMEOUT_S 60’ from core/embed/sys/power_manager/stm32u5/power_manager_internal.h. The commit title and message explicitly state this is a chore to remove an unused macro, with no changelog entry. No code references this macro, so removing it does not alter compiled behavior or introduce/remove any security boundary.
Changed components
core/embed/sys/power_manager/stm32u5/power_manager_internal.hInspect captured patch +0 / −1
diff --git a/core/embed/sys/power_manager/stm32u5/power_manager_internal.h b/core/embed/sys/power_manager/stm32u5/power_manager_internal.h
index 9992d0046..bc56cd25b 100644
--- a/core/embed/sys/power_manager/stm32u5/power_manager_internal.h
+++ b/core/embed/sys/power_manager/stm32u5/power_manager_internal.h
@@ -52,7 +52,6 @@
// hibernation
#define PM_AUTO_HIBERNATE_TIMEOUT_S (2 * 60 * 60) // 2 hours
-#define PM_SUSPENDED_CHARGING_TIMEOUT_S 60
#define PM_STABILIZATION_TIMEOUT_MS 2000
// Temperature controller parameters
#define PM_TEMP_CONTROL_IDLE_PERIOD_MS 2 * 60 * 1000 // 2 minutes
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.