Skip to content

Update the FreeRTOS-Kernel submodule to V10.6.1 #1013

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Update the FreeRTOS-Kernel submodule to V10.6.1 #1013

wants to merge 2 commits into from

Conversation

Skptak
Copy link
Member

@Skptak Skptak commented Aug 30, 2023 •
edited
Loading

Description

Update the FreeRTOS-Kernel submodule to V10.6.1
Add the manifest verification to the workflow as well

Test Steps

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.

@Skptak Skptak requested a review from a team as a code owner August 30, 2023 23:20
amazonKamath
amazonKamath previously approved these changes Aug 31, 2023
View reviewed changes
moninom1
moninom1 reviewed Aug 31, 2023
View reviewed changes
@moninom1
Copy link
Member

/bot run uncrustify

@chinglee-iot
Copy link
Member

FreeRTOS-Kernel/ThirdParty/GCC/Posix/portmacro.h is used before ConfigFiles/portmacro.h.
I think this causes xMPU_SETTINGS is undefined since it is defined in ConfigFiles/portmacro.h.
We can adjust the include order to see if the CI problem is solved.

The processed output of mock_task.c.

# 1 "/mnt/c/msys64/home/chinglee/plus_tcp/FreeRTOS-Plus-TCP/test/FreeRTOS-Kernel/portable/ThirdParty/GCC/Posix/portmacro.h" 1
# 31 "/mnt/c/msys64/home/chinglee/plus_tcp/FreeRTOS-Plus-TCP/test/FreeRTOS-Kernel/portable/ThirdParty/GCC/Posix/portmacro.h"
#define PORTMACRO_H

@kar-rahul-aws kar-rahul-aws mentioned this pull request Aug 31, 2023
Merged
2 tasks
@Skptak Skptak changed the title Update the FreeRTOS-Kernel submodule to V10.6.0 Update the FreeRTOS-Kernel submodule to V10.6.1 Aug 31, 2023
@Skptak Skptak force-pushed the SubmoduleUpdate branch 4 times, most recently from 7743047 to 92865b4 Compare August 31, 2023 21:07
Update the FreeRTOS-Kernel submodule to v10.6.0, update the tests to …
1dfb997 
…account for this, include the manifest checker as well
@Skptak Skptak closed this Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@moninom1 moninom1 moninom1 left review comments

@amazonKamath amazonKamath amazonKamath left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Skptak @moninom1 @chinglee-iot @amazonKamath