-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sys/net/gnrc/netif: Make aac_mode a flag field #16596
Conversation
Hmm, one test has failed because of unsufficient ROM (4 bytes overflow). Should the board ( |
I'd say just add it to the list - if it's this tight it's not worth the trouble. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me - @miri64 any objections?
Nope, was my proposal after all :-), let's gooooo. |
Contribution description
This PR is another spin-off from #16228. Its changes let the the
aac_mode
field ofgnrc_netif_ipv6_t
be treated as flag field.The members of the
net_gnrc_netif_aac
enum are updated accordingly. This way, it is possible for a net interface to use (for instance) both SLAAC and DHCPv6 for auto-adress configuration.Testing procedure
As this is curently more or less a refactoring, you can use the existing
gnrc
and unit tests should which should still be passing.Issues/PRs references