[riot-notifications] [RIOT-OS/RIOT] sys/auto_init: Allow to auto-initalize sensors and actuators independ of SAUL (#11871)
notifications at github.com
Sat Jul 20 22:40:34 CEST 2019
maribu commented on this pull request.
> @@ -1,3 +1,15 @@
+ifneq (,$(filter auto_init_actuators_%,$(USEMODULE)))
> So when you use(module +=) two sensors which provide auto_init functionality and activate auto_init for one of them, there is no stopping the other sensor from beeing auto_inited, right?
That would be a bug. If I had sensors foo and bar, and I would use `USEMODULE += foo bar auto_init_sensors_foo` than `auto_init_sensors` should get pulled in as dependency, but not `auto_init_sensors_bar`. That way foo should get initialized, but not bar.
The individual initialization is done in each submodule, e.g. `auto_init_sensors` should in absence of any of its submodule contain no actual initialization code (`void auto_init_sensors(void)` should be empty).
I'm not 100% sure that my implementation is all correct (therefore adapting one more sensor should be done to test this), but it is definitely intended that auto_init for sensors and actuators has do be enabled for each device individually, unless `auto_init_default` is used.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications