-
-
Notifications
You must be signed in to change notification settings - Fork 19.2k
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
Break apart and clean up pins.h #1412
Comments
Think about that idea of a abstraction layer so we can associate a function Cheers. Alex. 2015-01-25 0:45 GMT-02:00 Scott Lahteine notifications@github.com:
"Não é o mais forte da espécie que sobrevive, nem o mais inteligente. É Alex Borro |
First step was to split up the pins into separate files and clean them up. The result looks pretty good. The abstraction layer should be easier to conceive given this improvement. #1416 |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
To make the Marlin root folder even more interesting (and to aid future development) I'm in the process of going through pins.h and breaking it up into separate files based on MOTHERBOARD name. Whether or not this produces a better final result is yet to be seen, but it's a good exercise to get a handle on the slightly messy state of pin assignments. It may end up making it easier to maintain pins and add new board support in the future, but for now it's more of an academic exercise to see whether pins files can be more consistently structured, to eliminate redundancies, and to make it's easier to find the pins you're looking for.
The text was updated successfully, but these errors were encountered: