-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[BEETLE] Update BLE stack on Beetle board #3046
Conversation
I extensively tested it on Beetle and the results are attached.
FYI: @fvincenzo |
@mmorenobarm Thanks for the PR! /morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 0 All builds and test passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
We're not making releases against the 5.1-branch but in process of releasing 5.2-branch. Backporting only happens to a single branch at this point, in this case, its 5.2. |
@pan- LGTM? |
@mmorenobarm I got compilation issues with IAR:
|
I've tested with IAR 7.4 and the compilation fail because |
Yes, that is what I mentioned in my previous comment. |
Ok, IAR versions below 7.70 are not supported. Please forgot my previous comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@mmorenobarm Why the specific version of compiler is required? This does not sound right. Please elaborate. Can you provide more details? |
This patch updates DualTimer_DisableInterrupt to disable the interrupt timers individually. In addition, it updates lp_ticker accordingly. Signed-off-by: Marc Moreno <[email protected]>
This patch updates the BLE Cordio libraries. Signed-off-by: Marc Moreno <[email protected]>
@0xc0170 The idea is to know from which version the stdint.h file can be included, otherwise the standard types needs to be defined. The stdint.h file was introduced in C99. However, It seems the C99 is on by default in mbed. So, I decided to remove the checks. |
Added below the test results with the latest changes.
|
Thanks, one last CI run, and good to integrate |
/morph test |
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 949 All builds and test passed! |
Ports for Upcoming Targets Fixes and Changes 2966: Add kw24 support ARMmbed/mbed-os#2966 3068: MultiTech mDot - clean up PeripheralPins.c and add new pin names ARMmbed/mbed-os#3068 3089: Kinetis HAL: Remove clock initialization code from serial and ticker ARMmbed/mbed-os#3089 2943: [NRF5] NVIC_SetVector functionality ARMmbed/mbed-os#2943 2938: InterruptIn changes in NCS36510 HAL. ARMmbed/mbed-os#2938 3108: Fix sleep function for NRF52. ARMmbed/mbed-os#3108 3076: STM32F1: Correct timer master value reading ARMmbed/mbed-os#3076 3085: Add LOWPOWERTIMER capability for NUCLEO_F303ZE ARMmbed/mbed-os#3085 3046: [BEETLE] Update BLE stack on Beetle board ARMmbed/mbed-os#3046 3122: [Silicon Labs] Update of Silicon Labs HAL ARMmbed/mbed-os#3122 3022: OnSemi RAM usage fix ARMmbed/mbed-os#3022 3121: STM32F3: Correct UART4 and UART5 defines when using DEVICE_SERIAL_ASYNCH ARMmbed/mbed-os#3121 3142: Targets- NUMAKER_PFM_NUC47216 remove mbed 2 ARMmbed/mbed-os#3142
This pull request contains a set of patches to update the BLE stack on Beetle board.
The first patch is required to run the tests correctly.
The second patch updates the BLE stack.
This PR needs to be backported to 5.1 as well.