sys/auto_init: allow delayed initialisation of SAUL

Co-authored-by: Martine S. Lenders <m.lenders@fu-berlin.de>

Makefile.dep

@@ -34,6 +34,10 @@ ifneq (,$(filter auto_init_gnrc_netif,$(USEMODULE)))
   USEMODULE += gnrc_netif_init_devs
 endif
 
+ifneq (,$(filter auto_init_saul,$(USEMODULE)))
+  USEMODULE += saul_init_devs
+endif
+
 ifneq (,$(filter csma_sender,$(USEMODULE)))
   USEMODULE += random
   USEMODULE += xtimer

drivers/Makefile

@@ -1,3 +1,7 @@
 DIRS += $(dir $(wildcard $(addsuffix /Makefile, $(USEMODULE))))
 
+ifneq (,$(filter saul_init_devs,$(USEMODULE)))
+  DIRS += saul/init_devs
+endif
+
 include $(RIOTBASE)/Makefile.base

drivers/include/saul.h

@@ -279,6 +279,16 @@ typedef struct {
     uint8_t type;           /**< device class the device belongs to */
 } saul_driver_t;
 
+/**
+ * @brief   Initialize all available SAUL drivers.
+ *          This function is called automatically if the auto_init_saul
+ *          module is used.
+ *          If only the saul_init module is used instead, you can call
+ *          this function to manually set up all SAUL sensors at a later
+ *          time.
+ */
+void saul_init_devs(void);
+
 /**
  * @brief   Default not supported function
  */

drivers/saul/init_devs/Makefile

@@ -1,3 +1,3 @@
-MODULE = auto_init_saul
+MODULE = saul_init_devs
 
 include $(RIOTBASE)/Makefile.base

drivers/saul/init_devs/doc.txt

@@ -13,3 +13,11 @@
  *
  * This module contains auto init functions for drivers that support SAUL.
  */
+
+/**
+ * @defgroup drivers_saul_init_devs SAUL Device Initialisation
+ * @ingroup drivers_saul
+ * @brief Bulk initialisation of SAUL enabled drivers
+ *
+ * This module contains bulk init functions for drivers that support SAUL.
+ */

drivers/saul/init_devs/init.c

