Skip to content

boards/native: clean-up MTD and SPIFFS default config#8272

Merged
vincent-d merged 2 commits intoRIOT-OS:masterfrom
OTAkeys:pr/clean-up-mtd-native
Feb 9, 2018
Merged

boards/native: clean-up MTD and SPIFFS default config#8272
vincent-d merged 2 commits intoRIOT-OS:masterfrom
OTAkeys:pr/clean-up-mtd-native

Conversation

@vincent-d
Copy link
Copy Markdown
Member

Contribution description

In native board:

  • Clean-up MTD default config: move defines from board.c to board.h
  • Clean-up SPIFFS default config: use previously added MTD defines and clean-up doc

Issues/PRs references

None

@vincent-d vincent-d added Platform: native Platform: This PR/issue effects the native platform CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Dec 18, 2017
Copy link
Copy Markdown
Member

@jnohlgard jnohlgard left a comment

Choose a reason for hiding this comment

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

ACK, looks good

@vincent-d vincent-d added this to the Release 2018.04 milestone Feb 9, 2018
@vincent-d
Copy link
Copy Markdown
Member Author

All green, go

@vincent-d vincent-d merged commit 368818a into RIOT-OS:master Feb 9, 2018
@vincent-d vincent-d deleted the pr/clean-up-mtd-native branch February 9, 2018 13:49
jnohlgard pushed a commit to jnohlgard/RIOT that referenced this pull request Feb 22, 2018
jnohlgard pushed a commit to jnohlgard/RIOT that referenced this pull request Feb 22, 2018
jnohlgard pushed a commit that referenced this pull request Feb 26, 2018
tests/fatfs: Fix tests on native after configuration changes in #8272
panail pushed a commit to panail/RIOT that referenced this pull request Oct 29, 2018
panail pushed a commit to panail/RIOT that referenced this pull request Oct 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: native Platform: This PR/issue effects the native platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants