Skip to content

nrf: Generalize feather52 target #204

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

Merged
merged 1 commit into from
Jun 13, 2018
Merged

nrf: Generalize feather52 target #204

merged 1 commit into from
Jun 13, 2018

Conversation

glennrub
Copy link
Collaborator

This patch generalize the feather52 target to be a board without
an in-built Bluetooth stack or bootloader giving all flash memory to
micropython code.

This way the feather52 target can run any supported Bluetooth LE
stack the port supports for other nrf52832 targets. Hence, this
make Makefiles/linker scripts and BLE driver support easier
to maintain in the future.

This patch generalize the feather52 target to be a board without
an in-built Bluetooth stack or bootloader giving all flash memory to
micropython code.

This way the feather52 target can run any supported Bluetooth LE
stack the port supports for other nrf52832 targets. Hence, this
make Makefiles/linker scripts and BLE driver support easier
to maintain in the future.
@glennrub
Copy link
Collaborator Author

This PR should unblock PR #197

@glennrub
Copy link
Collaborator Author

Tested using external JLink on the top 10-pin debug header connector.

Without BLE:

  • led1 and led2 working as normal (UART REPL)

With BLE s132v3.0.0:

  • advertisement using ubluepy (UART REPL).
  • BLE REPL, updating led1 and led2.

@aykevl
Copy link

aykevl commented Jun 13, 2018

LGTM

@glennrub glennrub merged commit b50447f into tralamazza:master Jun 13, 2018
@glennrub glennrub deleted the make_feather52_generic_target branch July 18, 2018 20:31
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