netdev: Add netopt to retrieve netdev_ieee802154_t#9335
netdev: Add netopt to retrieve netdev_ieee802154_t#9335bergzand wants to merge 4 commits intoRIOT-OS:masterfrom
Conversation
To prevent issues with gnrc_netif_802154, this getter is added to retrieve the netdev_ieee802154_t struct. This struct is required for retrieving the panid and incrementing sequence number with packet transmission and more.
|
I think I have a problem understanding the problem here... Why can't you just use |
|
The problem comes from a design flaw IMO, placing the link layer frame state variables inside the device driver instead of inside the gnrc_netif layer. |
|
It would also remove the need for device drivers to specify the protocol via the netdev_ieee802154_t::proto member (e.g. https://github.com/RIOT-OS/RIOT/blob/master/drivers/at86rf2xx/at86rf2xx.c#L91-L96). The device driver IMO should not have to care about what protocol is running in a higher layer |
|
I had a discussion about this yesterday with @miri64. Our idea was to have a generic |
Agreed :) |
|
Closing this one as the approach is not the way to go:tm: here. |
Contribution description
This PR adds a get operation to retrieve the
netdev_ieee802154_tfrom a radio. This because the struct contains essential information required for constructing an ieee802154 frame.Issues/PRs references
required to save a new struct member in #9329