FreeRTOS / FreeRTOS-Kernel

FreeRTOS kernel files only, submoduled into https://github.com/FreeRTOS/FreeRTOS and various other repos.
https://www.FreeRTOS.org
MIT License
2.71k stars 1.11k forks source link

Add config option for event groups and stream buffers #994

Closed kar-rahul-aws closed 6 months ago

kar-rahul-aws commented 7 months ago

Description

This PR adds config option for event group and stream buffer functionality.

  1. Set configUSE_STREAM_BUFFERS = 1 to enable stream buffers.
  2. Set configUSE_EVENT_GROUPS = 1 to enable event groups.

Test Steps

MPU demo on STM32H743ZI builds and runs successfully.

Checklist:

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

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 82.38342% with 34 lines in your changes are missing coverage. Please review.

Project coverage is 93.00%. Comparing base (39dbff7) to head (cea8ec8).

Files Patch % Lines
event_groups.c 82.38% 10 Missing and 24 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #994 +/- ## ======================================= Coverage 93.00% 93.00% ======================================= Files 6 6 Lines 3200 3200 Branches 879 879 ======================================= Hits 2976 2976 Misses 111 111 Partials 113 113 ``` | [Flag](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/994/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/994/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | `93.00% <82.38%> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kstribrnAmzn commented 6 months ago

Why are there no testing details for this change? You changes to event_groups.c has less than 90% coverage. Are you planning on adding unit test changes here?

kar-rahul-aws commented 6 months ago

I will add the unit test coverage for the file event_groups.c once this PR is merged.

sonarcloud[bot] commented 6 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud