cyw43439: HCI implementation #271
Merged
Conversation
This file contains 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
|
This PR is now working once soypat/cyw43439#50 is merged into soypat/cyw43439#47 and that is merged into main. $ tinygo flash -size=short -target=pico -tags="cyw43439" -monitor -ldflags="-X main.DeviceAddress=48:9E:47:66:90:8B" ./examples/heartrate-monitor/
code data bss | flash ram
432504 16356 3352 | 448860 19708
Connected to /dev/ttyACM0. Press Ctrl-C to exit.
Enabling CYW43439 device
Enabled CYW43439 device
scanning...
found device: 48:9E:47:66:90:8B -70 Pixel 7a
connected to 48:9E:47:66:90:8B
discovering services/characteristics
found service 0000180d-0000-1000-8000-00805f9b34fb
found characteristic 00002a37-0000-1000-8000-00805f9b34fb
data: 90
data: 90
data: 90
data: 91
data: 91
data: 92
... |
deadprogram
changed the title
deadprogram
force-pushed
the
hci-transport-cyw43439
branch
2 times, most recently
from
e7a1c5c
to
1f725d8
Compare
|
The |
deadprogram
force-pushed
the
hci-transport-cyw43439
branch
from
1f725d8
to
18ba939
Compare
|
Added docs. NOW it is ready 😸 |
aykevl
reviewed
Jun 10, 2024
adapter_cyw43439.go
Outdated
| // Enable configures the BLE stack. It must be called before any | ||
| // Bluetooth-related calls (unless otherwise indicated). | ||
| func (a *Adapter) Enable() error { | ||
| time.Sleep(2 * time.Second) |
There was a problem hiding this comment.
That's pretty long! Is this really required?
There was a problem hiding this comment.
Actually I can remove this entirely.
aykevl
reviewed
Jun 10, 2024
Signed-off-by: deadprogram <[email protected]>
Signed-off-by: deadprogram <[email protected]>
Signed-off-by: deadprogram <[email protected]>
deadprogram
force-pushed
the
hci-transport-cyw43439
branch
from
3aa01ff
to
b0fd7f1
Compare
|
Thanks for review @aykevl changes made so now merging. |
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.
This PR is work in progress on support for the CYW43439 as used on the Raspberry Pi RP2040-W
To compile:
UPDATE: this is now working! Just awaiting some updates to https://github.com/soypat/cyw43439 and it can be completed.