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

periph/gpio: support for extension API (part 2) #9958

Closed
wants to merge 13 commits into from

Conversation

ZetaR60
Copy link
Contributor

@ZetaR60 ZetaR60 commented Sep 19, 2018

Contribution description

This is an implementation of GPIO redirection outlined by my proposal in #9582 (items 5-6) for the periph/gpio interface. In short, it allows other drivers (such as GPIO expanders) to be addressed using gpio_t so that all existent code can easily gain support through those drivers. For non-implementation detail and discussion, please see #9582

At @kYc0o 's suggestion, I am breaking the implementation of #9582 / #9690 into several pieces to make it easier to review. This PR only contains the API redirection code. For the intercept code (part 1) see #9860

Dependencies

This depends on #9860

Testing procedure

A test case is provided that implements a soft-driver for the GPIO extension interface. The test case uses this driver to confirm that interception and redirection of the API call are working properly. This has been tested and is working properly on mega-xplained.

Issues/PRs references

Partially replaces and closes #9190
Implements items 5-6 (redirection) of my proposal in #9582
For implementation of items 1-4 (interception) see #9860
Part of work tracked by #9690

@ZetaR60 ZetaR60 added Type: new feature The issue requests / The PR implemements a new feature for RIOT Impact: major The PR changes a significant part of the code base. It should be reviewed carefully State: waiting for other PR State: The PR requires another PR to be merged first labels Sep 19, 2018
@ZetaR60 ZetaR60 added this to the Release 2018.10 milestone Sep 19, 2018
@RIOT-OS RIOT-OS deleted a comment Sep 19, 2018
tests/extend_gpio/main.c Outdated Show resolved Hide resolved
drivers/Makefile.dep Outdated Show resolved Hide resolved
drivers/Makefile.dep Outdated Show resolved Hide resolved
drivers/Makefile.include Outdated Show resolved Hide resolved
@stale
Copy link

stale bot commented Apr 10, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Apr 10, 2020
@stale stale bot closed this May 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Impact: major The PR changes a significant part of the code base. It should be reviewed carefully State: stale State: The issue / PR has no activity for >185 days State: waiting for other PR State: The PR requires another PR to be merged first Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants