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

gpio: rebased bug fixes from #11880 #13145

Merged
merged 2 commits into from
Feb 8, 2019

Conversation

pabigot
Copy link
Collaborator

@pabigot pabigot commented Feb 7, 2019

Relates to #13076 and #13143. This, by the way, is exactly why something like:

   if ((flags & FOO_MASK) == FOO_VALUE) { }

is superior to:

   if (flags & FOO_VALUE) { }

Interrupts default to trigger on level for historical reasons, so use of
GPIO_INT_LEVEL` as a mask results in a zero value.  Use a mask macro to
isolate the trigger configuration.

Signed-off-by: Peter A. Bigot <pab@pabigot.com>
Interrupts default to trigger on level for historical reasons, so use of
GPIO_INT_LEVEL` as a mask results in a zero value.  Use a mask macro to
isolate the trigger configuration.

Signed-off-by: Peter A. Bigot <pab@pabigot.com>
@galak galak merged commit d080d1d into zephyrproject-rtos:master Feb 8, 2019
@pabigot pabigot deleted the pr/20190207a branch September 8, 2019 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants