Skip to content

Conversation

@abonislawski
Copy link
Member

Enable CONFIG_COLD_STORE_EXECUTE_DRAM for ACE 4.0

Copilot AI review requested due to automatic review settings September 25, 2025 06:54
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 enables cold store DRAM execution for the Intel ADSP ACE 4.0 NVL board by adding the CONFIG_COLD_STORE_EXECUTE_DRAM configuration option.

  • Adds CONFIG_COLD_STORE_EXECUTE_DRAM=y to the board configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@kv2019i
Copy link
Collaborator

kv2019i commented Sep 25, 2025

Btw, this will potentially conflict with #10236 that has been waiting for the 2.14 stable branch (not needed for that). But I can rebase. I have part3 coming up after #10236 ...

@abonislawski
Copy link
Member Author

No worries, I can rebase it after 10236

@abonislawski abonislawski force-pushed the nvl_cold_store_execution branch from 6e3f80c to d29bdd7 Compare September 25, 2025 09:14
Copy link
Collaborator

@softwarecki softwarecki left a comment

Choose a reason for hiding this comment

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

LGTM

@kv2019i kv2019i requested a review from lyakh September 25, 2025 16:41
@kv2019i
Copy link
Collaborator

kv2019i commented Sep 26, 2025

@abonislawski This had QB fails so I went with #10236 first.

@lgirdwood
Copy link
Member

@abonislawski some conflicts.

@kv2019i
Copy link
Collaborator

kv2019i commented Sep 30, 2025

@abonislawski The common board file changes are now done for Intel DSPs, so safe to rebase now.

Required changes to enable CONFIG_COLD_STORE_EXECUTE_DRAM for ACE 4.0

nvl.toml.h -> add COLD module.entry
novalake memory.h -> add __cold define support

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
Enable CONFIG_COLD_STORE_EXECUTE_DRAM for ACE 4.0

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
@abonislawski abonislawski force-pushed the nvl_cold_store_execution branch from d29bdd7 to 7f3ee6f Compare October 2, 2025 06:55
@abonislawski
Copy link
Member Author

Rebased

@kv2019i kv2019i merged commit 771d674 into thesofproject:main Oct 3, 2025
35 of 47 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.

6 participants