Skip to content
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

Zephyr port #2

Open
wants to merge 7 commits into
base: cmd_parsing
Choose a base branch
from
Open

Zephyr port #2

wants to merge 7 commits into from

Conversation

arturkow2000
Copy link

No description provided.

@arturkow2000 arturkow2000 marked this pull request as draft June 22, 2022 13:42
Make ms-tpm-20-ref compile on Zephyr.
Zephyr by default adds these flags to all components which causes
unnecessary warnings in ms-tpm-20-ref.
WolfSSL pulls in Zephyr headers which cause conflicts with some macros
defined ms-tpm-ref. Eliminating Zephyr headers from WolfSSL is not
feasible due to extensive usage of Zephyr types in public types. Neither
this can be (easily) eliminated from ms-tpm-ref due to the same reason.
The approach used is the least invasive method.
macpijan and others added 4 commits July 25, 2022 11:46
Fix macro names conflicts and compiler warnings
We keep only those functions that are called by ms-tpm. Previously
Platform_fp.h had declarations for all "platform" functions but many of
them were used only internally. Keeping stuff private gives better
distinction between ms-tpm (universal) and platform specific
implementation.
Simulation mode introduces additional test code and assertions, some of
which are invalid in non-simulation.
@arturkow2000 arturkow2000 marked this pull request as ready for review July 26, 2022 06:26
Copy link

@mgabryelski1 mgabryelski1 left a comment

Choose a reason for hiding this comment

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

I mainly check compliance in formal terms.

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