[riot-notifications] [RIOT-OS/RIOT] pkg/nimble: migrate to ztimer (#16317)

Hauke Petersen notifications at github.com
Mon Apr 12 16:18:20 CEST 2021


### Contribution description
With #15782 merged all prerequisites for migrating NimBLE from `xtimer` to `ztimer` are in. For a next step, I PRed the required NimBLE NPL changes to NimBLE (https://github.com/apache/mynewt-nimble/pull/967). These are needed in conjunction with the changes of this PR and finally allow for `xtimer`-free RIOT/NimBLE builds :-)

As stated in https://github.com/apache/mynewt-nimble/pull/967, appart from simplifying the NPL code in NImBLE, this change allows for significant energy savings. E.g. when running the `examples/nimble_gatt` application without STDIO (`USEMODULE+=stdio_null`) and the node is in connected state with a peer, the average current consumption drops from ~420µA to 72µA.

### Testing procedure
Verify that NimBLE is behaving as before. I suggest to run at least the following on any nrf52x board of you choice:
- `examples/nimble_gatt` -> connect with a smartphone (e.g. using the `nrf connect` APP on Android) and verify GATT services are working
- `examples/nimble_scanner` -> use the `scan` shell command and it should list some devices that it found
- `tests/nimble_l2cap`/`tests/nimble_l2cap_server` -> connect two boards and make sure flooding and inctest are running as expected
- `examples/gnrc_networking` -> connect two IP-over-BLE devices using the `ble` shell command and make sure they can ping each other

### Issues/PRs references
depends on https://github.com/apache/mynewt-nimble/pull/967

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/16317

-- Commit Summary --

  * TODO pkg/nimble: update to version XXX
  * pkg/nimble: switch from xtimer to ztimer
  * pkg/nimble/scanlist: switch to ZTIMER_USEC
  * pkg/nimble/scanlist: fix missing stdio.h include
  * tests/nimble_l2cap: fix missing xtimer.h include

-- File Changes --

    M pkg/nimble/Makefile (2)
    M pkg/nimble/Makefile.dep (9)
    M pkg/nimble/scanlist/nimble_scanlist.c (4)
    M pkg/nimble/scanlist/nimble_scanlist_print.c (1)
    M tests/nimble_l2cap/main.c (1)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/16317.patch
https://github.com/RIOT-OS/RIOT/pull/16317.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16317
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210412/ed51653e/attachment-0001.htm>


More information about the notifications mailing list