Skip to content

Update boot sequence docs #662

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

Merged
merged 4 commits into from
Aug 30, 2018
Merged

Conversation

c1728p9
Copy link
Contributor

@c1728p9 c1728p9 commented Aug 15, 2018

Add content for the updated Mbed 5 boot sequence.

Add content for the updated Mbed 5 boot sequence.
@c1728p9
Copy link
Contributor Author

c1728p9 commented Aug 15, 2018

This has a code dependency on ARMmbed/mbed-os#7794

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.

That boot sequence diagram is fantastic! Nice work!

@@ -7,13 +7,32 @@ Mbed OS provides two entry points for developers to hook into:
- `main(void)` - Default entry point. All the standard application code goes here.
- `mbed_main(void)` - Executed directly before `main`. The user can define this.

When execution reaches the entry points, a user can expect a fully initialized system that is ready to execute application code. For this to happen, the following must have occurred prior to this point:
When execution reaches the entry points, a user can expect a fully initialized system that is ready to execute application code. The Mbed OS boot sequence consists of four phases, target setup, toolchain setup, starting the RTOS and starting the Mbed application. These phases can be seen below:
Copy link
Contributor

Choose a reason for hiding this comment

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

The Mbed OS boot sequence consists of four phases,

@AnotherButler would this be a better place for a colon (:) instead of a comma (,)?

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 Yes, I like that. Neither one is wrong, but I think a colon is nicer here.

@AnotherButler
Copy link
Contributor

ping @mprse

Copy link
Contributor

@mprse mprse left a comment

Choose a reason for hiding this comment

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

👍

Update punctuation, as requested.
@AnotherButler
Copy link
Contributor

FYI @MarceloSalazar @ashok-rao

Amanda Butler added 2 commits August 29, 2018 13:46
Copy edit for active voice and formatting.
Standardize person and audience across documentation.
@AnotherButler AnotherButler merged commit 4917be3 into ARMmbed:development Aug 30, 2018
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.

5 participants