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 K8800 board entry to the boards.h file #19331

Merged
merged 2 commits into from
Sep 10, 2020

Conversation

riodoro1
Copy link
Contributor

Description

Add K8800 (Velleman Vertex Delta) controller to the boards.h file.
The board has its respective pins_K8800.h file which is properly included in pins.h.
All of the siblings of the Vertex Delta (K8200, K8400, K8600) are already in boards.h so it seems this is a slip of hand while writing the file.

The board is sadly almost at the top of the list so the numbers need to be changed for most boards.
If you think this part is going to cause problems maybe I should add this entry at the end, this would hovewer be less nice.

Benefits

"Completeness" of the file and gives me less effort to prepare my own branch which supports this printer fully.
I want to be able to have this branch easily updatable and the boards file would be one less hassle.

Configurations

not applicable

Related Issues

I didn't create a FR for this.

All the numbers are moved around to keep the list nice and tidy.
The pins file and everything else is already there.
@thinkyhead thinkyhead merged commit abf1222 into MarlinFirmware:bugfix-2.0.x Sep 10, 2020
@thinkyhead
Copy link
Member

Thanks for the patch! I'm not sure how that one got lost in the mix.

vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
@riodoro1 riodoro1 deleted the add-k8800-to-boards branch April 11, 2021 14:38
kageurufu pushed a commit to CR30-Users/Marlin-CR30 that referenced this pull request Apr 30, 2021
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.

2 participants