ble-nrf51822 icon indicating copy to clipboard operation
ble-nrf51822 copied to clipboard

Published 20 hours ago • verified publisher icon ARMmbed

V10 helper func

Open LiyouZhou opened this issue 9 years ago • 9 comments

Replace helper functions provided by ble-nrf51822 with ones provided by nordic sdk v10. Depends on https://github.com/ARMmbed/nrf51-sdk/pull/13.

@pan- Please review.

LiyouZhou avatar Dec 24 '15 12:12 LiyouZhou

bump

LiyouZhou avatar Jan 19 '16 15:01 LiyouZhou

I didn't forgot about these, it is just that I need time to analyze what is going on on the previous implementation and the new one. I will close this soon

pan- avatar Jan 19 '16 16:01 pan-

how's this gonna go now that there's an nrf52 sdk module?

ghost avatar Feb 16 '16 07:02 ghost

Automatic CI verification build not done, please verify manually.

ciarmcom avatar Feb 16 '16 07:02 ciarmcom

NRF52 doesn't change anything, the Nordic SDK still exist, softdevice still exist and NRF51 still exist. Nordic has made a nice work to make SDK v10 and SDK v11 compatible. It should not be an issue.

pan- avatar Feb 16 '16 12:02 pan-

sure, but there's a direct dependency on the nrf51 sdk yotta module, but no dependency on the nrf52-sdk yotta module

ghost avatar Feb 16 '16 12:02 ghost

The nrf52 port is far from ready at the moment. Later, I think we will have an nrf-sdk repository which encompass nordic SDK for nrf51 and nrf52. There is no difference between these two. At some point, I also think that this repository (ble-nrf51822) will become ble-nrf5X and will be used by nrf51 and nrf52 platform.

pan- avatar Feb 16 '16 12:02 pan-

@pan- What is your plan for this?

LiyouZhou avatar Feb 22 '16 10:02 LiyouZhou

I won't merge it now, I need time to understand all the implications of this change. But I did not forget about it.

pan- avatar Feb 22 '16 10:02 pan-