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

Import changes in kernel/src/common/utils.rs from #1618. #1924

Merged
merged 1 commit into from Jun 12, 2020

Conversation

gendx
Copy link
Contributor

@gendx gendx commented Jun 9, 2020

Pull Request Overview

This pull request adds a new static_buf! macros and related types, as previously discussed in #1618 and agreed upon in #1618 (comment).

This essentially imports the changes in kernel/src/common/utils.rs from #1618, as well as the necessary rebasing of the code.

Changes to the components themselves should further be discussed and agreed upon in #1618, and then a separate pull request (or multiple PRs) can migrate the code to new components.

Testing Strategy

This pull request was tested by CI.

TODO or Help Wanted

N/A - This part was extensively discussed in #1618.

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

Copy link
Member

@ppannuto ppannuto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

This has been thoroughly discussed at this point, I think it's good to go in. Thanks for pushing this forward @gendx!

@bors bors bot merged commit a48bb68 into tock:master Jun 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants