Merged
Conversation
Replaced RTX-specific code with CMSIS-RTOS RTX (including config file, service calls, task definitions, data types, datastructures etc.) Added CMSIS to the Keil project Updated ASF Module to call CMSIS APIs Added exit function so that the code compiles with Microlib Reduced Heap size for optimal memory usage Readjusted Stack size based on new stack requirement (after adding the CMSIS code) Updated Application modules for ASF and CMSIS changes Reduced number of system messages for optimal memory usage Added const to the ASF_TASK_ARG definition since arguments to a task/thread should not be modified Changed data type of priority to signed since priority can be negative in CMSIS Removed the function _TimerStart() since it is not used Added Task ready flag in sensor acquisition task to avoid queue overflow if messages are sent before the task is initialized Signed-off-by: Pankaj Goenka <pgoenka@audience.com>
Open
pankajgoenka
added a commit
that referenced
this pull request
Aug 18, 2015
Feature OSAL CMSIS: Merging code changes for incorporating OS abstraction layer(CMSIS), as approved by Rian.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closed CMSIS related review comments for the commit aa5c1a2.
Click here for the original pull request.
Please review the updated code.