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

Deprecate ubluepy #1

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

Conversation

glennrub
Copy link

I'm moving out ubluepy as a external cmodule that can be loaded using USER_C_MODULES.
This makes me do a few changes to the current proposal, explicitly the "MICROPY_PY_BLUETOOTH ?= 1" and "RF_STACK_ENABLED". The first to disable BLUETOOTH from micropython, but not the SoftDevice support as this is still needed if a Radio stack is hosted outside. The latter to override the "is_stack_enabled" in drivers like flash/random/temp. This is also applicable if someone would like to make a Radio module outside mainline source, and still utilize these drivers.

aykevl and others added 8 commits March 11, 2019 19:55
Rebased to master with the help of Mirko Vogt.
This is useful for more advanced use cases, like iBeacon/Eddystone.
At the moment, only a service is created. Characteristics within a
service cannot yet be added.
128-bit proprietary UUIDs can be passed using strings. Dashes and
capitalization are ignored.
Add support for adding characteristics to services. They can be read and
written on both the esp32 and the nrf. Events of any kind (notifications
etc.) haven't been implemented yet.
Removing nrf-port Bluetooth LE drivers and micropython
wrapper objects/modules. Also, deprecating the legacy
examples using ubluepy.
@glennrub
Copy link
Author

Created https://github.com/glennrub/ubluepy -> clone to "cmodules"

make BOARD=pca10056 USER_C_MODULES=cmodules SD=s140 MICROPY_PY_BLUETOOTH=0
This shows s140 still being enabled and linker scripts being used, but custom Radio stack driver being used.

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.

2 participants