-
Notifications
You must be signed in to change notification settings - Fork 20
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
Compatibility with gpiod 2.x #50
Conversation
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #50 +/- ##
==========================================
- Coverage 60.76% 60.71% -0.05%
==========================================
Files 13 14 +1
Lines 1631 1642 +11
==========================================
+ Hits 991 997 +6
- Misses 640 645 +5 ☔ View full report in Codecov by Sentry. |
I not so sure this is the case, I seem to get stuck in bootloader with this as is.
Adding a 4th pin state to reset both GPIO lines to True fixes it though. |
gpiod 2.x is posing a really annoying problem right now that I'm unsure of how to solve:
Possible solutions:
|
I'm pinning |
This would probably only work in reverse to downgrade the dep i.e. |
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [universal-silabs-flasher](https://togithub.com/NabuCasa/universal-silabs-flasher) | `==0.0.15` -> `==0.0.16` | [![age](https://developer.mend.io/api/mc/badges/age/pypi/universal-silabs-flasher/0.0.16?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/universal-silabs-flasher/0.0.16?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/universal-silabs-flasher/0.0.15/0.0.16?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/universal-silabs-flasher/0.0.15/0.0.16?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>NabuCasa/universal-silabs-flasher (universal-silabs-flasher)</summary> ### [`v0.0.16`](https://togithub.com/NabuCasa/universal-silabs-flasher/releases/tag/v0.0.16) [Compare Source](https://togithub.com/NabuCasa/universal-silabs-flasher/compare/v0.0.15...v0.0.16) #### What's Changed - Compatibility with gpiod 2.x by [@​puddly](https://togithub.com/puddly) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/50](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/50) - Migrate to zigpy CI and pre-commit hooks by [@​puddly](https://togithub.com/puddly) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/53](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/53) - Bootloader reset improvements by [@​darkxst](https://togithub.com/darkxst) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/51](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/51) - Disconnect stream writer without closing the transport by [@​puddly](https://togithub.com/puddly) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/58](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/58) - EBL parsing by [@​puddly](https://togithub.com/puddly) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/11](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/11) - Update README.md with bootloader reset option by [@​darkxst](https://togithub.com/darkxst) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/54](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/54) - Run `isort` to fix CI by [@​puddly](https://togithub.com/puddly) in [https://github.com/NabuCasa/universal-silabs-flasher/pull/59](https://togithub.com/NabuCasa/universal-silabs-flasher/pull/59) **Full Changelog**: NabuCasa/universal-silabs-flasher@v0.0.15...v0.0.16 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/marinatedconcrete/config). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMDMuMSIsInVwZGF0ZWRJblZlciI6IjM3LjEwMy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Resetting the GPIO pin state after releasing the line is now apparently deprecated, as the new kernel API reverts the pins to an unknown "default" state after the last line request is released.