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.

Sign up for GitHub

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

Jump to bottom

C++20 volatile warnings #1343

Open
GFoxPM opened this issue Apr 17, 2023 · 2 comments
Open

C++20 volatile warnings #1343

GFoxPM opened this issue Apr 17, 2023 · 2 comments
Assignees
@will-v-pi
Labels
cmsis compiler version issues
Milestone
2.1.0

Comments

@GFoxPM
Copy link

GFoxPM commented Apr 17, 2023 •
edited
Loading

Lots of warnings: compound assignment with 'volatile'-qualified left operand is deprecated [-Wvolatile]
Fixes like:
/src/rp2_common/cmsis/stub/CMSIS/Core/Include/mpu_armv7.h#L210

-  MPU->CTRL  &= ~MPU_CTRL_ENABLE_Msk;
+  MPU->CTRL  = MPU->CTRL & ~MPU_CTRL_ENABLE_Msk;
@GFoxPM GFoxPM changed the title c++20 warning fix C++20 volatile warnings Apr 20, 2023
@kilograham kilograham modified the milestones: 1.6.1, 1.6.0 May 20, 2024
@kilograham kilograham assigned kilograham and will-v-pi and unassigned kilograham Jul 20, 2024
@kilograham
Copy link
Contributor

@will-v-pi is this fixed with latest CMSIS headers?

@will-v-pi
Copy link

No, although I think it's being looked into - see ARM-software/CMSIS_5#1544 and ARM-software/CMSIS_6#121

These CMSIS stubs are taken directly from the upstream CMSIS repo, so we won't be modifying them in this repo. Once they're fixed there we'll update our headers here

@will-v-pi will-v-pi modified the milestones: 1.6.0, 1.7.0 Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@will-v-pi will-v-pi

Labels
cmsis compiler version issues
Projects
None yet
Milestone
2.1.0
Development

No branches or pull requests
3 participants
@kilograham @GFoxPM @will-v-pi