-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add ARM Cortex-M52 port support FreeRTOS#1305 #1334
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
Add ARM Cortex-M52 port support FreeRTOS#1305 #1334
Conversation
Fixes FreeRTOS#1305 - Add ARM_CM52 and ARM_CM52_NTZ port implementations for GCC and IAR compilers
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.
Can you please mention the compatibility of Arm Cortex-M52 with M-profile Vector Extension (MVE) in the following two files?
- https://github.com/cpussw01/FreeRTOS-Kernel/blob/feature/add-cortex-m52-support/examples/template_configuration/FreeRTOSConfig.h#L609
- https://github.com/cpussw01/FreeRTOS-Kernel/blob/feature/add-cortex-m52-support/portable/ThirdParty/GCC/ARM_TFM/README.md
Also I think it's worth adding Arm Cortex-M52 to the list of platforms that can call security services in TF-M through PSA APIs in here.
Thank you.
|
Since an example was created during the port validation, could you please open a PR for the Cortex-M52 demo? Thank you |
To make the work more clear. |
Yes, that's exactly what I meant. |
Thank you . Here I want explain and clarify something: Totally, I plan to develop a Coretx-M52 demo target for FVP_MPS2_Cortex-M52, and submit to FreeRTOS-Partner-Supported-Demos repo. But it take some times. |
No, the demo is not a blocker for this PR but it'd be great to add the demo whenever you can so that users and developers who are interested in using Arm Cortex-M52 with FreeRTOS-Kernel can have a reference example. Thanks |
Get it. I will do this work later. |
|
I find it that .github/scripts/kernel_checker.py execute failed. So , Do i need to modified this file to pass the Kernel-Checker item? |
|
Hi @cpussw01, Thank you for your contribution! Per https://www.freertos.org/Documentation/02-Kernel/03-Supported-devices/03-Official-vs-3rd-party, this PR should be opened to https://github.com/FreeRTOS/FreeRTOS-Kernel-Community-Supported-Ports (seeing as their is no demo and we have not been able to test this on our side). Once a demo project is created, we can consider doing a full review and adding it along officially supported ports. Best, Jason Carroll |
Hi Jason |
Assuming that this change is from ARM, is it possible to use the same license header as in port.c: * Copyright 2024-2025 Arm Limited and/or its affiliates
* <open-source-office@arm.com> |
I discussed this with @aggarg, and since this is pretty much a replica of the M55 port, we agreed to proceed with merging here. Just fix the headers as @aggarg outlined and we can merge. Thanks for your contribution! |
Hi aggarg
|
|
Hello, any things updated ? Or anything I can do for this PR request? |
|
@cpussw01 Nothing is needed from your end. I'll get back to you on this. |
Signed-off-by: Gaurav Aggarwal <[email protected]>
Signed-off-by: Gaurav Aggarwal <[email protected]>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1334 +/- ##
=======================================
Coverage 91.46% 91.46%
=======================================
Files 6 6
Lines 3256 3256
Branches 899 899
=======================================
Hits 2978 2978
Misses 132 132
Partials 146 146
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Gaurav Aggarwal <[email protected]>
Signed-off-by: Gaurav Aggarwal <[email protected]>
|
|
Thank you @cpussw01 for your contribution! |



Fixes #1305
Description
Changes:
CMakeLists.txt
portable/ARMv8M/ReadMe.txt
portable/ARMv8M/copy_files.py
portable/ARMv8M/non_secure/ReadMe.txt
portable/ARMv8M/secure/ReadMe.txt
portable/CMakeLists.txt
Add ports for Cortex-M52 ,there are copied form correspond CM55 dir, only portmacro.h CPU_NAME changed
portable/ARMv8M/non_secure/portable/GCC/ARM_CM52/
portable/ARMv8M/non_secure/portable/IAR/ARM_CM52/
portable/GCC/ARM_CM52/
portable/GCC/ARM_CM52_NTZ/
portable/IAR/ARM_CM52/
portable/IAR/ARM_CM52_NTZ/
Test Steps
Reference FreeRTOS/Demo/ThirdParty/Partner-Supported-Demos/CORTEX_M85_MPU_PXN_FVP_GCC/ Cortex-M85 demo
Create an Cortex-M52 demo , compile + build and test on Cortex-M52 device
Checklist:
Related Issue
#1305
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.