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

[TOPIC-GPIO] update CAN #20944

Merged

Conversation

karstenkoenig
Copy link
Collaborator

Switch both the CAN sample with it's LED blink pattern as well as the MCP2515 driver to the new GPIO API - #20017

@zephyrbot
Copy link
Collaborator

zephyrbot commented Nov 22, 2019

All checks are passing now.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

Copy link
Member

@mnkp mnkp left a comment

Choose a reason for hiding this comment

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

LGTM with one minor comment.

dts/bindings/can/microchip,mcp2515.yaml Outdated Show resolved Hide resolved
Mark the INT signal to be active low and use the new functions to get
gpio state and configure the gpio interrupt flanks.

Signed-off-by: Karsten Koenig <karsten.koenig.030@gmail.com>
Switched from deprecated gpio_pin_write to gpio_pin_set and also add the
LED GPIO flags to the gpio configuration.

Signed-off-by: Karsten Koenig <karsten.koenig.030@gmail.com>
@carlescufi carlescufi merged commit c088a31 into zephyrproject-rtos:topic-gpio Dec 2, 2019
@alexanderwachter
Copy link
Member

@carlescufi updatet the state in #20017 to Done.

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.

7 participants