Skip to content

MCUXpresso: Fix issue of ticker interrupt incorrectly firing #5316

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
Closed
wants to merge 1 commit into from
Closed

MCUXpresso: Fix issue of ticker interrupt incorrectly firing #5316

wants to merge 1 commit into from

Conversation

mmahadevan108
Copy link
Contributor

Fix for Issue #5279

Status

**READY

Todos

@mmahadevan108 mmahadevan108 mentioned this pull request Oct 13, 2017
Closed
@mprse
Copy link
Contributor

mprse commented Oct 17, 2017

Provided fix resolves Issue #5279.
Checked on K64F and ARM, GCC_ARM, IAR tool-chains.

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 17, 2017

@tommikas PLease can you restat jenkins cI?

@tommikas
Copy link
Contributor

@0xc0170 Looks like jenkins/pr-head is already green on this one?

@mbed-ci
Copy link

mbed-ci commented Oct 17, 2017

Build : FAILURE

Build number : 191
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

@0xc0170 0xc0170 closed this Oct 18, 2017
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017

I closed PR as travis did not even notice this PR (I browsed history more than 7 days ago). Ropening now,
and will trigger the rest of CI

@0xc0170 0xc0170 reopened this Oct 18, 2017
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017

@mmahadevan108 I am seding a new PR , cherrypick your commit, travis seems to be playing with us for this one

@0xc0170 0xc0170 mentioned this pull request Oct 18, 2017
Merged
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 18, 2017 •
edited
Loading

New PR sent, please review, we can close this one

@mbed-ci
Copy link

mbed-ci commented Oct 18, 2017

Test : SUCCESS

Build number : 116
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/5316/116

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 19, 2017

@mmahadevan108 I am closing this one, the replacement I sent earlier has all green, ready for integration. thanks for understanding

@0xc0170 0xc0170 closed this Oct 19, 2017
@0xc0170 0xc0170 removed the needs: CI label Oct 19, 2017
@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 243
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 244
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 245
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 246
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : ABORTED

Build number : 247
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 248
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : ABORTED

Build number : 250
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 251
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : ABORTED

Build number : 252
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 254
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 257
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 260
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 261
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : ABORTED

Build number : 262
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2017

Build : SUCCESS

Build number : 263
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5316/

Skipping test trigger, missing label 'NEED CI'

@ARMmbed ARMmbed deleted a comment from 0xc0170 Oct 19, 2017
@ARMmbed ARMmbed deleted a comment from mbed-ci Oct 19, 2017
@mmahadevan108 mmahadevan108 deleted the Fix_ustimer branch August 20, 2019 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

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

Successfully merging this pull request may close these issues.
6 participants
@mmahadevan108 @mprse @0xc0170 @tommikas @mbed-ci @theotherjimmy