<blockquote>
<p>I'm not entirely happy to see the "PHY" and "MAC" terminology here. Traditionally they refer to actual hardware ICs, which doesn't apply here.</p>
</blockquote>
<p>Hmmm when talking about PHY and MAC I'm refering to the definitions of IEEE802.15.4 MAC and PHY. Both components have their own interfaces and duties (see e.g <a href="http://www.ismlab.usf.edu/dcom/Ch8_802.15.4-2006.pdf" rel="nofollow">here</a>, section 6.2)</p>
<blockquote>
<p>"MAC" as in "handling the logical packet format" also doesn't fit. We will have netdev_802154 taking care of that, with another layer (that is replaceable) using its API to do actual access control.</p>
</blockquote>
<p>I agree. The thing is, stuff like PAN ID and addresses are pure MAC terminology, so at least they shouldn't be present in the driver (all .c files except at86rf2xx_netdev.c).</p>
<p>I think the title is misleading here. I'm not moving the PHY and MAC to the actual netdev interface, but I'm just removing the PHY and MAC component from the main driver implementation and put it for now in at86rf2xx_netdev.c</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/11717?email_source=notifications&email_token=ABE7WYCD3FBZPHJZO2E53LDP3HZV5A5CNFSM4HY7FFMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYBHADI#issuecomment-503476237">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYF2UFYRC74H2H6CN53P3HZV5ANCNFSM4HY7FFMA">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYG36FKFBILDW2AITYDP3HZV5A5CNFSM4HY7FFMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYBHADI.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/11717?email_source=notifications\u0026email_token=ABE7WYCD3FBZPHJZO2E53LDP3HZV5A5CNFSM4HY7FFMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYBHADI#issuecomment-503476237",
"url": "https://github.com/RIOT-OS/RIOT/pull/11717?email_source=notifications\u0026email_token=ABE7WYCD3FBZPHJZO2E53LDP3HZV5A5CNFSM4HY7FFMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYBHADI#issuecomment-503476237",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>