bcm283x: Issue 62 Change bcm283x to use ioctl for GPIO pins, not sysfs.#63
bcm283x: Issue 62 Change bcm283x to use ioctl for GPIO pins, not sysfs.#63maruel merged 9 commits intoperiph:mainfrom
Conversation
Codecov Report❌ Patch coverage is ❌ Your patch status has failed because the patch coverage (34.7%) is below the target coverage (60.0%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #63 +/- ##
=======================================
+ Coverage 26.8% 27.4% +0.5%
=======================================
Files 92 96 +4
Lines 11548 11774 +226
=======================================
+ Hits 3097 3221 +124
- Misses 8316 8418 +102
Partials 135 135 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@maruel can you give me an idea of how to fix this? periph.io/x/host/v3/bcm283xError: bcm283x/gpio.go:199:22: undefined: gpioioctl.GPIOLine |
NM, I see the problem. gpioioctl doesn't exist for Windows/MacOS, and that's messing with it. |
|
The stance periph.io takes is that the code is compilable on all platforms so users can read the documentation, e.g. in VSCode while on Windows but functionality that do not make sense (e.g. bcm238x on Windows) is okay to not work. |
|
Thanks! |
This fixes Edge detection does not work on Raspbian Bookworm #62
It changes the bcm283x/gpio.go to use gpioioctl rather than sysfs.
With this patch, obtaining GPIO pins via:
now work as expected.