Skip to content

drivers/at86rf215 : Expose Configurations to Kconfig#13885

Merged
leandrolanzieri merged 3 commits intoRIOT-OS:masterfrom
akshaim:Kconfig_at86rf215
Apr 22, 2020
Merged

drivers/at86rf215 : Expose Configurations to Kconfig#13885
leandrolanzieri merged 3 commits intoRIOT-OS:masterfrom
akshaim:Kconfig_at86rf215

Conversation

@akshaim
Copy link
Copy Markdown
Member

@akshaim akshaim commented Apr 17, 2020

Contribution description

This PR exposes compile configurations in at86rf215 network device driver to Kconfig. The following macro was define to print the macros.

#define STR(x) #x
#define SHOW_DEFINE(x) printf("%s=%s\n", #x, STR(x))

Testing procedure

The firmware was uploaded to FIT/IoT-LAB Test Bed and following results were obtained.

Default State:

Firmware Output

gnrc_netif: netdev init failed: -19
gnrc_netif: netdev init failed: -134
main(): This is RIOT! (Version: 2020.07-devel-82-g1d30-Kconfig_at86rf215)
CONFIG_AT86RF215_USE_CLOCK_OUTPUT=0
CONFIG_AT86RF215_TRIM_VAL=CONFIG_AT86RF215_TRIM_VAL

Usage with CFLAGS

/tests/driver_at86rf215/Makefile

CFLAGS += -DCONFIG_AT86RF215_USE_CLOCK_OUTPUT=1
CFLAGS += -DCONFIG_AT86RF215_TRIM_VAL=5

Firmware Output

gnrc_netif: netdev init failed: -19
gnrc_netif: netdev init failed: -134
main(): This is RIOT! (Version: 2020.07-devel-82-g1d30-Kconfig_at86rf215)
CONFIG_AT86RF215_USE_CLOCK_OUTPUT=1
CONFIG_AT86RF215_TRIM_VAL=5

Usage with Kconfig

/tests/at86rf215/

make menuconfig

Firmware Output

gnrc_netif: netdev init failed: -19
gnrc_netif: netdev init failed: -134
main(): This is RIOT! (Version: 2020.07-devel-82-g1d30-Kconfig_at86rf215)
CONFIG_AT86RF215_USE_CLOCK_OUTPUT=1
CONFIG_AT86RF215_TRIM_VAL=15

Note : The sensor is not available hence configurability of macros were only tested.

Issues/PRs references

#12888
@leandrolanzieri

@leandrolanzieri leandrolanzieri added Area: drivers Area: Device drivers Area: Kconfig Area: Kconfig integration Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Apr 17, 2020
Copy link
Copy Markdown
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

Looks good from my side.

@akshaim akshaim requested a review from leandrolanzieri April 17, 2020 17:10
@leandrolanzieri leandrolanzieri self-assigned this Apr 20, 2020
@leandrolanzieri
Copy link
Copy Markdown
Contributor

Please squash and rebase

akshaim and others added 3 commits April 22, 2020 15:54
Add Add CONFIG_ Prefix for AT86RF215_USE_CLOCK_OUTPUT and model
it as a bool

Co-Authored-By: Leandro Lanzieri <leandro.lanzieri@haw-hamburg.de>
Add CONFIG_ Prefix for configuring AT86RF215_TRIM_VAL
Expose Configurations to Kconfig

Co-Authored-By: benpicco <benpicco@googlemail.com>
Co-Authored-By: Leandro Lanzieri <leandro.lanzieri@haw-hamburg.de>
@akshaim akshaim force-pushed the Kconfig_at86rf215 branch from a2eccf7 to 7b2591e Compare April 22, 2020 10:27
@akshaim
Copy link
Copy Markdown
Member Author

akshaim commented Apr 22, 2020

Please squash and rebase

Done

Copy link
Copy Markdown
Contributor

@leandrolanzieri leandrolanzieri left a comment

Choose a reason for hiding this comment

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

ACK

@leandrolanzieri leandrolanzieri added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Reviewed: 1-fundamentals The fundamentals of the PR were reviewed according to the maintainer guidelines Reviewed: 2-code-design The code design of the PR was reviewed according to the maintainer guidelines Reviewed: 4-code-style The adherence to coding conventions by the PR were reviewed according to the maintainer guidelines Reviewed: 5-documentation The documentation details of the PR were reviewed according to the maintainer guidelines labels Apr 22, 2020
@leandrolanzieri leandrolanzieri merged commit ee74934 into RIOT-OS:master Apr 22, 2020
@akshaim akshaim deleted the Kconfig_at86rf215 branch April 22, 2020 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers Area: Kconfig Area: Kconfig integration CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Reviewed: 1-fundamentals The fundamentals of the PR were reviewed according to the maintainer guidelines Reviewed: 2-code-design The code design of the PR was reviewed according to the maintainer guidelines Reviewed: 4-code-style The adherence to coding conventions by the PR were reviewed according to the maintainer guidelines Reviewed: 5-documentation The documentation details of the PR were reviewed according to the maintainer guidelines Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants