-
Notifications
You must be signed in to change notification settings - Fork 136
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
cyw43439: HCI implementation #271
Conversation
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
... |
e7a1c5c
to
1f725d8
Compare
The |
1f725d8
to
18ba939
Compare
Added docs. NOW it is ready 😸 |
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.
Some nits, but the code looks clean.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's pretty long! Is this really required?
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.
Actually I can remove this entirely.
Signed-off-by: deadprogram <[email protected]>
Signed-off-by: deadprogram <[email protected]>
Signed-off-by: deadprogram <[email protected]>
3aa01ff
to
b0fd7f1
Compare
Thanks for review @aykevl changes made so now merging. |
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:/soypat/cyw43439 and it can be completed.