-
-
Notifications
You must be signed in to change notification settings - Fork 40.7k
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
Clean up ATSAM ifdefs #8808
Clean up ATSAM ifdefs #8808
Conversation
/sigh I know it's needed, but I would really love to see the massive PR from massdrop merged before this is, ... mostly so we can just get it in. |
Well, the only changes to the EDIT: Just grepped through the PR and there are exactly 0 instances of these old defines in the diff, so this should be OK to merge first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Thanks! |
* upstream: (1037 commits) [Keyboard] Add wasdat code controller (qmk#8858) fix sample code indent in feature_encoders.md (qmk#8883) [Keymap] Clean up my ergo keymaps and userspace (qmk#8857) idb 60 Bugfixes / Preparations for Open Source Hardware (qmk#8866) Rebound: add rev2 and thus rev1 as well (qmk#8630) Update vitamins included default keymap, enable NKRO, rev2 rgbsplit (qmk#8871) Update to xealousbrown. (qmk#8215) [Keyboard] DMQ Design SPIN (qmk#8820) Wheatfield Blocked65: Update RGBLED num (qmk#8725) Add VIA support to ID80 (qmk#8791) CFTKB Mysterium & Discipad VIA support (qmk#8794) Clean up ATSAM ifdefs (qmk#8808) [Docs] Japanese translation of docs/feature_encoders.md (qmk#8843) Add naked60 layout, clean up my userspace files and rules.mk. (qmk#8848) Fixing DecadePad Numlock LED Bug (qmk#8831) [Docs] Japanese translation of docs/feature_command.md (qmk#8672) Add support for YMD75 rev 2 (qmk#8853) Remove no-longer-necessary LTO checks from keyboards' config.h files (qmk#8773) Fix ta-65 tsangan layouts (qmk#8855) Fix Plain60 layout (qmk#8854) ...
Description
For some reason the ATSAM code uses these three-letter defines for NKRO/mousekeys/extrakeys/console/raw/virtser/etc, when it really doesn't need to. I also reformatted the descriptors to be more readable.
Types of Changes
Issues Fixed or Closed by This PR
Checklist