Skip to content

Conversation

@shiva-ti
Copy link
Collaborator

  • create new Security_overview.rst as the landing page for security documentation
  • provide comprehensive security framework details and features overview
  • add references to Security_overview in platform TOC files (AM62X, AM62AX, AM62PX, AM62LX)
  • add cross-reference links in related security documentation

@shiva-ti
Copy link
Collaborator Author

Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

also checkout the errors reported by vale at https://github.com/TexasInstruments/processor-sdk-doc/pull/533/files

.. figure:: ./images/security_framework.png

These security domains create a chain of trust protecting the
|__PART_FAMILY_DEVICE_NAMES__| from boot through runtime and storage,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just a very small nitpick: I think you should either remove the "the" before the device name or add "SoC" after the device name.

Everything else is okay.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thanks for pointing out, updated it

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks, LGTM

- create new Security_overview.rst as the landing page for security
documentation
- provide comprehensive security framework details and features
overview
- add references to Security_overview in platform TOC files (AM62X,
AM62AX, AM62PX, AM62LX)
- add cross-reference links in related security documentation

Signed-off-by: Shiva Tripathi <[email protected]>
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.

7 participants