@@ -0,0 +1,254 @@
+/**
+ * Auto initialization for SAUL devices
+ *
+ * Copyright (C) 2020 Freie Universität Berlin
+ *               2020 Kaspar Schleiser <kaspar@schleiser.de>
+ *               2013  INRIA.
+ *
+ * This file is subject to the terms and conditions of the GNU Lesser
+ * General Public License v2.1. See the file LICENSE in the top level
+ * directory for more details.
+ *
+ * @ingroup sys_auto_init
+ * @{
+ * @file
+ * @brief   initializes any used SAUL device that has a trivial init function
+ * @author  Oliver Hahm <oliver.hahm@inria.fr>
+ * @author  Hauke Petersen <hauke.petersen@fu-berlin.de>
+ * @author  Kaspar Schleiser <kaspar@schleiser.de>
+ * @author  Martine S. Lenders <m.lenders@fu-berlin.de>
+ * @}
+ */
+
+#include "kernel_defines.h"
+
+/**
+ * @brief   Initializes sensors and actuators for SAUL
+ */
+void saul_init_devs(void)
+{
+    if (IS_USED(MODULE_SAUL_ADC)) {
+        extern void auto_init_adc(void);
+        auto_init_adc();
+    }
+    if (IS_USED(MODULE_SAUL_GPIO)) {
+        extern void auto_init_gpio(void);
+        auto_init_gpio();
+    }
+    if (IS_USED(MODULE_SAUL_NRF_TEMPERATURE)) {
+        extern void auto_init_nrf_temperature(void);
+        auto_init_nrf_temperature();
+    }
+    if (IS_USED(MODULE_AD7746)) {
+        extern void auto_init_ad7746(void);
+        auto_init_ad7746();
+    }
+    if (IS_USED(MODULE_ADCXX1C)) {
+        extern void auto_init_adcxx1c(void);
+        auto_init_adcxx1c();
+    }
+    if (IS_USED(MODULE_ADS101X)) {
+        extern void auto_init_ads101x(void);
+        auto_init_ads101x();
+    }
+    if (IS_USED(MODULE_ADXL345)) {
+        extern void auto_init_adxl345(void);
+        auto_init_adxl345();
+    }
+    if (IS_USED(MODULE_BMP180)) {
+        extern void auto_init_bmp180(void);
+        auto_init_bmp180();
+    }
+    if (IS_USED(MODULE_BMX280)) {
+        extern void auto_init_bmx280(void);
+        auto_init_bmx280();
+    }
+    if (IS_USED(MODULE_BMX055)) {
+        extern void auto_init_bmx055(void);
+        auto_init_bmx055();
+    }
+    if (IS_USED(MODULE_CCS811)) {
+        extern void auto_init_ccs811(void);
+        auto_init_ccs811();
+    }
+    if (IS_USED(MODULE_DHT)) {
+        extern void auto_init_dht(void);
+        auto_init_dht();
+    }
+    if (IS_USED(MODULE_DS18)) {
+        extern void auto_init_ds18(void);
+        auto_init_ds18();
+    }
+    if (IS_USED(MODULE_DS75LX)) {
+        extern void auto_init_ds75lx(void);
+        auto_init_ds75lx();
+    }
+    if (IS_USED(MODULE_FXOS8700)) {
+        extern void auto_init_fxos8700(void);
+        auto_init_fxos8700();
+    }
+    if (IS_USED(MODULE_GROVE_LEDBAR)) {
+        extern void auto_init_grove_ledbar(void);
+        auto_init_grove_ledbar();
+    }
+    if (IS_USED(MODULE_HDC1000)) {
+        extern void auto_init_hdc1000(void);
+        auto_init_hdc1000();
+    }
+    if (IS_USED(MODULE_HTS221)) {
+        extern void auto_init_hts221(void);
+        auto_init_hts221();
+    }
+    if (IS_USED(MODULE_INA2XX)) {
+        extern void auto_init_ina2xx(void);
+        auto_init_ina2xx();
+    }
+    if (IS_USED(MODULE_INA3221)) {
+        extern void auto_init_ina3221(void);
+        auto_init_ina3221();
+    }
+    if (IS_USED(MODULE_IO1_XPLAINED)) {
+        extern void auto_init_io1_xplained(void);
+        auto_init_io1_xplained();
+    }
+    if (IS_USED(MODULE_ISL29020)) {
+        extern void auto_init_isl29020(void);
+        auto_init_isl29020();
+    }
+    if (IS_USED(MODULE_ITG320X)) {
+        extern void auto_init_itg320x(void);
+        auto_init_itg320x();
+    }
+    if (IS_USED(MODULE_JC42)) {
+        extern void auto_init_jc42(void);
+        auto_init_jc42();
+    }
+    if (IS_USED(MODULE_L3G4200D)) {
+        extern void auto_init_l3g4200d(void);
+        auto_init_l3g4200d();
+    }
+    if (IS_USED(MODULE_LIS2DH12)) {
+        extern void auto_init_lis2dh12(void);
+        auto_init_lis2dh12();
+    }
+    if (IS_USED(MODULE_LIS3DH)) {
+        extern void auto_init_lis3dh(void);
+        auto_init_lis3dh();
+    }
+    if (IS_USED(MODULE_LIS3MDL)) {
+        extern void auto_init_lis3mdl(void);
+        auto_init_lis3mdl();
+    }
+    if (IS_USED(MODULE_LPSXXX)) {
+        extern void auto_init_lpsxxx(void);
+        auto_init_lpsxxx();
+    }
+    if (IS_USED(MODULE_LSM303DLHC)) {
+        extern void auto_init_lsm303dlhc(void);
+        auto_init_lsm303dlhc();
+    }
+    if (IS_USED(MODULE_LSM6DSL)) {
+        extern void auto_init_lsm6dsl(void);
+        auto_init_lsm6dsl();
+    }
+    if (IS_USED(MODULE_LTC4150)) {
+        extern void auto_init_ltc4150(void);
+        auto_init_ltc4150();
+    }
+    if (IS_USED(MODULE_MAG3110)) {
+        extern void auto_init_mag3110(void);
+        auto_init_mag3110();
+    }
+    if (IS_USED(MODULE_MMA7660)) {
+        extern void auto_init_mma7660(void);
+        auto_init_mma7660();
+    }
+    if (IS_USED(MODULE_MMA8X5X)) {
+        extern void auto_init_mma8x5x(void);
+        auto_init_mma8x5x();
+    }
+    if (IS_USED(MODULE_MPL3115A2)) {
+        extern void auto_init_mpl3115a2(void);
+        auto_init_mpl3115a2();
+    }
+    if (IS_USED(MODULE_MPU9X50)) {
+        extern void auto_init_mpu9x50(void);
+        auto_init_mpu9x50();
+    }
+    if (IS_USED(MODULE_OPT3001)) {
+        extern void auto_init_opt3001(void);
+        auto_init_opt3001();
+    }
+    if (IS_USED(MODULE_PCA9685)) {
+        extern void auto_init_pca9685(void);
+        auto_init_pca9685();
+    }
+    if (IS_USED(MODULE_PH_OEM)) {
+        extern void auto_init_ph_oem(void);
+        auto_init_ph_oem();
+    }
+    if (IS_USED(MODULE_PIR)) {
+        extern void auto_init_pir(void);
+        auto_init_pir();
+    }
+    if (IS_USED(MODULE_PULSE_COUNTER)) {
+        extern void auto_init_pulse_counter(void);
+        auto_init_pulse_counter();
+    }
+    if (IS_USED(MODULE_QMC5883L)) {
+        extern void auto_init_qmc5883l(void);
+        auto_init_qmc5883l();
+    }
+    if (IS_USED(MODULE_SHT2X)) {
+        extern void auto_init_sht2x(void);
+        auto_init_sht2x();
+    }
+    if (IS_USED(MODULE_SHT3X)) {
+        extern void auto_init_sht3x(void);
+        auto_init_sht3x();
+    }
+    if (IS_USED(MODULE_SHTC1)) {
+        extern void auto_init_shtc1(void);
+        auto_init_shtc1();
+    }
+    if (IS_USED(MODULE_SDS011)) {
+        extern void auto_init_sds011(void);
+        auto_init_sds011();
+    }
+    if (IS_USED(MODULE_SI114X)) {
+        extern void auto_init_si114x(void);
+        auto_init_si114x();
+    }
+    if (IS_USED(MODULE_SI70XX)) {
+        extern void auto_init_si70xx(void);
+        auto_init_si70xx();
+    }
+    if (IS_USED(MODULE_SPS30)) {
+        extern void auto_init_sps30(void);
+        auto_init_sps30();
+    }
+    if (IS_USED(MODULE_TCS37727)) {
+        extern void auto_init_tcs37727(void);
+        auto_init_tcs37727();
+    }
+    if (IS_USED(MODULE_TMP006)) {
+        extern void auto_init_tmp00x(void);
+        auto_init_tmp00x();
+    }
+    if (IS_USED(MODULE_TSL2561)) {
+        extern void auto_init_tsl2561(void);
+        auto_init_tsl2561();
+    }
+    if (IS_USED(MODULE_TSL4531X)) {
+        extern void auto_init_tsl4531x(void);
+        auto_init_tsl4531x();
+    }
+    if (IS_USED(MODULE_VCNL40X0)) {
+        extern void auto_init_vcnl40x0(void);
+        auto_init_vcnl40x0();
+    }
+    if (IS_USED(MODULE_VEML6070)) {
+        extern void auto_init_veml6070(void);
+        auto_init_veml6070();
+    }
+}

