Skip to content

feat(shields): Add dumbpad#1137

Closed
halcyonCorsair wants to merge 3 commits intozmkfirmware:mainfrom
halcyonCorsair:dumbpad
Closed

feat(shields): Add dumbpad#1137
halcyonCorsair wants to merge 3 commits intozmkfirmware:mainfrom
halcyonCorsair:dumbpad

Conversation

@halcyonCorsair
Copy link
Copy Markdown
Contributor

Add dumbpad macropad.

Board/Shield Check-list

  • This board/shield is tested working on real hardware
  • Definitions follow the general style of other shields/boards upstream (Reference)
  • .zmk.yml metadata file added
  • Proper Copyright + License headers added to applicable files (Generally, we stick to "The ZMK Contributors" for copyrights to help avoid churn when files get edited)
  • General consistent formatting of DeviceTree files
  • [ x Keymaps do not use deprecated key defines (Check using the upgrader tool)
  • &pro_micro used in favor of &pro_micro_d/a if applicable
  • Kconfig.defconfig file correctly wraps all configuration in conditional on the shield symbol
  • .conf file has optional extra features commented out
  • Keyboard/PCB is part of a shipped group buy or is generally available in stock to purchase (OSH/personal projects without general availability should create a zmk-config repo instead)

@Nicell Nicell added enhancement New feature or request shields PRs and issues related to shields labels Mar 14, 2022

# Uncomment to turn on logging, and set ZMK logging to debug output
# CONFIG_ZMK_USB_LOGGING=y
# CONFIG_SENSOR_LOG_LEVEL_DBG=y
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just curious, what did you need this specific debug logging for?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That was leftover from some debugging. I'll remove.

@@ -0,0 +1,11 @@
file_format: "1"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Another one of these .zmk.ymls for the oled_top variant I think

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

@halcyonCorsair
Copy link
Copy Markdown
Contributor Author

@Nicell this is ready for review again when you have time.

Comment on lines +24 to +29
config LVGL_HOR_RES_MAX
default 128

config LVGL_VER_RES_MAX
default 32

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These lines should be removed as of #1352

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.


# Uncomment to turn on logging, and set ZMK logging to debug output
# CONFIG_ZMK_USB_LOGGING=y
# CONFIG_SENSOR_LOG_LEVEL_DBG=y
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think you mentioned you were going to remove this line, but I still see it 😄

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

@nmunnich
Copy link
Copy Markdown
Contributor

nmunnich commented Mar 1, 2025

ZMK is currently not accepting any new shields for keyboards. Please make a module instead.

@nmunnich nmunnich closed this Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request shields PRs and issues related to shields

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants