Closed robyoung closed 3 years ago
I am not seeing the flip-link errors from CI when I run this locally but I am seeing this from the ble example. I don't think we'll be able to upgrade the nrf-hals to0.14
just yet.
error[E0308]: mismatched types
--> examples/ble-beacon/src/main.rs:29:39
|
29 | let mut radio = BleRadio::new(p.RADIO, &p.FICR, BLE_TX_BUF, BLE_RX_BUF);
| ^^^^^^^ expected struct `nrf51::RADIO`, found struct `microbit::nrf51_pac::RADIO`
error[E0308]: mismatched types
--> examples/ble-beacon/src/main.rs:29:48
|
29 | let mut radio = BleRadio::new(p.RADIO, &p.FICR, BLE_TX_BUF, BLE_RX_BUF);
| ^^^^^^^ expected struct `nrf51::FICR`, found struct `FICR`
|
= note: expected reference `&nrf51::FICR`
found reference `&FICR`
For more information about this error, try `rustc --explain E0308`.
The rubble/ble issue seems more complicated than just updating the nrf-hal (see the linked pr above)
A possible temporary fix for better usage of the discovery book would be to ignore the ble-example failure.
Rubble is updated but not yet released. We could possibly update the dev-dependency of rubble to the current git master which would fix everything else. The alternative is to wait for their next release.
Oops, I completely forgot that this was what this PR was about and addressed this in #70 to unblock #69
@caemor I could have saved myself a lot of time if I'd looked here first because I basically exactly what you suggested 😫
⚠️ This does not currently work ⚠️ I'm leaving this open as somewhere to track progress.