Skip to content

Conversation

@pmelange
Copy link
Contributor

Description

Adds the invert (active-low) attribute to SysfsGPIO, NetworkSysfsGPIO, MatchedSysfsGPIO, GPIOSysFSExport, GpioDigitalOutputDriver and the GpioDigitalOutput agent.

Updates to the documentation and the sysfsgpio examples

Checklist

  • Documentation for the feature
  • Tests for the feature
  • The arguments and description in doc/configuration.rst have been updated
  • PR has been tested
The sysfsgpio resources (SysfsGPIO, NetworkSysfsGPIO and MatchedSysfsGPIO), the GPIOSysFSExport, the driver GpioDigitalOutputDriver and the GpioDigitalOutput agent have been modified to have an additional optional invert (active-low) attribute (default False) which can be used to invert the logical value used on the gpio line. Signed-off-by: Perry Melange <isprotejesvalkata@gmail.com>
… for SysfsGPIO Update the SysfsGPIO, MatchedSysfsGPIO to add the new invert (active-low) attribute. Signed-off-by: Perry Melange <isprotejesvalkata@gmail.com>
@pmelange pmelange force-pushed the 25.0.1-sysfsgpio_active_low branch from 3da7f0f to 7b1cb10 Compare September 11, 2025 16:57
Add the new invert (active-low) attribute to the examples and add calls to the new invert method Signed-off-by: Perry Melange <isprotejesvalkata@gmail.com>
@pmelange pmelange force-pushed the 25.0.1-sysfsgpio_active_low branch from 7b1cb10 to 03d2f1f Compare September 11, 2025 17:26
@codecov
Copy link

codecov bot commented Sep 11, 2025

Codecov Report

❌ Patch coverage is 62.50000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.4%. Comparing base (22f4b81) to head (03d2f1f).
⚠️ Report is 38 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
labgrid/util/agents/sysfsgpio.py 70.0% 3 Missing ⚠️
labgrid/driver/gpiodriver.py 0.0% 2 Missing ⚠️
labgrid/remote/exporter.py 0.0% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (22f4b81) and HEAD (03d2f1f). Click for more details.

HEAD has 71 uploads less than BASE
Flag BASE (22f4b81) HEAD (03d2f1f)
3.11 15 1
3.13 15 1
3.12 15 1
3.9 15 1
3.10 16 1
Additional details and impacted files
@@ Coverage Diff @@ ## master #1730 +/- ## ========================================= - Coverage 55.6% 45.4% -10.3%  ========================================= Files 172 172 Lines 13495 13502 +7 ========================================= - Hits 7514 6135 -1379  - Misses 5981 7367 +1386 
Flag Coverage Δ
3.10 45.4% <62.5%> (-10.3%) ⬇️
3.11 45.4% <62.5%> (-10.3%) ⬇️
3.12 45.4% <62.5%> (-10.3%) ⬇️
3.13 45.4% <62.5%> (-10.3%) ⬇️
3.9 45.4% <62.5%> (-10.3%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant