Skip to content

Conversation

@ccli8
Copy link
Contributor

@ccli8 ccli8 commented Jul 18, 2016

Add support for renamed target boards:
NUMBED_NUC472 --> NUMAKER_PFM_NUC472
NUMBED_M453 --> NUMAKER_PFM_M453

Add support for renamed target boards:
NUMBED_NUC472 --> NUMAKER_PFM_NUC472
NUMBED_M453 --> NUMAKER_PFM_M453
@PrzemekWirkus
Copy link
Contributor

@sg- Sam, please verify and we can merge + release!

@sg-
Copy link
Contributor

sg- commented Jul 20, 2016

Can confirm the IDs are correct but cannot find that the board alias names are correct. @ccli8 can you provide a link to the target.json with this board support?

@ccli8
Copy link
Contributor Author

ccli8 commented Jul 20, 2016

@sg- Do you mean target module in mbed OS 3.0? Their links are as below, but they are for old names: NUMBED_NUC472 and NUMBED_M453.
https://github.com/ARMmbed/target-nu-numbed-nuc472-armcc
https://github.com/ARMmbed/target-nu-numbed-m453-armcc

@cyliangtw
Copy link
Contributor

cyliangtw commented Jul 20, 2016

@sg- Sam, we got the notification of trademark violation from ARM on 07/B. So we revise the board name for mbed OS 5.0 and wish to apply the new IDs. So far, Eric already notified these 2 new names are fine after HQ review.

@sg-
Copy link
Contributor

sg- commented Jul 20, 2016

LGTM 👍

@sg-
Copy link
Contributor

sg- commented Jul 20, 2016

@ccli8 please just make sure the mbed OS 5 target.json file uses these specific names.

@PrzemekWirkus
Copy link
Contributor

PrzemekWirkus commented Jul 20, 2016

@ccli8 LIke @sg- said. We are tracking this but you need to make sure you are updating target.json with correct KEY with one of the canonical platform names you are using here.

Please remember there is one way 'platform name' mapping being done:
mbed-ls -> target.json -> HAL

I will merge and release to v1.1.3 ASAP!

@PrzemekWirkus PrzemekWirkus merged commit 9b2f098 into ARMmbed:master Jul 20, 2016
@PrzemekWirkus
Copy link
Contributor

Submitting dist\mbed-ls-1.1.3.zip to https://pypi.python.org/pypi
Server response (200): OK

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