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

Fix irq stats. #4783

Closed
wants to merge 2 commits into from
Closed

Fix irq stats. #4783

wants to merge 2 commits into from

Conversation

iabdalkader
Copy link
Contributor

  • This may have been overlooked, but only the M4 and M7 MCUs have an FPU and FPU_IRQn. The build will fail for other MCUs like the F0.
  • Additionally, FPU_IRQn is Not the last entry/IRQ number. For example see stm32f765xx.h IRQs numbers run up to 109.
  • I think 256 should be enough for all the supported STMs, unless it's better to define it for each MCU.

@iabdalkader
Fix irq stats.
a2bd6b8 
* This may have been overlooked, but only the M4 and M7 MCUs have an FPU and FPU_IRQn. The build will fail for other F0.
* Additionally, FPU_IRQn is Not the last entry/IRQ number. For example see stm32f765xx.h IRQs numbers run up to 109.
* I think 256 should be enough for all the supported STMs, unless it's better to define it for each MCU.
dpgeorge
dpgeorge reviewed May 14, 2019
View reviewed changes
ports/stm32/irq.h Outdated Show resolved Hide resolved
@dpgeorge
Copy link
Member

Thanks for updating, merged in 1646eff

@dpgeorge dpgeorge closed this May 15, 2019
@iabdalkader iabdalkader deleted the irq_stats branch May 15, 2019 13:11
tannewt pushed a commit to tannewt/circuitpython that referenced this pull request May 20, 2021
@dhalbert
Merge pull request micropython#4783 from tannewt/fix_sm_rp2040_7
74355e8 
Fix safe mode on rp2040 (in 7.x)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dpgeorge dpgeorge dpgeorge left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@iabdalkader @dpgeorge