[riot-notifications] [RIOT-OS/RIOT] net: NETOPT_MAX_{PACKET_SIZE -> L2_PDU_SIZE} (#10905)

Marian Buschsieweke notifications at github.com
Wed Jan 30 12:12:56 CET 2019


### Contribution description

This PR renames `NETOPT_MAX_PACKET_SIZE` into `NETOPT_MAX_L2_PDU_SIZE`, which should be clearer. The documentation is updated accordingly.


### Testing procedure

Please verify that the new name and the updated documentation make sense. Apart from that Murdock will verify that this does not break anything.

### Issues/PRs references

PR https://github.com/RIOT-OS/RIOT/pull/10340 implemented `netdev_driver_t::get()` for `NETOPT_MAX_PACKET_SIZE` incorrectly and returned the maximum frame size including the layer 2 header. This rename should make clear that the maximum PDU size should be returned instead of the full frame size.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * net: NETOPT_MAX_{PACKET_SIZE -> L2_PDU_SIZE}

-- File Changes --

    M cpu/esp_common/esp-now/esp_now_netdev.c (2)
    M cpu/nrf5x_common/radio/nrfmin/nrfmin.c (2)
    M drivers/cc110x/cc110x-netdev.c (2)
    M drivers/netdev_eth/netdev_eth.c (2)
    M drivers/netdev_ieee802154/netdev_ieee802154.c (2)
    M drivers/sx127x/sx127x_netdev.c (4)
    M drivers/xbee/xbee.c (2)
    M sys/include/net/netopt.h (4)
    M sys/net/crosslayer/netopt/netopt.c (2)
    M sys/net/gnrc/application_layer/tftp/gnrc_tftp.c (2)
    M sys/net/gnrc/netif/gnrc_netif.c (4)
    M sys/net/gnrc/netif/gnrc_netif_device_type.c (4)
    M sys/shell/commands/sc_gnrc_netif.c (6)

-- Patch Links --

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


More information about the notifications mailing list