Merge pull request #17284 from fjmolinas/pr_ztimer_no_periph_rtt

sys/ztimer: add 'ztimer_no_periph_rtt'

drivers/include/periph/rtt.h

@@ -22,6 +22,10 @@
  * On many CPUs, certain power states might need to be blocked in rtt_init(), so
  * that it is ensured that the RTT will function properly while it is enabled.
  *
+ * @warning     This module will be automatically be used as a backend for
+ *              ZTIMER_SEC and ZTIMER_MSEC. If direct access to RTT is needed
+ *              then include `ztimer_no_periph_rtt` to avoid auto-selection,
+ *              i.e.: `USEMODULE += ztimer_no_periph_rtt`.
  * @{
  * @file
  * @brief       Low-level RTT (Real Time Timer) peripheral driver interface

sys/include/ztimer.h

@@ -197,6 +197,11 @@
  *              if if these are missing it will use same basic timer
  *              as ZTIMER_USEC does.
  *
+ * If `periph_rtt` is required with direct access by another MODULE or
+ * application, `ztimer_no_periph_rtt` can be included to avoid automatic
+ * selection of `ztimer_periph_rtt` as a backend for ZTIMER_SEC and ZTIMER_MSEC.
+ * i.e.: `USEMODULE += ztimer_no_periph_rtt`.
+ *
  * These pointers are defined in `ztimer.h` and can be used like this:
  *
  *     ztimer_now(ZTIMER_USEC);

sys/ztimer/Kconfig

@@ -10,6 +10,12 @@ menu "ztimer - High level timer abstraction layer"
 config ZTIMER_CUSTOM_BACKEND_CONFIGURATION
     bool "Override default backend selection"
 
+config MODULE_ZTIMER_NO_PERIPH_RTT
+    bool "Disable ztimer_periph_rtt auto-inclusion"
+    help
+        This module disables the auto-inclusion of ztimer_periph_rtt as a backend
+        for ztimer_msec and ztimer_sec.
+
 menu "Backends"
     visible if ZTIMER_CUSTOM_BACKEND_CONFIGURATION
 
@@ -56,7 +62,7 @@ config MODULE_ZTIMER_MSEC
 choice
     bool "Backend"
     depends on MODULE_ZTIMER_MSEC
-    default ZTIMER_MSEC_BACKEND_RTT
+    default ZTIMER_MSEC_BACKEND_RTT if !MODULE_ZTIMER_NO_PERIPH_RTT
 
 config ZTIMER_MSEC_BACKEND_TIMER
     bool "Timer"
@@ -85,7 +91,7 @@ config ZTIMER_SEC_BACKEND_TIMER
 config ZTIMER_SEC_BACKEND_RTT
     bool "RTT"
     depends on HAS_PERIPH_RTT
-    select MODULE_ZTIMER_PERIPH_RTT
+    select MODULE_ZTIMER_PERIPH_RTT if !MODULE_ZTIMER_NO_PERIPH_RTT
 
 config ZTIMER_SEC_BACKEND_RTC
     bool "RTC"

sys/ztimer/Makefile.dep

@@ -78,35 +78,26 @@ ifneq (,$(filter ztimer_usec,$(USEMODULE)))
   USEMODULE += ztimer_periph_timer
 endif
 
-ifneq (,$(filter ztimer_msec,$(USEMODULE)))
+ifneq (,$(filter ztimer_msec ztimer_sec,$(USEMODULE)))
   USEMODULE += ztimer
-  FEATURES_OPTIONAL += periph_rtt
+  ifeq (,$(filter ztimer_no_periph_rtt,$(USEMODULE)))
-  # HACK: periph_rtt will get used only in the next iteration but an updated
+    FEATURES_OPTIONAL += periph_rtt
-  # state for FEATURES_USED is needed here so include `features_check.inc.mk`
+    # HACK: periph_rtt will get used only in the next iteration but an updated
-  # here instead.
+    # state for FEATURES_USED is needed here so include `features_check.inc.mk`
-  # An other option would be to check FEATURES_PROVIDED this would avoid the
+    # here instead.
-  # order of inclusion problem but it would no take into account possible conflicts
+    # An other option would be to check FEATURES_PROVIDED this would avoid the
-  # and is also currently not allowed in the build system.
+    # order of inclusion problem but it would no take into account possible conflicts
-  # An other alternative would be to delay to the next loop, but this produce a
+    # and is also currently not allowed in the build system.
-  # case where another loop is not executed and the conditional not evaluated
+    # An other alternative would be to delay to the next loop, but this produce a
-  # If these kind of usecases pop up before Kconfig migration is completed
+    # case where another loop is not executed and the conditional not evaluated
-  # then another alternative would be introduce a variable to require an extra
+    # If these kind of usecases pop up before Kconfig migration is completed
-  # loop independent of USEMODULE, FEATURES_REQUIRED and USEPKG
+    # then another alternative would be introduce a variable to require an extra
-  include $(RIOTMAKE)/features_check.inc.mk
+    # loop independent of USEMODULE, FEATURES_REQUIRED and USEPKG
-  ifneq (,$(filter periph_rtt,$(FEATURES_USED)))
+    include $(RIOTMAKE)/features_check.inc.mk
-    USEMODULE += ztimer_periph_rtt
+    ifneq (,$(filter periph_rtt,$(FEATURES_USED)))
-  else
+      USEMODULE += ztimer_periph_rtt
-    USEMODULE += ztimer_periph_timer
+    else
-  endif
+      USEMODULE += ztimer_periph_timer
-endif
+    endif
-
-ifneq (,$(filter ztimer_sec,$(USEMODULE)))
-  USEMODULE += ztimer
-  FEATURES_OPTIONAL += periph_rtt
-  # HACK: see above
-  ifneq (,$(filter periph_rtt,$(FEATURES_USED)))
-    USEMODULE += ztimer_periph_rtt
-  else
-    USEMODULE += ztimer_periph_timer
   endif
 endif

sys/ztimer/Makefile.include

@@ -4,6 +4,8 @@ ifneq (,$(filter ztimer_xtimer_compat,$(USEMODULE)))
   PSEUDOMODULES += xtimer
 endif
 
+MODULES_ZTIMER_ON_RTT_CONFLICT = rtt_rtc gnrc_lwmac gnrc_gomach
+
 # By defaul use highest possible RTT_FREQUENCY for platforms that allow it. This
 # might not be the most optimized for conversion guarantees that ztimer_periph_rtt
 # will have a capable backend.
@@ -12,4 +14,10 @@ ifneq (,$(filter ztimer_periph_rtt,$(USEMODULE)))
     RTT_FREQUENCY ?= RTT_MAX_FREQUENCY
     CFLAGS += -DRTT_FREQUENCY=$(RTT_FREQUENCY)
   endif
+
+  MODULES_ZTIMER_ON_RTT_CONFLICTING = $(filter $(MODULES_ZTIMER_ON_RTT_CONFLICT),$(USEMODULE))
+  ifneq (0,$(words $(MODULES_ZTIMER_ON_RTT_CONFLICTING)))
+    $(info $(COLOR_YELLOW)WARNING! The following modules conflict with 'ztimer_periph_rtt': '$(MODULES_ZTIMER_ON_RTT_CONFLICTING)')
+    $(info To disable ztimer periph_rtt auto-inclusion add 'ztimer_no_periph_rtt' to 'USEMODULE'$(COLOR_RESET))
+  endif
 endif

tests/ztimer_msg/Makefile

@@ -2,10 +2,10 @@ include ../Makefile.tests_common
 
 USEMODULE += ztimer_usec
 
-# uncomment this to test using ztimer msec on rtt
+# uncomment this to test using ztimer msec
-#USEMODULE += ztimer_msec ztimer_periph_rtt
+#USEMODULE += ztimer_msec
 
-# uncomment this to test using ztimer sec on rtc
+# uncomment this to test using ztimer sec
-#USEMODULE += ztimer_sec ztimer_periph_rtc
+#USEMODULE += ztimer_sec
 
 include $(RIOTBASE)/Makefile.include

tests/ztimer_underflow/Makefile

@@ -6,7 +6,6 @@ USEMODULE += ztimer
 USEMODULE += ztimer_usec
 ifeq ($(TEST_ZTIMER_CLOCK), ZTIMER_MSEC)
   USEMODULE += ztimer_msec
-  USEMODULE += ztimer_periph_rtt
 	# the same for Kconfig
   ifeq (1,$(TEST_KCONFIG))
     KCONFIG_ADD_CONFIG += $(APPDIR)/app.config.msec.test
@@ -14,7 +13,6 @@ ifeq ($(TEST_ZTIMER_CLOCK), ZTIMER_MSEC)
 endif
 ifeq ($(TEST_ZTIMER_CLOCK), ZTIMER_SEC)
   USEMODULE += ztimer_sec
-  USEMODULE += ztimer_periph_rtc
 endif
 
 CFLAGS += -DTEST_ZTIMER_CLOCK=$(TEST_ZTIMER_CLOCK)