fix(pm): Use Zephyr created device slots.

* Avoid overwriting random memory by using iterable section created by Zephyr PM.
hypan599 · Mar 28, 2024 · 41d8180 · 41d8180
1 parent d0e0ecb
commit 41d8180
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 16 deletions.
diff --git a/app/CMakeLists.txt b/app/CMakeLists.txt
@@ -8,10 +8,6 @@ set(ZEPHYR_EXTRA_MODULES "${ZMK_EXTRA_MODULES};${CMAKE_CURRENT_SOURCE_DIR}/modul
 find_package(Zephyr REQUIRED HINTS ../zephyr)
 project(zmk)
 
-if(CONFIG_ZMK_PM_DEVICE_SUSPEND_RESUME)
-  zephyr_linker_sources(SECTIONS include/linker/zmk-pm-devices.ld)
-endif()
-
 zephyr_linker_sources(SECTIONS include/linker/zmk-behaviors.ld)
 zephyr_linker_sources(RODATA include/linker/zmk-events.ld)
 

diff --git a/app/include/linker/zmk-pm-devices.ld b/app/include/linker/zmk-pm-devices.ld
diff --git a/app/src/pm.c b/app/src/pm.c
@@ -21,7 +21,7 @@ LOG_MODULE_DECLARE(zmk, CONFIG_ZMK_LOG_LEVEL);
 // TODO: Tweak this to smarter runtime PM of subsystems on sleep.
 
 #ifdef CONFIG_ZMK_PM_DEVICE_SUSPEND_RESUME
-TYPE_SECTION_START_EXTERN(const struct device *, zmk_pm_device_slots);
+TYPE_SECTION_START_EXTERN(const struct device *, pm_device_slots);
 
 #if !defined(CONFIG_PM_DEVICE_RUNTIME_EXCLUSIVE)
 /* Number of devices successfully suspended. */
@@ -57,7 +57,7 @@ int zmk_pm_suspend_devices(void) {
             return ret;
         }
 
-        TYPE_SECTION_START(zmk_pm_device_slots)[zmk_num_susp] = dev;
+        TYPE_SECTION_START(pm_device_slots)[zmk_num_susp] = dev;
         zmk_num_susp++;
     }
 
@@ -66,7 +66,7 @@ int zmk_pm_suspend_devices(void) {
 
 void zmk_pm_resume_devices(void) {
     for (int i = (zmk_num_susp - 1); i >= 0; i--) {
-        pm_device_action_run(TYPE_SECTION_START(zmk_pm_device_slots)[i], PM_DEVICE_ACTION_RESUME);
+        pm_device_action_run(TYPE_SECTION_START(pm_device_slots)[i], PM_DEVICE_ACTION_RESUME);
     }
 
     zmk_num_susp = 0;