Skip to content

Conversation

@softwarecki
Copy link
Collaborator

Fix typo in variable name from heap_part_cached to heap_cached_part.

Fixes #10431

Fix typo in variable name from heap_part_cached to heap_cached_part.

Fixes thesofproject#10431

Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
Copilot AI review requested due to automatic review settings December 11, 2025 12:10
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 corrects a typo in a variable name within the userspace proxy module's memory initialization function. The variable reference is changed from heap_part_cached to heap_cached_part to match the actual variable name defined elsewhere in the codebase.

  • Fixed incorrect variable name reference in memory partition array initialization

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

#endif

struct k_mem_partition *parts_ptr[] = {
#ifdef HEAP_PART_CACHED
Copy link
Member

Choose a reason for hiding this comment

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

Can we align the Kconfig name in a subsequent PR? It seems like a copy paste type issue.

@kv2019i
Copy link
Collaborator

kv2019i commented Dec 11, 2025

Known issue seen https://sof-ci.01.org/sofpr/PR10432/build17976/devicetest/index.html?model=PTLP_RVP_SDW&testcase=verify-kernel-boot-log . One new in https://sof-ci.01.org/sofpr/PR10432/build17976/devicetest/index.html but this seems to be a DUT specific issue, not related to this PR. Proceeding with merge.

@kv2019i kv2019i merged commit 1fa4e22 into thesofproject:main Dec 11, 2025
36 of 43 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.

[BUG] github tests: build-linux ptl fails to heap_uncached_part failure

5 participants