Skip to content

examples/gnrc_{minimal,networking}: simplify XBee setup#6971

Closed
aabadie wants to merge 1 commit intoRIOT-OS:masterfrom
aabadie:examples_xbee_config
Closed

examples/gnrc_{minimal,networking}: simplify XBee setup#6971
aabadie wants to merge 1 commit intoRIOT-OS:masterfrom
aabadie:examples_xbee_config

Conversation

@aabadie
Copy link
Copy Markdown
Contributor

@aabadie aabadie commented Apr 26, 2017

This PR is an attempt to simplify the configuration of an XBee with networking examples. The idea is to apply the XBee UART flag only when xbee module is loaded.

As a consequence, one can simply build a networking firmware using a XBee with the following command:

USEMODULE=xbee make BOARD=<board> -C examples/gnrc_minimal flash term

If you think this PR is worth, it can also be applied to other networking examples: emcute, microcoap, nanocoap, gcoap, dtls ?

@aabadie aabadie added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking labels Apr 26, 2017
@aabadie aabadie added this to the Release 2017.07 milestone Apr 26, 2017
@miri64 miri64 added the GNRC label Apr 26, 2017
@haukepetersen
Copy link
Copy Markdown
Contributor

Actually, I would give this PR another direction and remove all the xbee specific lines from the makefile(s) alltogether! There is no need to define the UART_DEV(1) for the xbee device, as this is the default parameter in drivers/include/xbee_params.h anyway...

-> see #6982

@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented Apr 28, 2017

I also thought the same but the thing is that keeping those lines shows new users how to deal with Xbee and the networking examples.
Maybe the best solution is to follow #6982 and add documentation (maybe in the README itself) about how to compile the example for Xbee ?

@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented May 3, 2017

closing in favor #6982

@aabadie aabadie closed this May 3, 2017
@aabadie aabadie deleted the examples_xbee_config branch June 8, 2018 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: network Area: Networking 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.

4 participants