Initial support for nRF51822 #17
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Minimum support for output pins.
Make sure code specific to the newly added platform is built on CI.
| script: | ||
| - make build/main.elf | ||
| - make clean-all | ||
| - make PLATFORM=nrf_pca10001 CHIP=nrf51822 APPS=c_blinky |
There was a problem hiding this comment.
Oh great! So this will test compilation for both platforms. I think APPS=c_blinky might be somewhat extraneous since it just gets compiled to a blob and linked -- there is nothing really specific to this platform here.
But no matter, it's at least a place for now to document how to get an example that actually runs on the board.
There was a problem hiding this comment.
Once UART is implemented for this board, I can simply drop APPS and still have a functional default build. Should be my next task.
This was referenced Feb 7, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add support for the PCA10001 board with the nRF51822 SoC. This initial support contains just the minimal support for the "blinky" app to work: