<p><b>@maribu</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/11871#discussion_r305592431">sys/Makefile.dep</a>:</p>
<pre style='color:#555'>> @@ -1,3 +1,15 @@
+ifneq (,$(filter auto_init_actuators_%,$(USEMODULE)))
</pre>
<blockquote>
<p>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?</p>
</blockquote>
<p>That would be a bug. If I had sensors foo and bar, and I would use <code>USEMODULE += foo bar auto_init_sensors_foo</code> than <code>auto_init_sensors</code> should get pulled in as dependency, but not <code>auto_init_sensors_bar</code>. That way foo should get initialized, but not bar.</p>
<p>The individual initialization is done in each submodule, e.g. <code>auto_init_sensors</code> should in absence of any of its submodule contain no actual initialization code (<code>void auto_init_sensors(void)</code> should be empty).</p>
<p>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 <code>auto_init_default</code> is used.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/RIOT-OS/RIOT/pull/11871?email_source=notifications&email_token=ABE7WYCNPJ27W3BTZASU4UTQANZ4FA5CNFSM4IFE2XHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB7CBZMY#discussion_r305592431">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYH3F7Z4AOPREZKZZGLQANZ4FANCNFSM4IFE2XHA">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYDSRAGMJOEVCZZVIGLQANZ4FA5CNFSM4IFE2XHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB7CBZMY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/RIOT-OS/RIOT/pull/11871?email_source=notifications\u0026email_token=ABE7WYCNPJ27W3BTZASU4UTQANZ4FA5CNFSM4IFE2XHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB7CBZMY#discussion_r305592431",
"url": "https://github.com/RIOT-OS/RIOT/pull/11871?email_source=notifications\u0026email_token=ABE7WYCNPJ27W3BTZASU4UTQANZ4FA5CNFSM4IFE2XHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB7CBZMY#discussion_r305592431",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>