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

Support for Infineon TriCore TC2XX/TC3XX #229

Merged
merged 2 commits into from
Dec 22, 2022

Conversation

dgarske
Copy link
Contributor

@dgarske dgarske commented Jul 15, 2022

Using macro WOLFTPM_INFINEON_TRICORE.

@dgarske dgarske self-assigned this Jul 15, 2022
@dgarske
Copy link
Contributor Author

dgarske commented Sep 23, 2022

@Anthony-Tatowicz : When you have some spare cycles would you mind reviewing this code to see if you think it will build okay for the TC2X? No urgency.

Copy link
Contributor

@tatowicz tatowicz left a comment

Choose a reason for hiding this comment

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

Looks good

@dgarske
Copy link
Contributor Author

dgarske commented Dec 21, 2022

Looks good

Have you been able to build these sources? I coded this up but wasn't able to try building it. The customer I sent it to seemed to like it, but I never did get any coding feedback.

@dgarske
Copy link
Contributor Author

dgarske commented Dec 21, 2022

Looks good

Have you been able to build these sources? I coded this up but wasn't able to try building it. The customer I sent it to seemed to like it, but I never did get any coding feedback.

Do you think this could work for any Infineon Tri-Core (TC2XX or TC3XX)? If so I'll update the macro and notes to expand the coverage.

@tatowicz
Copy link
Contributor

Looks good

Have you been able to build these sources? I coded this up but wasn't able to try building it. The customer I sent it to seemed to like it, but I never did get any coding feedback.

Do you think this could work for any Infineon Tri-Core (TC2XX or TC3XX)? If so I'll update the macro and notes to expand the coverage.

Yes It will work for both the only way this would break is an iLLD update changing the API

@tatowicz
Copy link
Contributor

Looks good

Have you been able to build these sources? I coded this up but wasn't able to try building it. The customer I sent it to seemed to like it, but I never did get any coding feedback.

Yes, these sources build with Tricore, however to get the full TPM building with tasking takes some extra configuration!

@tatowicz
Copy link
Contributor

But yes expand coverage to 3xx

@dgarske dgarske removed their assignment Dec 22, 2022
@dgarske dgarske changed the title Experimental support for Infineon TC27X Experimental support for Infineon TriCore TC2XX/TC3XX Dec 22, 2022
@dgarske dgarske changed the title Experimental support for Infineon TriCore TC2XX/TC3XX Support for Infineon TriCore TC2XX/TC3XX Dec 22, 2022
@tatowicz tatowicz merged commit 04908e5 into wolfSSL:master Dec 22, 2022
@dgarske dgarske deleted the infineon_tc27x branch December 22, 2022 01:25
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.

2 participants