Skip to content

native: remove feature periph_gpio_irq#9976

Merged
haukepetersen merged 2 commits intoRIOT-OS:masterfrom
haukepetersen:fix_native_gpioirqfeatureremove
Sep 20, 2018
Merged

native: remove feature periph_gpio_irq#9976
haukepetersen merged 2 commits intoRIOT-OS:masterfrom
haukepetersen:fix_native_gpioirqfeatureremove

Conversation

@haukepetersen
Copy link
Copy Markdown
Contributor

Contribution description

See #9974. native does not implement any GPIO irq features, so it should not declare that feature in the board, and also the function stubs are not longer needed.

Testing procedure

Buildtest should suffice.

Issues/PRs references

#9974

@haukepetersen haukepetersen added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Platform: native Platform: This PR/issue effects the native platform 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 labels Sep 20, 2018
@haukepetersen haukepetersen changed the title Fix native gpioirqfeatureremove native: remove feature periph_gpio_irq Sep 20, 2018
@cladmi
Copy link
Copy Markdown
Contributor

cladmi commented Sep 20, 2018

To keep a working git history, the commits should be in the inverted order with disabling the feature before removing the code.

@haukepetersen haukepetersen force-pushed the fix_native_gpioirqfeatureremove branch from 76a9832 to a965532 Compare September 20, 2018 12:51
@haukepetersen
Copy link
Copy Markdown
Contributor Author

changed order of commits

Copy link
Copy Markdown
Contributor

@cladmi cladmi left a comment

Choose a reason for hiding this comment

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

ACK waiting for murdock.

Github is still displaying them in the old order but it is a known issue with github displaying by 'author' date, checking locally displays the correct commit order.

@haukepetersen
Copy link
Copy Markdown
Contributor Author

All green -> go go go

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 Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) 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