-
-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
[Keyboard] Add hardwareabstraction/handwire Board #18504
[Keyboard] Add hardwareabstraction/handwire Board #18504
Conversation
keyboards/hardwareabstraction/handwire/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
keyboards/hardwareabstraction/handwire/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
keyboards/hardwareabstraction/handwire/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
keyboards/hardwareabstraction/handwire/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
keyboards/hardwareabstraction/handwire/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
#endif | ||
|
||
#ifdef BUZZER_ENABLE | ||
case KC_BZTOG: |
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.
Is there a reason for this extra code? It looks like it's a lot of duplication of the solenoid code.
And if that is the case, the solenoid code actually supports multiple pins. SOLENOID_PINS { E6, B2 }
could be used.
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.
The reason I didn't declare another solenoid pin was because I want to be able to individually turn off the buzzer and the solenoid. From my knowledge, it didn't look like there was a way to turn off an individual pin while keeping the other one on.
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
If this is a handwired board, it should probably live under |
Evidentially I did not name this board that well. There's an actual PCB for it, but some of the components require some manual soldering. If a new name is needed / required I can help support that. |
Co-authored-by: Tom Barnes <barnestom@me.com>
keyboards/hardwareabstraction/handwire/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
Co-authored-by: Drashna Jaelre <drashna@live.com>
…k_firmware into hardwareabstraction-keyboards
* upstream/master: Remove xealousbrown keyboard (qmk#18658) Bugfix waffling60 rev d solder (qmk#18673) [Keyboard] Add hardwareabstraction/handwire Board (qmk#18504) Vertex AngleR2 Layout Refactor (qmk#18661) [Keyboard][Keymap] Fix random keys being sent on Mac + userspace changes (qmk#15648) [Keyboard] New IDOBAO ID61 (a.k.a. Denwir D60) (qmk#17362) [Keyboard] Add ANAVI Macro Pad 10 (qmk#18620) Fix future naming collisions. (qmk#18645) Synth Labs Solo Configurator Fixes (qmk#18642) Update CRKBD keymap (qmk#18619) Remove all `BACKLIGHT_LEVELS 0` (qmk#18630) DD mappings for build.debounce_type and backlight.levels (qmk#18628) Add encoder map support for Keychron Q1v1 and Q2 (qmk#18598) Studio Kestra Nascent Refactor (qmk#18625) [Keyboard] Add supersplit (qmk#18611)
Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Tom Barnes <barnestom@me.com>
Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Tom Barnes <barnestom@me.com>
Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Tom Barnes <barnestom@me.com>
Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Tom Barnes <barnestom@me.com>
Description
Add support for handwire board. Utilizes ATMEGA32u4 microcontroller with a default and via keymap./
Keyboard features an OLED display, Solenoid, and Active Buzzer
Types of Changes
Issues Fixed or Closed by This PR
Checklist