Skip to content

ieee802154: add config macros to config doc group#10626

Merged
miri64 merged 2 commits intoRIOT-OS:masterfrom
jia200x:pr/ieee802154_conf
Dec 18, 2018
Merged

ieee802154: add config macros to config doc group#10626
miri64 merged 2 commits intoRIOT-OS:masterfrom
jia200x:pr/ieee802154_conf

Conversation

@jia200x
Copy link
Copy Markdown
Member

@jia200x jia200x commented Dec 18, 2018

Contribution description

See #10566

I also added some @brief tags to these macros, and moved the /* in dBm */ comment to the brief of IEEE802154_DEFAULT_TXPOWER

Testing procedure

Compile documentation. A new group for the configuration macros of net_ieee802154 should show up both in the config group doc as well in the doc of the respective net_ieee802154 (sub-)module changed here.

Issues/PRs references

Addresses #10566 in part

@jia200x jia200x added Area: doc Area: Documentation TF: Config Marks issues and PRs related to the work of the Configuration Task Force labels Dec 18, 2018
@jia200x jia200x requested a review from miri64 December 18, 2018 10:43
@jia200x jia200x added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 18, 2018
@miri64
Copy link
Copy Markdown
Member

miri64 commented Dec 18, 2018

Other than that. make doc produces the expected output. You may squash immediately.

@jia200x
Copy link
Copy Markdown
Member Author

jia200x commented Dec 18, 2018

squashed!

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR TF: Config Marks issues and PRs related to the work of the Configuration Task Force

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants