-
Notifications
You must be signed in to change notification settings - Fork 3k
K64F: Add MMCAU driver needed for mbedtls #4078
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
K64F: Add MMCAU driver needed for mbedtls #4078
Conversation
cc @MartinLatal |
All the mbedtls related files should be in the features target, and not the mbed-os target (except entropy), so these files should be added to : features/mbedtls/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/drivers/ and not targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/drivers/ |
Thank you. All the drivers associated with the hardware IP blocks are placed in the HAL drivers folder. |
Signed-off-by: Mahadevan Mahesh <[email protected]>
8fb8d2f
to
b700ea3
Compare
Renamed the ARM library file to use the .ar extension. |
I restarted travis. Failed for not related change, lets review again once it finishes.
As these files are driver files, and those reside in hal under MCUXpresso folder, all good with this PR? Please review (there's one more PR for another targets). |
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.
Personally I would put the mbedtls related files in the mbedtls targets folder, but since all of the driver files are in the HAL I would keep them here as well
retest uvisor |
/morph test-nightly |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
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 fine to me. Of course I did not review the binaries.
@sg- @mmahadevan108 will the outcome of this workshop eventually make it into master? FYI: @MarceloSalazar Organization internal: @markus-becker-tridonic-com @andreas-myka-tridonic-com |
Description
Include the MMCAU K64 SDK driver needed to supporting mbedtls
Status
**READY