Skip to content

netopt: add NETOPT_CSMA_MINBE, NETOPT_CSMA_MAXBE #7481

Merged
miri64 merged 3 commits intoRIOT-OS:masterfrom
jnohlgard:pr/netopt-csma-be
Aug 17, 2017
Merged

netopt: add NETOPT_CSMA_MINBE, NETOPT_CSMA_MAXBE #7481
miri64 merged 3 commits intoRIOT-OS:masterfrom
jnohlgard:pr/netopt-csma-be

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

These options can be used to get/set the backoff exponent parameters MinBE and MaxBE used by the CSMA/CA algorithm (see IEEE Std 802.15.4-2015, http://standards.ieee.org/about/get/802/802.15.html).
Also improved the documentation for the other CSMA options while I was touching that file.

Used by the kw41z radio in #7107

@jnohlgard jnohlgard added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking labels Aug 17, 2017
@jnohlgard jnohlgard added this to the Release 2017.10 milestone Aug 17, 2017
Copy link
Copy Markdown
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

ACK

@miri64 miri64 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Aug 17, 2017
@jnohlgard
Copy link
Copy Markdown
Member Author

btw, is it OK to add stuff in the middle of the enumeration like this or should new items go at the bottom?
It does break binary compatibility, but I don't think anyone is doing binary releases of object code in RIOT at the moment.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 17, 2017

It should be okay... at least for now.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 17, 2017

netopt2str() needs to be updated

@jnohlgard
Copy link
Copy Markdown
Member Author

@miri64 updated netopt2str, amended

@miri64 miri64 merged commit 9ea1c6e into RIOT-OS:master Aug 17, 2017
@jnohlgard jnohlgard deleted the pr/netopt-csma-be branch August 18, 2017 06:09
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 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.

2 participants