Skip to content

Conversation

@dcpleung
Copy link
Member

@dcpleung dcpleung commented Oct 5, 2019

This updates the gpio_dw driver to the new GPIO API. It also adds an overlay to the gpio_basic_api test, and updates the DTS files for board emsdp/em_starterkit.

This has been tested with gpio_basic_api.

@dcpleung dcpleung requested a review from galak October 5, 2019 00:18
@zephyrbot zephyrbot added area: Boards area: Tests Issues related to a particular existing or missing test labels Oct 5, 2019
@zephyrbot
Copy link

zephyrbot commented Oct 5, 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.

@dcpleung dcpleung force-pushed the dw_gpio_api branch 2 times, most recently from 654b130 to 0029fc2 Compare October 7, 2019 17:09
@carlescufi
Copy link
Member

@dcpleung
The topic branch has been rebased. Could you please rebase this PR against it?

This adds a board overlay for em_starterkit in the gpio_basic_api
test.

Signed-off-by: Daniel Leung <[email protected]>
Changes GPIO_INT_ACTIVE_LOW to GPIO_ACTIVE_LOW.

Signed-off-by: Daniel Leung <[email protected]>
Changes	GPIO_INT_ACTIVE_LOW to GPIO_ACTIVE_LOW.

Signed-off-by: Daniel Leung <[email protected]>
Update driver code and board files to use new GPIO configuration flags
such as GPIO_ACTIVE_LOW. Also add implementation of new port_* driver
API as well as gpio_pin_interrupt_configure function.

The interrupt triggering on both edges is a remnant from the old
Quark SE which has a customized DesignWare GPIO block. So remove
the support for both edges as the board is no longer supported.

Signed-off-by: Daniel Leung <[email protected]>
@dcpleung
Copy link
Member Author

@dcpleung
The topic branch has been rebased. Could you please rebase this PR against it?

Done.

@MaureenHelm MaureenHelm merged commit 0a423ad into zephyrproject-rtos:topic-gpio Oct 14, 2019
@dcpleung dcpleung deleted the dw_gpio_api branch October 14, 2019 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Boards area: GPIO area: Tests Issues related to a particular existing or missing test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants