Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

west.yml: update zephyr to 0837ad5be84 #9913

Merged

Conversation

tmleman
Copy link
Contributor

@tmleman tmleman commented Mar 20, 2025

Total of 432 commits.

Changes include:

3df1ffe1d5b xtensa: mmu: Fix partition permission definitions
6249470b669 intel_adsp: ace30: mm: Set write permissions during memory mapping
e7f2e1412d5 soc: intel_adsp: set MMU permissions for .cold* sections

Total of 432 commits.

Changes include:

3df1ffe1d5b xtensa: mmu: Fix partition permission definitions
6249470b669 intel_adsp: ace30: mm: Set write permissions during memory
            mapping
e7f2e1412d5 soc: intel_adsp: set MMU permissions for .cold* sections

Signed-off-by: Tomasz Leman <[email protected]>
@kv2019i
Copy link
Collaborator

kv2019i commented Mar 21, 2025

Seems we have an issue with compiler setttings as this was now caught in stub3 build:

https://github.com/thesofproject/sof/actions/runs/13980547833/job/39144648495?pr=9913

 /home/runner/work/sof/sof/workspace/zephyr/kernel/thread.c:456:2: error: arithmetic on a pointer to void is a GNU extension [-Werror,-Wpointer-arith]
        LOG_DBG("stack %p for thread %p: obj_size=%zu buf_start=%p "

The error is coming from Zephyr code, so I think we can proceed with merging this and file a bug to follow-up (either SOF or Zephyr, depending on whether our compiler settings are too strict in SOF, or whether there's a problem in main of Zephyr).

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.

One new compiler error from udpated Zephyr code. I think we can merge this if a follow-up ticket is filed.

@lgirdwood
Copy link
Member

@tmleman can you log/fix this on Zephyr side and link here. Alternatively we could cast or do the arithmetic outside the LOG() ?

@tmleman
Copy link
Contributor Author

tmleman commented Mar 25, 2025

@lgirdwood @kv2019i The issue has already been reported in Zephyr: zephyrproject-rtos/zephyr#87475

@kv2019i kv2019i merged commit 9392160 into thesofproject:main Mar 25, 2025
45 of 49 checks passed
@lyakh lyakh mentioned this pull request Mar 26, 2025
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.

3 participants