[riot-notifications] [RIOT-OS/RIOT] netdev_ieee802154/radios: refactor PAN ID reset to generic ieee802154 reset (#10384)

Koen Zandberg notifications at github.com
Tue Nov 13 21:23:09 CET 2018


### Contribution description

This PR refactors the PAN ID from the radio reset function to the generic `netdev_ieee802154_reset` and sets it via the `netdev::set` call. This allows for removal of the `dev->netdev` accesses in the radio specific `set_pan` calls.

### Testing procedure

Verify that initialization and reset of the PAN ID works for the affected radios

- [ ] cc2538
- [ ] socket_zep
- [ ] at86rf2xx
- [ ] cc2420
- [ ] kw2xrf
- [ ] mrf24j40

### Issues/PRs references

Chipping away at dependencies for #7736 
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * netdev_ieee802154: Add PAN ID to reset function
  * cc2538_rf: Remove PAN ID initialization from reset
  * zep: Remove PAN ID initialization from reset
  * at86rf2xx: Remove PAN ID initialization from reset
  * cc2420: Remove PAN ID initialization from reset
  * kw2xrf: Remove PAN ID initialization from reset
  * mrf24j40: Remove PAN ID initialization from reset
  * at86rf2xx: Remove netdev access from set_pan
  * cc2420: Remove netdev access from set_pan
  * kw2xrf: Remove netdev access from set_pan
  * mrf24j40: Remove netdev access from set_pan

-- File Changes --

    M cpu/cc2538/include/cc2538_rf.h (1)
    M cpu/cc2538/radio/cc2538_rf.c (1)
    M cpu/native/socket_zep/socket_zep.c (1)
    M drivers/at86rf2xx/at86rf2xx.c (2)
    M drivers/at86rf2xx/at86rf2xx_getset.c (1)
    M drivers/cc2420/cc2420.c (1)
    M drivers/cc2420/cc2420_getset.c (1)
    M drivers/include/at86rf2xx.h (7)
    M drivers/include/cc2420.h (5)
    M drivers/include/kw2xrf.h (5)
    M drivers/kw2xrf/kw2xrf.c (5)
    M drivers/kw2xrf/kw2xrf_getset.c (3)
    M drivers/mrf24j40/mrf24j40.c (2)
    M drivers/mrf24j40/mrf24j40_getset.c (1)
    M drivers/netdev_ieee802154/netdev_ieee802154.c (3)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/10384.patch
https://github.com/RIOT-OS/RIOT/pull/10384.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/10384
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20181113/a5c20219/attachment.html>


More information about the notifications mailing list