Skip to content

Conversation

kstribrnAmzn
Copy link
Member

Add atomic flag for 16-bit ticks

Description

This change allows the PIC24 family of
16 bit processors to read the tick count
without a critical section when the tick
count is also 16 bits.

Test Steps

Related Issue

Inspired from discussion - https://forums.freertos.org/t/xtaskgettickcount-with-critical-section-on-16-bit-mcu/14860/5

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

This change allows the PIC24 family of
16 bit processors to read the tick count
without a critical section when the tick
count is also 16 bits.

Inspired from discussion - https://forums.freertos.org/t/xtaskgettickcount-with-critical-section-on-16-bit-mcu/14860/5
@kstribrnAmzn kstribrnAmzn requested a review from a team as a code owner April 14, 2022 20:04
@codecov
Copy link

codecov bot commented Apr 14, 2022

Codecov Report

Merging #488 (63204dc) into main (d5b95c9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #488   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files           4        4           
  Lines        1270     1270           
  Branches      343      343           
=======================================
  Hits         1174     1174           
  Misses         50       50           
  Partials       46       46           
Flag Coverage Δ
unittests 92.44% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5b95c9...63204dc. Read the comment docs.

@kstribrnAmzn kstribrnAmzn merged commit 15bc866 into FreeRTOS:main Apr 15, 2022
@kstribrnAmzn kstribrnAmzn deleted the atomicTickPic24 branch April 15, 2022 15:14
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