Skip to content

Switch itsybitsy_m4 BSP to latest HAL release.#954

Merged
jbeaurivage merged 1 commit intoatsamd-rs:masterfrom
codepainters:itsy_m4_update
Mar 24, 2026
Merged

Switch itsybitsy_m4 BSP to latest HAL release.#954
jbeaurivage merged 1 commit intoatsamd-rs:masterfrom
codepainters:itsy_m4_update

Conversation

@codepainters
Copy link
Copy Markdown
Contributor

Summary

Changes necessary to make itsybitsy_m4 BSP/examples build properly with HAL version 0.22.
I had to make this change to test #948 on my ItsyBitsy M4 Express board.

BTW, I can see some inconsistencies when it comes to using HAL from repo vs pulling a crate. Some BSPs (e.g. feather_m4) use path to reference the repo:

[dependencies.atsamd-hal]
default-features = false
path = "../../hal"
version = "0.22.2"

while others (like itsybitsy_m4) simply pull the crate as a dependency.
Is there any policy in place?

@codepainters
Copy link
Copy Markdown
Contributor Author

Hmm, it looks like sercom_interrupt example fails compiling, let me check.

@jbeaurivage
Copy link
Copy Markdown
Contributor

Hey @codepainters, this isn't a proper review (yet), but I just wanted to chime in on the path vs crates.io dependencies. We use a tier system for our BSPs, where Tier 1 are always pulling the HAL from the latest master, while Tier 2 specify a crates.io version, which may or may not be the latest release. This helps alleviate maintainer burden when breaking changes happen in the HAL, while still having some boards to test against. You can find the boards' tiers in the readme or in crates.json.

This board is a Tier 2, so it's correct to leave the dep to the crates.io release.

@codepainters
Copy link
Copy Markdown
Contributor Author

Thanks for the clarification! Now it makes much more sense.


[dependencies.atsamd-hal]
version = "0.21"
version = "0.22.2"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This could probably be 0.23 now.

@jbeaurivage jbeaurivage merged commit 9320d05 into atsamd-rs:master Mar 24, 2026
108 of 109 checks passed
@github-actions github-actions bot mentioned this pull request Mar 23, 2026
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.

2 participants