Skip to content
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

Update default value of configSTACK_DEPTH_TYPE in config file #987

Merged

Conversation

kar-rahul-aws
Copy link
Member

Description

This PR updates the default configuration value of configSTACK_DEPTH_TYPE in FreeRTOS.h configuration file to StackType_t.

Test Steps

Run and Build MPU demo successfully.

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

Bug reported in issue 986

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

@kar-rahul-aws kar-rahul-aws requested a review from a team as a code owner February 8, 2024 06:52
Copy link

sonarqubecloud bot commented Feb 8, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@kar-rahul-aws kar-rahul-aws merged commit bd65223 into FreeRTOS:main Feb 13, 2024
16 checks passed
@kar-rahul-aws kar-rahul-aws deleted the update_configSTACK_DEPTH_TYPE branch February 13, 2024 12:15
laroche pushed a commit to laroche/FreeRTOS-Kernel that referenced this pull request Apr 18, 2024
…e.txt (FreeRTOS#987)

* Added a link to the community supported demos repo in the demos readme.txt

* Update FreeRTOS/Demo/readme.txt

Co-authored-by: Aniruddha Kanhere <[email protected]>

* Update and rename readme.txt to readme.md

---------

Co-authored-by: Aniruddha Kanhere <[email protected]>
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.

3 participants