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

FUTURE_SEQUANA: InterruptIn implementation bug fix #11404

Merged
merged 1 commit into from
Sep 24, 2019

Conversation

lrusinowicz
Copy link
Contributor

Description

Fixed HAL API implementation for InterruptIn:

  • Interrupt was not enabled by default after configuration as it should be.
  • Interrupt-to-object linking was not handled properly.

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Release Notes

Fixed HAL API implementation for InterruptIn:
 - Interrupt was not enabled by default after configuration as it should be.
 - Interrupt-to-object linking was not handled properly.
@ciarmcom
Copy link
Member

ciarmcom commented Sep 3, 2019

@lrusinowicz, thank you for your changes.
@maclobdell @ARMmbed/mbed-os-maintainers please review.

@ciarmcom ciarmcom requested review from maclobdell and a team September 3, 2019 15:00
@0xc0170 0xc0170 requested a review from a team September 4, 2019 07:40
@0xc0170
Copy link
Contributor

0xc0170 commented Sep 6, 2019

CI started meanwhile

@mbed-ci
Copy link

mbed-ci commented Sep 6, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 9, 2019

@ARMmbed/team-cypress Does this look good?

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 17, 2019

@ARMmbed/team-cypress Does this look good?

Please review

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 24, 2019

CI restarted meanwhile

@mbed-ci
Copy link

mbed-ci commented Sep 24, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants