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

CurieMailbox: remove HW access code which is now in cores/arduino #549

Open
wants to merge 1 commit into
base: codk-m-corelibs
Choose a base branch
from

Conversation

eriknyquist
Copy link
Contributor

@eriknyquist eriknyquist commented May 17, 2017

The mailbox HW initialisation now happens in cores/arduino/main.cpp.
All of the core code for accessing the mailbox HW now lives in
cores/arduino/mailbox.cpp; the CurieMailbox library is now just a wrapper
that calls the methods from mailbox.cpp, plus a ring buffer.

NOTE: this must not be merged until #547 has been merged into master, and also migrated over to the target branch of this PR (codk-m-corelibs)

@eriknyquist eriknyquist added this to the Bootes milestone May 17, 2017
@eriknyquist eriknyquist self-assigned this May 17, 2017
The mailbox HW initialisation now happens in cores/arduino/main.cpp.
All of the core code for accessing the mailbox HW now lives in
cores/arduino/mailbox.cpp; the CurieMailbox library is now just a wrapper
that calls the methods from mailbox.cpp, plus a ring buffer.
@eriknyquist eriknyquist changed the title Add IPM (Inter-Processor-Mailbox) driver to cores/arduino CurieMailbox: remove HW access code which is now in cores/arduino May 18, 2017
@kitsunami kitsunami removed this from the Bootes milestone Aug 30, 2017
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

3 participants