[riot-notifications] [RIOT-OS/RIOT] Pr/cc2538 rf/undedup address (#10425)

Koen Zandberg notifications at github.com
Sat Nov 17 22:40:03 CET 2018


### Contribution description

This PR modifies the behaviour of the cc2538_rf driver to not propagate the link layer address to the netdev_ieee802154_t struct. This helps the goal of separating the netdev and the ieee802154_t layer by another bit. The end goal is to remove the short and long address from the netdev_ieee802154_t struct and match the behaviour of the ethernet drivers.

The performance impact of this change should be negligible, the NETOPT_ADDRESS(_LONG) call is only used on initialization and by the ifconfig command.

### Testing procedure

Test whether:

 -  ifconfig still shows the correct link layer addresses. These should not have changed between this PR and master.
 - ping still works. This to verify that the link layer address is also properly configured in the radio, if the ping echo/reply fails, the address filter of the radio might be configured in the wrong endianness.

### Issues/PRs references

related to #10401, part of #7736 

**disclaimer**: I don't own a board with the cc2538 on it so I'm flying blind here. As long as the actual address getters work, the code *should* work.
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/10425

-- Commit Summary --

  * cc2538_rf: Add NETOPT_ADDRESS{,_LONG} to getters
  * cc2538_rf: Don't propagate address to 802154 layer

-- File Changes --

    M cpu/cc2538/radio/cc2538_rf_netdev.c (20)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/10425.patch
https://github.com/RIOT-OS/RIOT/pull/10425.diff

-- 
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/10425
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20181117/fd2b010d/attachment.html>


More information about the notifications mailing list