Open
Conversation
added 4 commits
July 21, 2013 20:55
|
I tried to merge it with my changes and got stack linker issues (not sure if I did the merge operation right). Does anyone know if the accelerometer streaming function was working? I found that P1IE7 was not enabled. So I made this change to Accelerometer.c: Anyway, use an iphone simulator and sending the E1 command to 8881 (BLE). I get data for a while until the watch sends ConnParamIndMsg message and we lose connection. So shouldn't the ACCELEROMETER_INT_ENABLE(); be placed in that case statement? |
added 4 commits
August 2, 2013 09:15
…iting for a certain time for an event
interrupts during an ISR. However, some FreeRTOS functions like xTaskRemoveFromEventList (used by SendMessageToQueueFromIsr) require that interrupts are disabled when they are called. This occasionally resulted in wrong thread context switches. For example, the Wrapper task frooze occaisionally, resulting in watch dog resets.
Conflicts: Watch/Application/LcdDisplay.c
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
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.
Hi myang,
another set of bug fixes. One memory leak introduced by the latest new features. And two fixes for interoperation with the Android community edition of MWM. The removed GetRtc message resulted in MWM unable to update the time of the watch. And the number of messages that can be tied to the buttons of the watch were too low to handle all features of MWM. The result was that some buttons were not working in some situations.
Bye
Matthias