makefiles/pseudomodules.inc.mk

@@ -210,7 +210,6 @@ PSEUDOMODULES += crypto_aes_unroll
 PSEUDOMODULES += auto_init_%
 NO_PSEUDOMODULES += auto_init_can
 NO_PSEUDOMODULES += auto_init_loramac
-NO_PSEUDOMODULES += auto_init_saul
 NO_PSEUDOMODULES += auto_init_security
 NO_PSEUDOMODULES += auto_init_storage
 NO_PSEUDOMODULES += auto_init_usbus

sys/auto_init/Makefile

@@ -2,7 +2,7 @@ ifneq (,$(filter gnrc_netif_init,$(USEMODULE)))
   DIRS += netif
 endif
 
-ifneq (,$(filter auto_init_saul,$(USEMODULE)))
+ifneq (,$(filter saul_init,$(USEMODULE)))
   DIRS += saul
 endif
 

sys/auto_init/auto_init.c

@@ -26,237 +26,6 @@
 #include "kernel_defines.h"
 #include "log.h"
 
-/**
- * @brief   Initializes sensors and actuators for SAUL
- */
-void _auto_init_saul(void)
-{
-    if (IS_USED(MODULE_SAUL_ADC)) {
-        extern void auto_init_adc(void);
-        auto_init_adc();
-    }
-    if (IS_USED(MODULE_SAUL_GPIO)) {
-        extern void auto_init_gpio(void);
-        auto_init_gpio();
-    }
-    if (IS_USED(MODULE_SAUL_NRF_TEMPERATURE)) {
-        extern void auto_init_nrf_temperature(void);
-        auto_init_nrf_temperature();
-    }
-    if (IS_USED(MODULE_AD7746)) {
-        extern void auto_init_ad7746(void);
-        auto_init_ad7746();
-    }
-    if (IS_USED(MODULE_ADCXX1C)) {
-        extern void auto_init_adcxx1c(void);
-        auto_init_adcxx1c();
-    }
-    if (IS_USED(MODULE_ADS101X)) {
-        extern void auto_init_ads101x(void);
-        auto_init_ads101x();
-    }
-    if (IS_USED(MODULE_ADXL345)) {
-        extern void auto_init_adxl345(void);
-        auto_init_adxl345();
-    }
-    if (IS_USED(MODULE_BMP180)) {
-        extern void auto_init_bmp180(void);
-        auto_init_bmp180();
-    }
-    if (IS_USED(MODULE_BMX280)) {
-        extern void auto_init_bmx280(void);
-        auto_init_bmx280();
-    }
-    if (IS_USED(MODULE_BMX055)) {
-        extern void auto_init_bmx055(void);
-        auto_init_bmx055();
-    }
-    if (IS_USED(MODULE_CCS811)) {
-        extern void auto_init_ccs811(void);
-        auto_init_ccs811();
-    }
-    if (IS_USED(MODULE_DHT)) {
-        extern void auto_init_dht(void);
-        auto_init_dht();
-    }
-    if (IS_USED(MODULE_DS18)) {
-        extern void auto_init_ds18(void);
-        auto_init_ds18();
-    }
-    if (IS_USED(MODULE_DS75LX)) {
-        extern void auto_init_ds75lx(void);
-        auto_init_ds75lx();
-    }
-    if (IS_USED(MODULE_FXOS8700)) {
-        extern void auto_init_fxos8700(void);
-        auto_init_fxos8700();
-    }
-    if (IS_USED(MODULE_GROVE_LEDBAR)) {
-        extern void auto_init_grove_ledbar(void);
-        auto_init_grove_ledbar();
-    }
-    if (IS_USED(MODULE_HDC1000)) {
-        extern void auto_init_hdc1000(void);
-        auto_init_hdc1000();
-    }
-    if (IS_USED(MODULE_HTS221)) {
-        extern void auto_init_hts221(void);
-        auto_init_hts221();
-    }
-    if (IS_USED(MODULE_INA2XX)) {
-        extern void auto_init_ina2xx(void);
-        auto_init_ina2xx();
-    }
-    if (IS_USED(MODULE_INA3221)) {
-        extern void auto_init_ina3221(void);
-        auto_init_ina3221();
-    }
-    if (IS_USED(MODULE_IO1_XPLAINED)) {
-        extern void auto_init_io1_xplained(void);
-        auto_init_io1_xplained();
-    }
-    if (IS_USED(MODULE_ISL29020)) {
-        extern void auto_init_isl29020(void);
-        auto_init_isl29020();
-    }
-    if (IS_USED(MODULE_ITG320X)) {
-        extern void auto_init_itg320x(void);
-        auto_init_itg320x();
-    }
-    if (IS_USED(MODULE_JC42)) {
-        extern void auto_init_jc42(void);
-        auto_init_jc42();
-    }
-    if (IS_USED(MODULE_L3G4200D)) {
-        extern void auto_init_l3g4200d(void);
-        auto_init_l3g4200d();
-    }
-    if (IS_USED(MODULE_LIS2DH12)) {
-        extern void auto_init_lis2dh12(void);
-        auto_init_lis2dh12();
-    }
-    if (IS_USED(MODULE_LIS3DH)) {
-        extern void auto_init_lis3dh(void);
-        auto_init_lis3dh();
-    }
-    if (IS_USED(MODULE_LIS3MDL)) {
-        extern void auto_init_lis3mdl(void);
-        auto_init_lis3mdl();
-    }
-    if (IS_USED(MODULE_LPSXXX)) {
-        extern void auto_init_lpsxxx(void);
-        auto_init_lpsxxx();
-    }
-    if (IS_USED(MODULE_LSM303DLHC)) {
-        extern void auto_init_lsm303dlhc(void);
-        auto_init_lsm303dlhc();
-    }
-    if (IS_USED(MODULE_LSM6DSL)) {
-        extern void auto_init_lsm6dsl(void);
-        auto_init_lsm6dsl();
-    }
-    if (IS_USED(MODULE_LTC4150)) {
-        extern void auto_init_ltc4150(void);
-        auto_init_ltc4150();
-    }
-    if (IS_USED(MODULE_MAG3110)) {
-        extern void auto_init_mag3110(void);
-        auto_init_mag3110();
-    }
-    if (IS_USED(MODULE_MMA7660)) {
-        extern void auto_init_mma7660(void);
-        auto_init_mma7660();
-    }
-    if (IS_USED(MODULE_MMA8X5X)) {
-        extern void auto_init_mma8x5x(void);
-        auto_init_mma8x5x();
-    }
-    if (IS_USED(MODULE_MPL3115A2)) {
-        extern void auto_init_mpl3115a2(void);
-        auto_init_mpl3115a2();
-    }
-    if (IS_USED(MODULE_MPU9X50)) {
-        extern void auto_init_mpu9x50(void);
-        auto_init_mpu9x50();
-    }
-    if (IS_USED(MODULE_OPT3001)) {
-        extern void auto_init_opt3001(void);
-        auto_init_opt3001();
-    }
-    if (IS_USED(MODULE_PCA9685)) {
-        extern void auto_init_pca9685(void);
-        auto_init_pca9685();
-    }
-    if (IS_USED(MODULE_PH_OEM)) {
-        extern void auto_init_ph_oem(void);
-        auto_init_ph_oem();
-    }
-    if (IS_USED(MODULE_PIR)) {
-        extern void auto_init_pir(void);
-        auto_init_pir();
-    }
-    if (IS_USED(MODULE_PULSE_COUNTER)) {
-        extern void auto_init_pulse_counter(void);
-        auto_init_pulse_counter();
-    }
-    if (IS_USED(MODULE_QMC5883L)) {
-        extern void auto_init_qmc5883l(void);
-        auto_init_qmc5883l();
-    }
-    if (IS_USED(MODULE_SHT2X)) {
-        extern void auto_init_sht2x(void);
-        auto_init_sht2x();
-    }
-    if (IS_USED(MODULE_SHT3X)) {
-        extern void auto_init_sht3x(void);
-        auto_init_sht3x();
-    }
-    if (IS_USED(MODULE_SHTC1)) {
-        extern void auto_init_shtc1(void);
-        auto_init_shtc1();
-    }
-    if (IS_USED(MODULE_SDS011)) {
-        extern void auto_init_sds011(void);
-        auto_init_sds011();
-    }
-    if (IS_USED(MODULE_SI114X)) {
-        extern void auto_init_si114x(void);
-        auto_init_si114x();
-    }
-    if (IS_USED(MODULE_SI70XX)) {
-        extern void auto_init_si70xx(void);
-        auto_init_si70xx();
-    }
-    if (IS_USED(MODULE_SPS30)) {
-        extern void auto_init_sps30(void);
-        auto_init_sps30();
-    }
-    if (IS_USED(MODULE_TCS37727)) {
-        extern void auto_init_tcs37727(void);
-        auto_init_tcs37727();
-    }
-    if (IS_USED(MODULE_TMP006)) {
-        extern void auto_init_tmp00x(void);
-        auto_init_tmp00x();
-    }
-    if (IS_USED(MODULE_TSL2561)) {
-        extern void auto_init_tsl2561(void);
-        auto_init_tsl2561();
-    }
-    if (IS_USED(MODULE_TSL4531X)) {
-        extern void auto_init_tsl4531x(void);
-        auto_init_tsl4531x();
-    }
-    if (IS_USED(MODULE_VCNL40X0)) {
-        extern void auto_init_vcnl40x0(void);
-        auto_init_vcnl40x0();
-    }
-    if (IS_USED(MODULE_VEML6070)) {
-        extern void auto_init_veml6070(void);
-        auto_init_veml6070();
-    }
-}
-
 void auto_init(void)
 {
     if (IS_USED(MODULE_AUTO_INIT_RANDOM)) {
@@ -421,7 +190,8 @@ void auto_init(void)
 
     if (IS_USED(MODULE_AUTO_INIT_SAUL)) {
         LOG_DEBUG("Auto init SAUL.\n");
-        _auto_init_saul();
+        extern void saul_init_devs(void);
+        saul_init_devs();
     }
 
     if (IS_USED(MODULE_AUTO_INIT_GNRC_RPL)) {