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

system(gpio): add edge polling function #1015

Merged

Conversation

gen2thomas
Copy link
Collaborator

@gen2thomas gen2thomas commented Oct 25, 2023

Solved issues and/or description of the change

this prepares the usage of upcoming HC-SR04 driver, see #1012, independently of cdev event-handler

additionally:

  • fix that all sysfs options are applied, not only the first one
  • fix in sysfs that Unexport() will be called always after an export was successfully
  • fix in sysf that the originated error will be returned if Unexport() was called successfully
  • tests for sysfs improved

Manual test

  • OS and Version (Win/Mac/Linux): Linux
  • Adaptor(s) and/or driver(s):
    • raspberry
    • tinkerboard

Checklist

  • The PR's target branch is 'hybridgroup:dev'
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes (e.g. by run make test)
  • No linter errors exist locally (e.g. by run make fmt_check)
  • I have performed a self-review of my own code
  • The gpio README is extended

@gen2thomas gen2thomas force-pushed the feature/gpio_simulate_edgedetect branch 8 times, most recently from 551c55b to 99e214f Compare October 26, 2023 17:08
@gen2thomas gen2thomas force-pushed the feature/gpio_simulate_edgedetect branch from 99e214f to d466949 Compare October 26, 2023 18:38
@gen2thomas gen2thomas marked this pull request as ready for review October 26, 2023 18:41
@gen2thomas gen2thomas merged commit 1f09353 into hybridgroup:dev Oct 26, 2023
5 checks passed
@gen2thomas gen2thomas deleted the feature/gpio_simulate_edgedetect branch October 26, 2023 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant