[riot-notifications] [RIOT-OS/RIOT] sys/ztimer: add modules to Kconfig (#15716)

Leandro Lanzieri notifications at github.com
Fri Jan 8 10:06:23 CET 2021


@leandrolanzieri commented on this pull request.



> +config HAVE_ZTIMER_BACKEND
+    bool
+    help
+        Indicates that a ztimer backend is provided.
+
+endmenu # Backends
+
+menu "Clocks"
+
+config MODULE_ZTIMER_USEC
+    bool "Microseconds"
+    depends on MODULE_ZTIMER_PERIPH_TIMER
+
+config MODULE_ZTIMER_MSEC
+    bool "Milliseconds"
+    depends on HAVE_ZTIMER_BACKEND

> So the mock clock should IMO be hidden in make menuconfig.

The problem is that if it is hidden then it can't be enabled via a `.config` file in the test application. What the user can see through `menuconfig` is the same that is configurable via a `.config` file. An extra symbol to indicate that a test is being run to make it visible only then could be added, but it seems a bit overkill. Maybe I can just move it out of this menu.

-- 
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/15716#discussion_r553822306
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210108/b90d095b/attachment.htm>


More information about the notifications mailing list