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.

Sign up for GitHub

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

Jump to bottom

protocol/driver: add DigitalInputProtocol, GpioDigitalInputDriver #1458

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

protocol/driver: add DigitalInputProtocol, GpioDigitalInputDriver #1458

wants to merge 2 commits into from
+148 −10

Commits on Aug 19, 2024

  1. protocol/driver: add DigitalInputProtocol, GpioDigitalInputDriver 

    this adds a new protocol `DigitalInputProtocol` and a GPIO driver
that implements it.
The `DigitalOutputProtocol` ABC now also inherits it, ensuring it can be
used in all existing driver implementations.
Additionally the client now automatically uses `DigitalInputProtocol`
when the command is `io get`.

By seperating the input from output protocol it is possible
to make sure the GPIO line is configured in direction `input` in sysfs,
which avoids possible shorts when reading out digital states from DUT's.
Otherwise, one would have to rely on the circuitry of the GPIO line to
prevent that.

Signed-off-by: Felix Zwettler <[email protected]>
    @flxzt
    flxzt committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    2be6470 View commit details
    Browse the repository at this point in the history

  2. remote: restore previous client io command behaviour 

    Signed-off-by: Felix Zwettler <[email protected]>
    @flxzt
    flxzt committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    97819ca View commit details
    Browse the repository at this point in the history