-
Notifications
You must be signed in to change notification settings - Fork 178
Migration: New porting guide from Cloud team #619
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
Migration: New porting guide from Cloud team #619
Conversation
I can at least rebase/filter-branch out the directory capitalization for you |
New porting guide for porting Mbed OS before porting Mbed Cloud
7f7299b
to
a2ba887
Compare
a2ba887
to
541fde7
Compare
8edef02
to
8ab77cc
Compare
Alright, I have updated the engineering content, removed passive vioce where possible and re-did the formatting to match other documents in this repo. Now: how can we merge this with the other tutorial in this repo. |
I suggest we wait for Amanda. She's back on Monday. Thank you for all your help! |
@AnotherButler .. ping (hoping you are back in office)! |
@ashok-rao I'm reviewing this, and then I may move it into our porting section. I hope to have my edits complete tomorrow morning my time. |
Make initial edits for active voice, spacing and style.
@@ -0,0 +1,225 @@ | |||
## Mbed OS porting guide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any of the images on this page. Do we want to include them here or delete them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The last commit replaced all of them with the single tree structure at the bottom of the document.
- Requires the macro `CPU_DEVICE_1` to compile the vendor-provided HAL for the device mounted on the board. | ||
- Requires the macro `FAMILY_MY_FAMILY` to compile the vendor-provided HAL for the family of the device. | ||
|
||
All of these requirements directly map to relevant tags in the `targets.json` entry for our target. This is shown in step 3 below. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: We don't have a step 3 anymore. What does this reference?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following step.
@AnotherButler I think we can remove the "Amanda is on holiday. please hold." label (which I incidentally find funny). |
Oh I put that there... |
Add content to this page, so all porting target content is in one place.
Closing in favor of #625 |
Original text from @jen3andruska: