Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use GPIO_FLAGS instead of GPIO_INT_CONF #7493

Merged

Conversation

diegosueiro
Copy link
Contributor

Rename GPIO_INT_CONF to GPIO_FLAGS in order to be able to use the definition generated by the devicetree, eliminating the need for definition in the board.h file.

@codecov-io
Copy link

codecov-io commented May 12, 2018

Codecov Report

Merging #7493 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7493   +/-   ##
=======================================
  Coverage   55.25%   55.25%           
=======================================
  Files         467      467           
  Lines       51659    51659           
  Branches     9889     9889           
=======================================
  Hits        28543    28543           
  Misses      19206    19206           
  Partials     3910     3910

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5ce10f1...7551150. Read the comment docs.

Rename GPIO_INT_CONF to GPIO_FLAGS in order to be able to use
the definition generated by the devicetree, eliminating the need
for definition in the board.h file

Signed-off-by: Diego Sueiro <diego.sueiro@gmail.com>
@diegosueiro
Copy link
Contributor Author

@pgielda and @kgugala,

Is there anything blocking this PR?

Copy link
Collaborator

@kgugala kgugala left a comment

Choose a reason for hiding this comment

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

This PR seems to be OK

@nashif nashif merged commit 9255bf5 into zephyrproject-rtos:master Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants