Skip to content

board/mips-malta: include the cpu features instead of common cpu#10064

Merged
smlng merged 1 commit intoRIOT-OS:masterfrom
cladmi:pr/board/mips-malta/cpu_features
Oct 1, 2018
Merged

board/mips-malta: include the cpu features instead of common cpu#10064
smlng merged 1 commit intoRIOT-OS:masterfrom
cladmi:pr/board/mips-malta/cpu_features

Conversation

@cladmi
Copy link
Copy Markdown
Contributor

@cladmi cladmi commented Sep 27, 2018

Contribution description

board/mips-malta: include the cpu features instead of common cpu

This change was missed when the cpu 'mips32r2_generic' was separated
from 'mips32r2_common'. The target file already includes 'mips32r2_common'.

Testing procedure

The FEATURES_PROVIDED do not change between master and this PR for mips-malta:

make --no-print-directory -C examples/hello-world/ info-debug-variable-FEATURES_PROVIDED BOARD=mips-malta
periph_timer cpp periph_pm

Issues/PRs references

Inconsistency found while working on #9913

This change was missed when the cpu 'mips32r2_generic' was separated
from 'mips32r2_common'. The target file already includes 'mips32r2_common'.
@cladmi cladmi added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports labels Sep 27, 2018
Copy link
Copy Markdown
Member

@smlng smlng left a comment

Choose a reason for hiding this comment

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

tested ACK, output of make info-build is also (mostly) equal, this PR only adds the Makefile.features from the generic folder

@smlng smlng merged commit a9fcff3 into RIOT-OS:master Oct 1, 2018
@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Oct 1, 2018

Thanks for reviewing.

@cladmi cladmi deleted the pr/board/mips-malta/cpu_features branch October 1, 2018 13:05
@jia200x jia200x added this to the Release 2018.10 milestone Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: boards Area: Board ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants