Skip to content

Remove deprecated items and repo cleanup #875

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

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jbeaurivage
Copy link
Contributor

@jbeaurivage jbeaurivage commented May 12, 2025

Summary

Since we have a breaking release lined up, it's a good opportunity to sneak in breaking changes to clean up the repo.

  • Bump MSRV to 1.85.1 and Rust edition to 2024 (atsamd-hal and atsamd-hal-macros).
  • rustfmt clean up, mostly imports in alphabetical order
  • Change CI to use cargo fmt instead of rustfmt. It will automatically respect the individual
    crates' Rust editions.
  • Remove some unused dependencies: void and vcell
  • Update defmt, rng-core and embedded-sdmmc dependencies
  • Remove a bunch of deprecated code:
  • spi_future
  • Some items previously used for SERCOM+DMA. Now the embedded-hal implementations can be used with DMA instead.
  • embedded-hal v0.2. This was long overdue!

Closes #784.

I'd like to wait on #814 before merging, because there's no point in touching the adc module - which has some old ehal 0.2 impls - here, since it underwent a major rewrite.

Note to maintainers:

Please merge this PR using rebase. Please do not squash the commits!

@jbeaurivage jbeaurivage marked this pull request as draft May 12, 2025 20:52
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from f5a2ef1 to c6c93c5 Compare May 12, 2025 20:56
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from c6c93c5 to 6d0eb0d Compare May 12, 2025 21:29
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from 6d0eb0d to c51ae9d Compare May 12, 2025 21:34
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from c51ae9d to d22f2af Compare May 12, 2025 21:37
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from d22f2af to 4a75154 Compare May 12, 2025 21:40
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from 4a75154 to e870e36 Compare May 12, 2025 21:44
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from b44081e to ca1069c Compare May 12, 2025 22:05
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@nyurik
Copy link

nyurik commented May 12, 2025

I love the overall cleaner feel! Would it make sense to rename gen to something that is not a keyword in 2024?

@jbeaurivage
Copy link
Contributor Author

Would it make sense to rename gen to something that is not a keyword in 2024?

I don't think it's a big deal to keep it that way, I would definitely rename it if it was user-facing, but the gen function name comes to us from the PAC. I'm not really inclined to start messing with register field names through XSL patches, especially since the field name would now be different from the datasheet.

jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from ca1069c to 678c031 Compare May 12, 2025 22:22
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 12, 2025
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 13, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from 19ffec5 to 3db2d65 Compare May 13, 2025 17:23
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 13, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from 3db2d65 to 1a11179 Compare May 13, 2025 17:28
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 13, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from 1a11179 to 2134bd0 Compare May 13, 2025 17:32
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 13, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from 2134bd0 to d768c25 Compare May 13, 2025 17:36
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 13, 2025
@jbeaurivage jbeaurivage force-pushed the remove-deprecated-items branch from d768c25 to 6bea13e Compare May 13, 2025 17:38
jbeaurivage added a commit to jbeaurivage/atsamd that referenced this pull request May 13, 2025
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.

Tracking issue - remove deprecated code
2 participants