-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
include: drivers: gpio: fix gpio_dt_flags_t overflow #72946
include: drivers: gpio: fix gpio_dt_flags_t overflow #72946
Conversation
ce13dc8
to
8bab1c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not simply move GPIO_INT_WAKEUP
to bit 6?
8bab1c4
to
0c61f61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but please update the commit message to describe this new solution better.
when GPIO_INT_WAKEUP flag is used in a DT gpios property the gpio_dt_flags_t var that holds the flags overflows Hence moving GPIO_INT_WAKEUP flag from bit 28 to bit 6 Signed-off-by: Abderrahmane Jarmouni <abderrahmane.jarmouni-ext@st.com>
0c61f61
to
7b5c565
Compare
done |
when
GPIO_INT_WAKEUP
flag is used in a DTgpios
property, thegpio_dt_flags_t
var that holds the flags overflows.