Skip to content

Use define instead of enum for pins in variant.h #356

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

Merged
merged 1 commit into from
Oct 16, 2018

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented Oct 15, 2018

This PR is based on #309 thanks @ghent360
This squash the #309 and rebase it on master.
Include:

  • Update 2 variants merged recently: REMRAM and RAK811.
  • Remove PEND as no more relevant
  • Remove duplicated pins which is not required anymore as PEND removed (even if in fact it was not required before)
  • Cosmetic update (comments, space, ...)

Note: https://github.com/stm32duino/wiki/wiki/Add-a-new-variant-(board) updated

@fpistm fpistm added this to the 1.4.0 milestone Oct 15, 2018
@fpistm fpistm self-assigned this Oct 15, 2018
Remove PEND as no more relevant
Remove duplicated pins which is not required anymore as PEND removed

Signed-off-by: Venelin Efremov <[email protected]>
@fpistm fpistm merged commit bfce9b2 into stm32duino:master Oct 16, 2018
@fpistm fpistm deleted the pr-309-review branch October 16, 2018 12:35
benwaffle pushed a commit to benwaffle/Arduino_Core_STM32 that referenced this pull request Apr 10, 2019
Use define instead of enum for pins in variant.h
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