Skip to content

cpu/esp*: compile time configurations for ESP MCUs#10766

Merged
gschorcht merged 2 commits intoRIOT-OS:masterfrom
gschorcht:esp_compile_time_configs
Jan 17, 2019
Merged

cpu/esp*: compile time configurations for ESP MCUs#10766
gschorcht merged 2 commits intoRIOT-OS:masterfrom
gschorcht:esp_compile_time_configs

Conversation

@gschorcht
Copy link
Copy Markdown
Contributor

Contribution description

This PR adds the groups for ESP MCUs to the config group and adds first compile-time configuration macros to these groups.

Testing procedure

Generate the documentation with make doc and check Modules / Compile time configurations page for the modules ESP8266 compile configurations and ESP32 compile configurations.

Issues/PRs references

This PR is prerequisite for PR #10762 and further upcoming PRs.

@gschorcht gschorcht added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: doc Area: Documentation Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Jan 14, 2019
@gschorcht gschorcht requested a review from jia200x January 14, 2019 11:25
@jia200x
Copy link
Copy Markdown
Member

jia200x commented Jan 14, 2019

great! Thank you!

@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 Jan 14, 2019
@gschorcht
Copy link
Copy Markdown
Contributor Author

@jia200x But it is not complete yet, just a beginning.

@gschorcht
Copy link
Copy Markdown
Contributor Author

@jia200x Is it possible to get this very small changes reviewed and mergeds? Thanks.

Copy link
Copy Markdown
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

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

Tested, looked at the code, and ACK!

@gschorcht
Copy link
Copy Markdown
Contributor Author

@MrKevinWeiss Thank you for reviewing.

@gschorcht gschorcht merged commit c061729 into RIOT-OS:master Jan 17, 2019
@gschorcht gschorcht deleted the esp_compile_time_configs branch January 17, 2019 15:39
@aabadie aabadie added this to the Release 2019.01 milestone Jan 21, 2019
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 Platform: ESP Platform: This PR/issue effects ESP-based platforms 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