-
Notifications
You must be signed in to change notification settings - Fork 935
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
feat: add basic support for nRF51 chips to embassy-nrf #2469
Conversation
I think we could (should?) avoid adding the dependency on portable-atomic,
|
added nrf51-dk to HIL rig. Wire connected between P0.13 and P0.14 for GPIO etc testing. |
@Dirbaio Thanks! I'll add a HIL test as well as addressing the comments soon! |
5ed263b
to
db0f4a0
Compare
bender run |
Cannot deterministically guarantee the upper bound
d06d6c2
to
17098a7
Compare
17098a7
to
ee90ee1
Compare
@Dirbaio Two issues remaining:
|
that's fine. we already have some stm32's that do HIL from flash because not enough RAM too...
Fixed in probe-rs/probe-rs#2119 bender run |
duh of course it doesn't work if I don't put back the nrf51-dk in the HL rig. bender run |
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.
thank you! 🥳
TODO:
@Dirbaio would you be able to hook your nrf51-dk into the HIL setup? My own dk is fried, but I verified code is running on my microbit v1.