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

<p>I understand the need for conditional compilation here, and the code tests fine. However, there is a larger issue here related to the first question in the Maintainer Guidelines. This question also applies to <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="463802356" data-permission-text="Issue title is private" data-url="https://github.com/RIOT-OS/RIOT/issues/11787" data-hovercard-type="pull_request" data-hovercard-url="/RIOT-OS/RIOT/pull/11787/hovercard" href="https://github.com/RIOT-OS/RIOT/pull/11787">#11787</a>, so we might as well discuss it here for both PRs.</p>
<blockquote>
<p>Does the reasoning for this PR make sense?</p>
</blockquote>
<p>To me, "make sense" includes whether a user in a non-experimental setup would need this PR. It seems like if we open up the mainline code to what is needed for experiments, then that significantly increases the scope of what is acceptable. If it's just needed for an experiment, then why not just create a branch with the modifications for that purpose. Do we have a policy here?</p>
<p>At any rate, I think this PR is marginally acceptable for my "non-experimental" test. OTOH, it's hard to imagine how <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="463802356" data-permission-text="Issue title is private" data-url="https://github.com/RIOT-OS/RIOT/issues/11787" data-hovercard-type="pull_request" data-hovercard-url="/RIOT-OS/RIOT/pull/11787/hovercard" href="https://github.com/RIOT-OS/RIOT/pull/11787">#11787</a> would be useful in a non-experimental setup.</p>
<p>I don't feel super strongly about this, but I hate to reduce readability in mainline code with conditionals that only a single experimental setup will ever need.</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/11784?email_source=notifications&email_token=ABE7WYFBYSVC4BD4FPBBTWDQCDYUJA5CNFSM4H5GCBYKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCACN7QQ#pullrequestreview-268754882">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYBCSJKRVGXGHPK7N53QCDYUJANCNFSM4H5GCBYA">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYFAYKRQ2AEFRLFEIBDQCDYUJA5CNFSM4H5GCBYKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCACN7QQ.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/11784?email_source=notifications\u0026email_token=ABE7WYFBYSVC4BD4FPBBTWDQCDYUJA5CNFSM4H5GCBYKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCACN7QQ#pullrequestreview-268754882",
"url": "https://github.com/RIOT-OS/RIOT/pull/11784?email_source=notifications\u0026email_token=ABE7WYFBYSVC4BD4FPBBTWDQCDYUJA5CNFSM4H5GCBYKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCACN7QQ#pullrequestreview-268754882",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>