Skip to content

net/eui64: add eui64_to_eui48()#15096

Merged
maribu merged 2 commits intoRIOT-OS:masterfrom
benpicco:eui64_to_eui48
Sep 30, 2020
Merged

net/eui64: add eui64_to_eui48()#15096
maribu merged 2 commits intoRIOT-OS:masterfrom
benpicco:eui64_to_eui48

Conversation

@benpicco
Copy link
Contributor

Contribution description

Provide helper function to mangle a EUI-64 into an EUI-48. (Like eui48_to_eui64())
This can be useful to get predictable, locally unique addreses.

Testing procedure

Issues/PRs references

#15083 (comment)

@benpicco benpicco requested a review from maribu September 26, 2020 14:58
@benpicco benpicco added Area: network Area: Networking Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Sep 26, 2020
@benpicco benpicco removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 29, 2020
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK. Please also squash the suggested documentation clarification (feel free to rephrase) right in.

@maribu maribu added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 29, 2020
@benpicco benpicco force-pushed the eui64_to_eui48 branch 2 times, most recently from a2e4ad0 to 0f40588 Compare September 29, 2020 21:44
Move eui48_set_local() and eui48_clear_group() to the top so we can
use it later in the file.
Provide helper function to mangle a EUI-64 into an EUI-48.
This can be useful to get predictable, locally unique addreses.
@benpicco benpicco added State: waiting for CI update State: The PR requires an Update to CI to be performed first and removed State: waiting for CI update State: The PR requires an Update to CI to be performed first labels Sep 30, 2020
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@maribu maribu merged commit d684372 into RIOT-OS:master Sep 30, 2020
@maribu maribu added this to the Release 2020.10 milestone Sep 30, 2020
@benpicco benpicco deleted the eui64_to_eui48 branch September 30, 2020 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants