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

Fix ISR powerdown in mbed_application.c #4954

Merged
merged 1 commit into from Aug 29, 2017

Conversation

Projects
None yet
5 participants
@c1728p9
Contributor

c1728p9 commented Aug 22, 2017

Fix a calculation error in in powerdown_nvic preventing the last
group of 32 interrupts from getting powered down. The ARMv7m reference
manual states ""the total number of interrupt lines is up
to (32*(INTLINESNUM+1)).".

Also rename isr_count to isr_groups_32 for clarity, since this is
actually the number of 32 interrupt groups.

@c1728p9

This comment has been minimized.

Contributor

c1728p9 commented Aug 22, 2017

This should fix #4953.

Fix ISR powerdown in mbed_application.c
Fix a calculation error in in powerdown_nvic preventing the last
group of 32 interrupts from getting powered down. The ARMv7m reference
manual states "the total number of interrupt lines is up
to (32*(INTLINESNUM+1)).".

Also rename isr_count to isr_groups_32 for clarity, since this is
actually the number of 32 interrupt groups.

@c1728p9 c1728p9 force-pushed the c1728p9:fix_isr_clear branch to 29ad5bb Aug 22, 2017

@c1728p9

This comment has been minimized.

Contributor

c1728p9 commented Aug 22, 2017

/morph test

@0xc0170 0xc0170 added the needs: CI label Aug 23, 2017

@studavekar

This comment has been minimized.

Collaborator

studavekar commented Aug 28, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Aug 29, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1129

All builds and test passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Aug 29, 2017

@theotherjimmy theotherjimmy merged commit 2d65f49 into ARMmbed:master Aug 29, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment