[riot-notifications] [RIOT-OS/RIOT] make/ztimer: auto-pull timer backend deps for ztimer_msec (#16334)

Hauke Petersen notifications at github.com
Thu Apr 15 10:52:20 CEST 2021


### Contribution description
For `ZTIMER_MSEC` we have currently a slightly screwed situation, where every (or at least many) modules that use that timer try to figure out the best underlying timer for themselves, leading to duplicated dependency definitions throughout the build system. On top one can easily run into trouble when using `ztimer_msec` in a application, but at the same time there is no fitting ztimer backend included by any other module in the build (e.g. see  #16322).

So my suggested solution is to pull in `ztimer_periph_rtt` if available, or `ztimer_periph_timer` otherwise in the case `ztimer_msec` is used.

### Testing procedure
Use `make info-modules` on all effected test/example applications an verify that the `ztimer_periph_rtt` and `periph_rtt` modules are still included as before.

### Issues/PRs references
#16322 pointed towards this problem

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

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

-- Commit Summary --

  * ztimer: pull in periph module for ZTIMER_MSEC
  * make: remove redundant deps for ztimer_periph_rtt

-- File Changes --

    M drivers/sx127x/Makefile.dep (6)
    M pkg/nimble/Makefile.dep (4)
    M pkg/openwsn/Makefile.dep (4)
    M sys/Makefile.dep (6)
    M sys/ztimer/Makefile.dep (5)

-- Patch Links --

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


More information about the notifications mailing list