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

Add Bonsai C4 as a platform board file #18901

Merged
merged 5 commits into from
Nov 13, 2022
Merged

Add Bonsai C4 as a platform board file #18901

merged 5 commits into from
Nov 13, 2022

Conversation

infinityis
Copy link
Contributor

This creates Bonsai C4 as a board file (as recommended here), which makes the ProMicro to Bonsai C4 converter work without having to configure hardware, change/add code, etc.

This also removes the Bonsai C4 template code from the keyboards/customMK folder which demonstrated how to configure the hardware, since that is no longer needed, and updates the converter documentation accordingly.

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).

@zvecr
Copy link
Member

zvecr commented Nov 12, 2022

Conflicts will need to be resolved, otherwise it will fall into the next develop cycle.

@infinityis
Copy link
Contributor Author

Conflicts have been resolved

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.

Just a small fix, rest LGTM.

platforms/chibios/boards/BONSAI_C4/configs/config.h Outdated Show resolved Hide resolved
Co-authored-by: Nick Brassel <nick@tzarc.org>
@infinityis
Copy link
Contributor Author

Fixed, thank you!

@zvecr zvecr merged commit a42ab90 into qmk:develop Nov 13, 2022
ramonimbao pushed a commit to ramonimbao/qmk_firmware that referenced this pull request Nov 28, 2022
* Set up Bonsai C4 as a platform board file

* corrections and improvements based on testing and feedback

* Added VBUS sensing as default capability for improved split support using Bonsai C4

* Update clock divisor for SPI flash

Co-authored-by: Nick Brassel <nick@tzarc.org>

Co-authored-by: Nick Brassel <nick@tzarc.org>
elpekenin pushed a commit to elpekenin/qmk_firmware that referenced this pull request Dec 7, 2022
* Set up Bonsai C4 as a platform board file

* corrections and improvements based on testing and feedback

* Added VBUS sensing as default capability for improved split support using Bonsai C4

* Update clock divisor for SPI flash

Co-authored-by: Nick Brassel <nick@tzarc.org>

Co-authored-by: Nick Brassel <nick@tzarc.org>
crembz pushed a commit to crembz/qmk_firmware that referenced this pull request Dec 18, 2022
* Set up Bonsai C4 as a platform board file

* corrections and improvements based on testing and feedback

* Added VBUS sensing as default capability for improved split support using Bonsai C4

* Update clock divisor for SPI flash

Co-authored-by: Nick Brassel <nick@tzarc.org>

Co-authored-by: Nick Brassel <nick@tzarc.org>
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.

3 participants