[WIP] Update the microbit part of the book #575
Draft
Conversation
This file contains 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
Updated to mention the 2021 edition in stead of 2018
Cargo.toml:
* edition 2018 -> 2021
* cortex-m 0.7.3 -> 0.7.7, added feature `critical-section-single-core`
* cortex-m-rt 0.7.0 -> 0.7.3
* rtt-target 0.3.1 -> 0.5.0, removed feature `cortex-m`
* panic-rtt-target 0.1.2 -> 0.1.3, removed feature `cortex-m`
src/main.rs:
* To prevent a link error regarding undefined symbols
(`_critical_section_1_0_acquire` and `_critical_section_1_0_release`)
I have added `use cortex_m as _;`
The code compiles, and I see the RTT output from cargo embed.
Cargo.toml * edition 2018 -> 2021 * microbit-v2 0.12.0 -> 0.14.0 * microbit 0.12.0 -> 0.14.0 * cortex-m 0.7.3 -> 0.7.7, added feature `critical-section-single-core` * cortex-m-rt 0.7.0 -> 0.7.3 * rtt-target 0.3.1 -> 0.5.0, removed feature `cortex-m` * panic-rtt-target 0.1.2 -> 0.1.3, removed feature `cortex-m`
The microbit crate apparently don't reexport the hal's OutputPin trail, there for add embedded-hal as a dependency.
A new version of the microbit crates has been released.
Cargo.toml
* microbit-v2 0.14.0 -> 0.15.0
Which updated HAL crates to version 0.18.0
* microbit 0.14.0 -> 0.15.0
In connection with this I moved all the examples in the book out of book
text and made them project examples, and used an include statement, like
it's done in `my-solution`, which makes checking the examples faster,
and I also cought a couple of typos I made when I updated the text,
which rust-analyser didn't like.
The examples can be run manually with commands like:
`cargo embed --target thumbv7em-none-eabihf --features v2 --example
it-blinks-1`
Don't know if this messes with any automatic tests yet.
BartMassey
requested changes
Jun 21, 2024
| @@ -45,22 +45,23 @@ $ gdb target/thumbv6m-none-eabi/debug/led-roulette | |||
| > as long as it does not crash, you are fine. | |||
|
|
|||
| Next we will have to connect to the GDB stub. It runs on `localhost:1337` per default so in order to | |||
| connect to it run the following: | |||
| connect to it run the following on the gdb commandline (`(gdb)`): | |||
There was a problem hiding this comment.
This is a bit confusing, I think. How about "on the gdb command line (that is, at the (gdb) prompt):"
| @@ -2,28 +2,29 @@ | |||
| name = "led-roulette" | |||
| version = "0.1.0" | |||
| authors = ["Henrik Böving <hargonix@gmail.com>"] | |||
| edition = "2018" | |||
| edition = "2021" | |||
|
|
|||
| [dependencies.microbit-v2] | |||
There was a problem hiding this comment.
Most of the number corrections (version numbers, runtimes, entry points, sizes, etc) will change quickly as the ecosystem updates. A note to that effect should be put here somewhere, so that we aren't doing this on some regular basis. Probably should just freeze things at some point and make careful notes about the reference point, I guess? I don't know — it's a hard problem.
In a fantasy world, this kind of stuff would be done automatically in CI. But that sounds even harder.
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.
This is currently a work in progress.
Update the crate versions to the latest versions as I work my way through the book.
I plan on squasing the commits into one per chapter when this is ready.