micro:bit: chapter 04; 05 #271
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
left a TODO at the f3 picture since I need to replace it later on with another GIF of the micro:bit doing this
|
rust-highfive
added
S-waiting-on-review
eldruin
reviewed
Sep 24, 2020
| You might be surprised to hear that while "Arm-based" chips are quite | ||
| popular, the company behind the "Arm" trademark ([Arm Holdings][]) doesn't | ||
| actually manufacture chips for purchase. Instead, their primary business | ||
| model is to just *design* parts of chips. They will then license those designs to |
There was a problem hiding this comment.
We might need to update this once the nVIDIA buyout goes through.
Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>
cargo-embed would previously crash once connecting a GDB to its GDB stub, this is apparently because some regression between 0.8.0 and 0.9 either * stopped halting the chip automatically once a probe attached * changed the general behaviour of how the attaching works so much that a halt is now required Either way this seems to fix it so digging into the depths of probe-rs does not seem to be necessary.
Only cargo embed 0.9.1 includes my fix to overwrite the soft device that is on the micro:bit board by default in its initial state -> update from 0.9.0 to 0.9.1.
cargo-embed as of now only supports monitor reset, not monitor reset halt or anything else like that. Remove those from the GDB appendix.
Originally the blinky was introduced in a single chapter, instead split it up into two chapters, one introducing embedded-hal and lighting an LED up and the other (as of now empty one) for introducing the delay abstractions and letting an LED blink.
|
I left a few notes asking for resources to link to in chapter 5 in the code, would be nice if someone could come up with some suggestions! |
eldruin
reviewed
Sep 27, 2020
Commas and the likes were never my strength anyways... Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>
Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>
Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>
Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>
|
The PR now contains everything that is required for ch04 and 05 and should be ready for review. Once review of the last parts passes the only thing we'd need is a nice resource for LED matrices and could actually merge this \o/ |
Co-authored-by: Diego Barrios Romero <eldruin@gmail.com>
eldruin
approved these changes
Oct 10, 2020
|
Build succeeded: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A rewrite of chapter 04 and partially chapter 05.
I'm already uploading this now since the cargo-embed GDB stub is blowing up on my machine when trying to debug this chip so I'll probably have to spend some time debugging, however during that time my current work can already be reviewed so feel free to take a look at it!