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 <[email protected]>
… for SysfsGPIO

Update the SysfsGPIO, MatchedSysfsGPIO to add the new invert (active-low)
attribute.

Signed-off-by: Perry Melange <[email protected]>
@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 <[email protected]>
@pmelange pmelange force-pushed the 25.0.1-sysfsgpio_active_low branch from 7b1cb10 to 03d2f1f Compare September 11, 2025 17:26
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).
✅ 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 70 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 15 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant