[riot-notifications] [RIOT-OS/RIOT] sys/shell/sc_nimble_netif: switch to ztimer (#16324)

Hauke Petersen notifications at github.com
Tue Apr 13 14:47:47 CEST 2021


### Contribution description
While the NimBLE pkg including `nimble_netif` is on its way to be migrated to `ztimer`, the nimble_netif shell command still depends on xtimer. This PR changes this by using `ZTIMER_MSEC` also for the `nimble_netif` shell commands.

To prevent pulling in the `ztimer_msec` module explicitly for the shell command, I rebased this PR on top of #16317, as that PR pulls in the needed dep...

### Testing procedure
Simply flash `examples/gnrc_networking` (using NimBLE -> `USEMODULE+=nimble_netif`) on any applicable Nordic board and run the `ble scan X` shell command. If the scan duration fits the given duration, everything is still in order.

### Issues/PRs references
depends on #16317
You can view, comment on, or merge this pull request online at:

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

-- 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
  * sys/shell/sc_nimble_netif: use ztimer

-- 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 sys/shell/commands/sc_nimble_netif.c (8)
    M tests/nimble_l2cap/main.c (1)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/16324.patch
https://github.com/RIOT-OS/RIOT/pull/16324.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/16324
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210413/73d41ed5/attachment.htm>


More information about the notifications mailing list