Skip to content

Add page about systick and tickless mode #248

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
Sep 15, 2017

Conversation

bulislaw
Copy link
Member

@AnotherButler
Copy link
Contributor

Queries:

  • Are these APIs or just modes?
  • It says these should be invisible to users. Does that mean mostly vendors will be using them? Should this go into porting?

@AnotherButler
Copy link
Contributor

@bulislaw I've added some copy edits. Please review my changes, and make sure I didn't accidentally change the content.

@bulislaw
Copy link
Member Author

Are these APIs or just modes?
It says these should be invisible to users. Does that mean mostly vendors will be using them?Should this go into porting?

They are features really, but you are probably right, I wanted to put somewhere the description of how Mbed OS works. Considering it's more useful from the vendor's point of view I'll move it to the porting guide (and rebase on top of the new engine branch).

@bulislaw bulislaw changed the base branch from 5.5 to new_engine September 12, 2017 09:45
@bulislaw
Copy link
Member Author

@AnotherButler I've rebased the doc on top of the new_engine branch, moved it to the contributing directory and added some info that make it part of the PG.

Delete unnecessary word, and change mood for consistent mood/tense and voice.
@AnotherButler
Copy link
Contributor

@bulislaw Nice work. I've added some minor copy edits.

@0xc0170 Please review, and let me know when I can merge.

@AnotherButler AnotherButler removed the request for review from 0xc0170 September 15, 2017 19:40
@AnotherButler
Copy link
Contributor

Thanks for the quick review, @c1728p9 I'd forgotten both @bulislaw and @0xc0170 were on holiday.

@AnotherButler AnotherButler merged commit 68b791c into ARMmbed:new_engine Sep 15, 2017
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.

3 participants