Skip to content

Conversation

kotkcy
Copy link
Contributor

@kotkcy kotkcy commented Sep 11, 2019

This PR updates the Cypress PSoC 6 crypto target for Mbed TLS library resides in Mbed OS 5.13:

  • Updated implementation of the Cypress hardware acceleration target for mbedTLS (features/mbedtls/targets/TARGET_Cypress) - renamed all directories from PSOC6_xx to MXCRYPTO_xx
  • Updated targets.json file to use new mbedTLS hardware acceleration scheme for Cypress MCUs (targets/targets.json)

This PR is based on PR #11046, so actual changes only related to crypto acceleration support in MbedTLS library.

Pull request type

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

Reviewers

Please suggest

Release Notes

@ciarmcom ciarmcom requested review from a team September 12, 2019 01:00
@ciarmcom
Copy link
Member

@kotkcy, thank you for your changes.
@ARMmbed/mbed-os-maintainers @ARMmbed/mbed-os-crypto please review.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 12, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Sep 12, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@kotkcy
Copy link
Contributor Author

kotkcy commented Sep 12, 2019

@0xc0170 Please merge this PR asap.

@OPpuolitaival
Copy link
Contributor

Retriggered CI because of testing it a bit

@mbed-ci
Copy link

mbed-ci commented Sep 13, 2019

Test run: FAILED

Summary: 3 of 4 test jobs failed
Build number : 2
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_build-IAR
  • jenkins-ci/mbed-os-ci_build-ARM
  • jenkins-ci/mbed-os-ci_build-GCC_ARM

@OPpuolitaival
Copy link
Contributor

There was CI internal problems..fixed and updating this checks status soon

@0xc0170 0xc0170 merged commit bf1aa5c into ARMmbed:master Sep 13, 2019
@kotkcy kotkcy deleted the pr/mxcrypto_alts branch September 13, 2019 17:28
@kotkcy kotkcy restored the pr/mxcrypto_alts branch September 13, 2019 18:21
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