Implement the Pigweed-app example on EFR32 #4630
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
pullapprove
bot
requested review from
andy31415,
BroderickCarlin,
bzbarsky-apple,
chrisdecenzo,
hawk248,
jelderton,
mspang,
saurabhst and
woody-apple
andy31415
reviewed
Feb 3, 2021
szatmz
reviewed
Feb 3, 2021
szatmz
approved these changes
Feb 3, 2021
andy31415
approved these changes
Feb 3, 2021
jelderton
approved these changes
Feb 4, 2021
jmartinez-silabs
force-pushed
the
pw-efr
branch
from
abdb714 to
f627cdc
Compare
mspang
requested changes
Feb 5, 2021
- Init uart driver and create function to read and write a character - link those functions with pigweed backend - move some files that can be shared for all EFR32 examples to efr32/platform - create GN build structure for pigweed usage with EFR32 platform Implement the Pigweed-app example on EFR32 - Init uart driver and create function to read and write a character - link those functions with pigweed backend - move some files that can be shared for all EFR32 examples to efr32/platform - create GN build structure for pigweed usage with EFR32 platform Change uart driver from uartdrv to retargetSerial, using the dma was causing issues reading with pigweed proto Cleanup in the build GN files Some cleanup for the build
Change copyright to 2021 for new files
woody-apple
approved these changes
Feb 5, 2021
…e for all examples. That config is only used when pw_rpc is build Create a with_pw_rpc.gni file in efr examples. This gni file need to be imported in the build args when we will want to build a example with pw_rpc (this is still a WIP) This doesn't impact pigweed-app example because it always runs the pw_rpc
jmartinez-silabs
force-pushed
the
pw-efr
branch
from
f627cdc to
a615898
Compare
mspang
approved these changes
Feb 5, 2021
|
@woody-apple @andy31415 @mspang Can this be Merged? |
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.
Problem
Need of a Pigweed rcp example on EFR32 platforms
Summary of Changes
example/pigweed-app/efr32/pw_sys_io