Skip to content

Conversation

@0mhu
Copy link
Contributor

@0mhu 0mhu commented Apr 28, 2024

Add GCC/ARM_CM0/mpu_wrappers_v2_asm.c and GCC/ARM_CM0/portasm.c as source files to 'freertos_kernel_port' library. This fixes the FREERTOS_PORT "GCC_ARM_CM0" CMake configuration.

Test Steps

Use Cmake to integrate FreeRTOS into a Cortex M0 project via set (FREERTOS_PORT "GCC_ARM_CM0"). It does not compile.

/usr/lib/gcc/arm-none-eabi/13.2.0/../../../../arm-none-eabi/bin/ld: .../FreeRTOS-Kernel/portable/GCC/ARM_CM0/port.c:835:(.text.SysTick_Handler+0x22): undefined reference to vClearInterruptMask'`

ulSetInterruptMask (=vClearInterruptMask) is defined in portasm.c which is not included in the build.

This PR is a fix for that.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add GCC/ARM_CM0/mpu_wrappers_v2_asm.c and GCC/ARM_CM0/portasm.c as source files to 'freertos_kernel_port' library.
This fixes the FREERTOS_PORT "GCC_ARM_CM0" CMake configuration.
@0mhu 0mhu requested a review from a team as a code owner April 28, 2024 18:01
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@moninom1 moninom1 self-requested a review April 29, 2024 07:17
@moninom1 moninom1 merged commit 30afc1a into FreeRTOS:main Apr 29, 2024
moninom1 pushed a commit to moninom1/FreeRTOS-Kernel that referenced this pull request Sep 30, 2025
This PR fixes CBMC and unit test for single core FreeRTOS in the FreeRTOS-Kernel PR - FreeRTOS#716.

- xYieldPendings and xIdleTaskHandles are now an array. Update in FreeRTOS unit test.
-  Update CBMC patches.
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.

4 participants