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
chips/qemu_rv32_virt_chip: implement basic UART RX and add process console to board #3365
Conversation
bors r+ |
Merge conflict. |
2deddd0
371b507
to
2deddd0
Compare
Rebased accordingly, merging if checks succeed. bors r+ |
3365: chips/qemu_rv32_virt_chip: implement basic UART RX and add process console to board r=lschuermann a=lschuermann ### Pull Request Overview Implements basic UART RX support (without abort) and adds the process console component to the board. I didn't want to invest time in supporting receive-abort, as this needs to be migrated to the new UART HIL anyways (#3046). ### Testing Strategy This pull request was tested by using the process console. ### TODO or Help Wanted N/A ### Documentation Updated - [x] ~Updated the relevant files in `/docs`,~ or no updates are required. ### Formatting - [x] Ran `make prepush`. Co-authored-by: Leon Schuermann <leon@is.currently.online>
This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried. Additional information: Response status code: 422
{"message":"At least 1 approving review is required by reviewers with write access. Required status check \"netlify/docs-tockosorg/deploy-preview\" is failing.","documentation_url":"https://docs.github.com/articles/about-protected-branches"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors r+
3365: chips/qemu_rv32_virt_chip: implement basic UART RX and add process console to board r=hudson-ayers a=lschuermann ### Pull Request Overview Implements basic UART RX support (without abort) and adds the process console component to the board. I didn't want to invest time in supporting receive-abort, as this needs to be migrated to the new UART HIL anyways (#3046). ### Testing Strategy This pull request was tested by using the process console. ### TODO or Help Wanted N/A ### Documentation Updated - [x] ~Updated the relevant files in `/docs`,~ or no updates are required. ### Formatting - [x] Ran `make prepush`. Co-authored-by: Leon Schuermann <leon@is.currently.online>
This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried. Additional information: Response status code: 422
{"message":"Required status check \"netlify/docs-tockosorg/deploy-preview\" is failing.","documentation_url":"https://docs.github.com/articles/about-protected-branches"} |
34e038a
2deddd0
to
34e038a
Compare
@hudson-ayers Could you give another approval perhaps? This time the Netlify CI didn't glitch out, so maybe fourth time's the charm? 😆 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors r+
Pull Request Overview
Implements basic UART RX support (without abort) and adds the process console component to the board. I didn't want to invest time in supporting receive-abort, as this needs to be migrated to the new UART HIL anyways (#3046).
Testing Strategy
This pull request was tested by using the process console.
TODO or Help Wanted
N/A
Documentation Updated
Updated the relevant files inor no updates are required./docs
,Formatting
make prepush
.