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

[Silicon Labs] Update board IDs #166

Merged
merged 2 commits into from
Apr 3, 2017

Conversation

stevew817
Copy link
Contributor

Updated board IDs for Silicon Labs targets.

Updated board IDs for Silicon Labs targets.
Because of the limitation in the mbed website. See PR #4059 in ARMmbed/mbed-os.
Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IDs look ok. Looks like the platforms haven't been renamed in the mbed site yet though. Not sure what the order of that is, but I suspect that we need to do it in the tools first. @maclobdell or @screamerbg any input here?

@bridadan
Copy link
Contributor

And just for reference, here is the PR in mbed OS that renames the boards: ARMmbed/mbed-os#4059

@screamerbg
Copy link
Contributor

We are aligned with Steven

@stevew817
Copy link
Contributor Author

stevew817 commented Mar 29, 2017

@bridadan These targets aren't public on the website yet, which is why we can get away with renaming them :)

@stevew817
Copy link
Contributor Author

@bridadan ARMmbed/mbed-os#4059 got merged, so this one should probably merge as well?

@bridadan
Copy link
Contributor

bridadan commented Apr 3, 2017

@stevew817 Fine with me, but I don't have merge powers 😄 That's up to @mazimkhan or @adbridge.

@mazimkhan mazimkhan merged commit 9c9641a into ARMmbed:master Apr 3, 2017
@stevew817 stevew817 deleted the siliconlabs/series1config2 branch April 3, 2017 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants