[riot-notifications] [RIOT-OS/RIOT] sys/auto_init: Allow to auto-initalize sensors and actuators independ of SAUL (#11871)

Marian Buschsieweke 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:
https://github.com/RIOT-OS/RIOT/pull/11871#discussion_r305592431
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190720/ed94a2fb/attachment-0001.htm>


More information about the notifications mailing list