-
Notifications
You must be signed in to change notification settings - Fork 99
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
wolfHSM integration #511
wolfHSM integration #511
Conversation
bc32c21
to
59429a4
Compare
rebased to fix merge conflicts that arose since filing the PR (just to config files), no code changes introduced |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good. All minor feedback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! Minor nits. Update if you want. A couple of questions.
59429a4
to
5cc9950
Compare
5cc9950
to
ca85bdc
Compare
@dgarske addressed review comments and rebased to fix conflicts introduced by some of the latest PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Over to @danielinux
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, and documentation is complete. Just a minor thing, the new options in keygen.c: I see those are mentioned in the keystore doucment, they should be also breifly mentioned in signing.md, as all the command line options for the key tools are usually grouped there.
Adds wolfHSM client support to wolfBoot.
Initial support for the following targets:
wolfHSM offload has been added for the following algorithms:
Additional changes: