[riot-notifications] [RIOT-OS/RIOT] ztimer: pull in xtimer_on_ztimer only if ztimer_periph_timer is used (#14036)

Juergen Fitschen notifications at github.com
Tue May 19 11:52:10 CEST 2020


@jue89 commented on this pull request.



> @@ -23,11 +23,11 @@ ifneq (,$(filter ztimer,$(USEMODULE)))
   endif
 endif
 
-# unless ztimer_xtimer_compat is used, make xtimer use ztimer as backend.
-ifneq (,$(filter ztimer,$(USEMODULE)))
+# unless ztimer_xtimer_compat is used, make xtimer use ztimer_usec as backend.
+ifneq (,$(filter ztimer_periph_timer,$(USEMODULE)))

IMHO, that ist too wide. I came up with this PR when I tried to multiplex `periph_rtt` and, thus, just required `ztimer_msec`. This works perfectly and doesn't interfere with `xtimer` as `ztimer` doesn't make use of `periph_timer`.

Without this PR `xtimer_on_ztimer` is pulled in anyway and creates unnecessary complexity.

-- 
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/14036#discussion_r427175505
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200519/9237eb2e/attachment.htm>


More information about the notifications mailing list