boards: ace: power: disable unused asynchronous runtime PM#10148
Conversation
|
@tmleman is this because we never use |
This patch removes the `CONFIG_PM_DEVICE_RUNTIME_ASYNC` configuration from the Intel ADSP board files, including ACE15 MTPM, ACE20 LNL, ACE30 PTL, ACE30 WCL ACE40 NVL and ACE40 NVLS. The asynchronous runtime power management feature is not utilized by any device drivers in our project, rendering it unnecessary. By excluding this configuration, we prevent the compilation of dead code related to power management, optimizing the build process and reducing potential overhead. Signed-off-by: Tomasz Leman <tomasz.m.leman@intel.com>
e6adeb2 to
6ab073a
Compare
Yes. |
There was a problem hiding this comment.
Pull Request Overview
This PR disables unused asynchronous runtime power management across Intel ADSP board configurations to optimize build size and prevent compilation of dead code. The change systematically adds CONFIG_PM_DEVICE_RUNTIME_ASYNC=n to all Intel ADSP ACE board configuration files since no device drivers in the project utilize this feature.
- Adds explicit disabling of asynchronous runtime PM configuration
- Targets all Intel ADSP ACE board variants (ACE15, ACE20, ACE30, ACE40)
- Optimizes build process by excluding unused power management code
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| app/boards/intel_adsp_ace15_mtpm.conf | Disables async runtime PM for ACE15 MTPM board |
| app/boards/intel_adsp_ace20_lnl.conf | Disables async runtime PM for ACE20 LNL board |
| app/boards/intel_adsp_ace30_ptl.conf | Disables async runtime PM for ACE30 PTL board |
| app/boards/intel_adsp_ace30_wcl.conf | Disables async runtime PM for ACE30 WCL board |
| app/boards/intel_adsp_ace40_nvl.conf | Disables async runtime PM for ACE40 NVL board |
| app/boards/intel_adsp_ace40_nvls.conf | Disables async runtime PM for ACE40 NVLS board |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
All code is in Zephyr. |
kv2019i
left a comment
There was a problem hiding this comment.
Ack, looks good! We can't disable configuration options via the defconfig system, so this setting must be done for each board file separately (so this PR is good).
@lgirdwood there's nothing to remove on the SOF side. The code is in Zephyr side. The feature is enabled by default, and we don't use it. |
|
SOFCI TEST |
This patch removes the
CONFIG_PM_DEVICE_RUNTIME_ASYNCconfiguration from the Intel ADSP board files, including ACE15 MTPM, ACE20 LNL, ACE30 PTL, ACE30 WCL ACE40 NVL and ACE40 NVLS.The asynchronous runtime power management feature is not utilized by any device drivers in our project, rendering it unnecessary. By excluding this configuration, we prevent the compilation of dead code related to power management, optimizing the build process and reducing potential overhead.