Skip to content
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

Remove arm_atsam platform #24337

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Remove arm_atsam platform #24337

merged 1 commit into from
Sep 2, 2024

Conversation

zvecr
Copy link
Member

@zvecr zvecr commented Aug 28, 2024

Description

Planned changes to the report sending code means updating the arm_atsam integration. However these changes are mostly just to keep the code base building, rarely tested, and active development of this area has stopped.

With the RIOT port being mostly functional now, its time for the deprecated platform code to be removed.

Support for existing keyboards will return after the prerequisite PRs are merged. Till then, users could choose to build old tags (or use the various configurator apps and precompiled firmware).

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Copy link
Member

@tzarc tzarc left a comment

Choose a reason for hiding this comment

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

Make it so.

@zvecr zvecr changed the title Remove arm_atsam platform Remove arm_atsam platform Aug 28, 2024
Copy link
Member

@KarlK90 KarlK90 left a comment

Choose a reason for hiding this comment

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

Yeah!

@zvecr zvecr merged commit 3bd303f into develop Sep 2, 2024
8 checks passed
@zvecr zvecr deleted the rm_arm_atsam branch September 2, 2024 02:48
waffle87 pushed a commit to waffle87/qmk_firmware that referenced this pull request Sep 6, 2024
waffle87 pushed a commit to waffle87/qmk_firmware that referenced this pull request Sep 6, 2024
itsme-zeix pushed a commit to itsme-zeix/qmk_firmware that referenced this pull request Oct 2, 2024
ilham-agustiawan pushed a commit to ilham-agustiawan/qmk_firmware that referenced this pull request Nov 30, 2024
smallketchup82 pushed a commit to smallketchup82/qmk_firmware that referenced this pull request Dec 1, 2024
jlaptavi pushed a commit to jlaptavi/qmk_firmware that referenced this pull request Dec 3, 2024
DmNosachev pushed a commit to DmNosachev/qmk_firmware that referenced this pull request Dec 7, 2024
SyrupSplashin pushed a commit to SyrupSplashin/qmk_firmware that referenced this pull request Dec 10, 2024
LeonMusCoden pushed a commit to LeonMusCoden/qmk_firmware that referenced this pull request Jan 5, 2025
MichaelScofield45 pushed a commit to MichaelScofield45/qmk_firmware that referenced this pull request Jan 8, 2025
pashashocky pushed a commit to pashashocky/qmk that referenced this pull request Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants