Skip to content

Conversation

@tmleman
Copy link
Contributor

@tmleman tmleman commented Jun 6, 2025

This pull request encompasses two significant changes:

  • Zephyr Update:
  • Transition to Default Power Management Policy:
    • The custom PM policy configuration (CONFIG_PM_POLICY_CUSTOM) has been removed from multiple Intel ADSP ACE board configurations, transitioning to Zephyr's default PM policy.

tmleman added 2 commits June 6, 2025 11:10
Total of 1263 commits.

Changes include:

* fbafada9b64 soc: intel_adsp: Manage power gating based on core
              activity
* 5a9394a1e07 llext:  add dynamic heap allocation support
* 3aace9520f1 llext: Do not print symbol resolution messages
* a8d44c37c49 pm: policy: remove redundant ifdef exclusion of
              device_power_lock
* b6429254914 pm: policy: fix mocks of
              CONFIG_PM_POLICY_DEVICE_CONSTRAINTS
* 437b56d22e8 pm: device_runtime: Make async optional
* 64e38bab556 pm: device_runtime: Allow to use a dedicated wq
* 1b230f87411 pm: policy: default: Removed unused code
* 042bf2c3fae kernel: userspace: Fix memory leak in
              dynamic_object_create
* 240b56008ce drivers: ssp: unify clock selection flow across ACE
              platforms

Signed-off-by: Tomasz Leman <tomasz.m.leman@intel.com>
This patch removes the `CONFIG_PM_POLICY_CUSTOM` configuration from
various Intel ADSP ACE board configurations, transitioning to the
default power management policy provided by Zephyr. Previously, a custom
PM policy was implemented to handle specific conditions that could not
be addressed by the default policy. However, recent updates to the
Zephyr kernel now encompass all necessary checks, allowing us to utilize
the default policy effectively.

The change affects the following board configurations:
- intel_adsp_ace15_mtpm.conf
- intel_adsp_ace20_lnl.conf
- intel_adsp_ace30_ptl.conf
- intel_adsp_ace30_ptl_sim.conf
- intel_adsp_ace30_wcl.conf
- intel_adsp_ace30_wcl_sim.conf

By adopting the default policy, we simplify the configuration and
maintenance of these board files while ensuring consistent power
management behavior across different platforms.

Signed-off-by: Tomasz Leman <tomasz.m.leman@intel.com>
Copilot AI review requested due to automatic review settings June 6, 2025 09:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Zephyr commit revision in west.yml and removes the custom power management policy from several Intel ADSP ACE board configuration files to transition to Zephyr's default PM policy.

  • Zephyr revision in west.yml updated to commit 8c6b2c8c88773810d64a47ed3b6206b7d6e8290f.
  • Removal of CONFIG_PM_POLICY_CUSTOM from multiple board configuration files in app/boards.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
west.yml Updated Zephyr commit revision to align with the upstream changes.
app/boards/intel_adsp_ace30_wcl_sim.conf Removed custom power management policy setting.
app/boards/intel_adsp_ace30_wcl.conf Removed custom power management policy setting.
app/boards/intel_adsp_ace30_ptl_sim.conf Removed custom power management policy setting.
app/boards/intel_adsp_ace30_ptl.conf Removed custom power management policy setting.
app/boards/intel_adsp_ace20_lnl.conf Removed custom power management policy setting.
app/boards/intel_adsp_ace15_mtpm.conf Removed custom power management policy setting.

@lgirdwood lgirdwood merged commit 97f869c into thesofproject:main Jun 9, 2025
43 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants