Skip to content

PSA-SPM documentation follow-up #8873

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 6 commits into from
Dec 4, 2018
Merged

PSA-SPM documentation follow-up #8873

merged 6 commits into from
Dec 4, 2018

Conversation

orenc17
Copy link
Contributor

@orenc17 orenc17 commented Nov 26, 2018

Description

  • Fix Doxygen
  • Remove outdated doc folder (replaced by mbed-os handbook and PSA-FF)

depends os #8744

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[X] Docs update
[ ] Test update
[ ] Breaking change

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 28, 2018

travis-ci/psa-autogen Pending — Local psa-autogen testing is in progress

CAn you review this job? Looks like it makes travis to fail

@orenc17
Copy link
Contributor Author

orenc17 commented Nov 28, 2018

Creating a separate PR for that in a minute #8889

* It basically generates and exposes a unique handle identifier [handle] per
* handle memory [handle_mem] it receives from the user.
/*
* It generates and exposes a unique handle identifier (handle) per
Copy link
Contributor

Choose a reason for hiding this comment

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

The Handle Manager module generates and exposes a unique...

* Handles generation is done exclusively.
* Once we got a handle, removing a handle or getting its memory can be
* Handle generation is done exclusively.
* Once you got a handle, removing a handle or getting its memory can be
Copy link
Contributor

Choose a reason for hiding this comment

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

Once you have a handle, you can remove or get its memory non-exclusively.

* @param[in] handle_mem A pointer to a pre-allocated handle memory to get a handle identifier for
* @param[in] friend_pid The partition id which is allowed to get_mem() and destroy() in addition to the handle owner.
* @param[in] friend_pid The partition id which is allowed to `get_mem()` and `destroy()` in addition to the handle owner.
Copy link
Contributor

Choose a reason for hiding this comment

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

partition ID

@AnotherButler
Copy link
Contributor

@cmonr Any way we can get this for RC2 instead of 5.11.1?

@AnotherButler
Copy link
Contributor

Thanks @0xc0170 👍

@cmonr cmonr dismissed melwee01’s stale review December 3, 2018 16:54

Comments addressed.

@cmonr
Copy link
Contributor

cmonr commented Dec 3, 2018

@ARMmbed/mbed-docs All good?

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 4, 2018

@melwee01 I am reviewing now, can you also please? we need to start CI asap

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 4, 2018

@melwee01 Be careful with spaces (there was one space left in the latest edit), I fixed it in 6ae3501 (astyle was failing)

Ready for CI

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 4, 2018

CI started

@mbed-ci
Copy link

mbed-ci commented Dec 4, 2018

Test run: SUCCESS

Summary: 4 of 4 test jobs passed
Build number : 1
Build artifacts
Build logs

@0xc0170 0xc0170 merged commit 6edc81d into ARMmbed:master Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants