Skip to content

More bug fixes #37

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

grueni75
Copy link
Contributor

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

@mbruner63
Copy link

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:

 case MSG_OPT_ACCEL_STREAMING: 
    ENTER_STANDBY_MODE();
    Control &= ~WUF_ENABLE;
    Control |= DRDYE_DATA_AVAILABLE;
    AccelerometerWrite(KIONIX_CTRL_REG1, &Control, ONE_BYTE);  
    ENTER_OPERATING_MODE();
// Enable Accelerometer
    ACCELEROMETER_INT_ENABLE();
    break;

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?

Gruenewald, Matthias added 4 commits August 2, 2013 09:15
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.
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.

4 participants