Skip to content
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

stm32/btstack: Add support for btstack on WB55 / rfcore. #9309

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andrewleech
Copy link
Sponsor Contributor

This MR allows building stm32wb55 boards with MICROPY_BLUETOOTH_NIMBLE = 0 / MICROPY_BLUETOOTH_BTSTACK = 1

@andrewleech andrewleech marked this pull request as draft September 14, 2022 07:53
@@ -413,9 +411,10 @@ STATIC size_t tl_process_msg(volatile tl_list_node_t *head, unsigned int ch, par
bool added_to_free_queue = false;
size_t len = 0;
while (cur != head) {
volatile tl_list_node_t *next = tl_list_unlink(cur);

Copy link
Sponsor Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change (moving the line up) is what solved my HardFault...
In the processing of tl_parse_hci_msg() below a new message might be sent which changes the cur / head of ipcc_mem_ble_evt_queue
"snapshotting" the cur / next here before processing preserves them. Indeed removing it from the linked list before processing perhaps makes sense? Or might be dangerous, I'm not entirely certain...

@projectgus
Copy link
Contributor

This is an automated heads-up that we've just merged a Pull Request
that removes the STATIC macro from MicroPython's C API.

See #13763

A search suggests this PR might apply the STATIC macro to some C code. If it
does, then next time you rebase the PR (or merge from master) then you should
please replace all the STATIC keywords with static.

Although this is an automated message, feel free to @-reply to me directly if
you have any questions about this.

tannewt pushed a commit to tannewt/circuitpython that referenced this pull request Jun 17, 2024
Update readme to show the support status of the new esps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants