Skip to content

Allow same GPIO for switching and sensing #223

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

Open
xaver-k wants to merge 4 commits into
base: devel
Choose a base branch
from
Open

Allow same GPIO for switching and sensing #223

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
99b36af
re-use GPIO-object if GPIOs for switching and sensing are the same
xaver-k Aug 1, 2021
891e81f
switch inversion of GPIO-pins
xaver-k Aug 2, 2021
e29b10b
clean up handling of inverted GPIO pins
xaver-k Aug 3, 2021
29f7645
remove python3-only type hint
xaver-k Sep 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
80 changes: 44 additions & 36 deletions octoprint_psucontrol/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,7 @@ def cleanup_gpio(self):
for k, pin in self._configuredGPIOPins.items():
self._logger.debug("Cleaning up {} pin {}".format(k, pin.name))
try:
# should work, even if we try to clean up the same GPIO twice
pin.close()
except Exception:
self._logger.exception(
Expand All @@ -181,44 +182,55 @@ def configure_gpio(self):
self._logger.info("Using GPIO for On/Off")
self._logger.info("Configuring GPIO for pin {}".format(self.config['onoffGPIOPin']))

if not self.config['invertonoffGPIOPin']:
initial_output = 'low'
else:
initial_output = 'high'

try:
pin = periphery.GPIO(self.config['GPIODevice'], self.config['onoffGPIOPin'], initial_output)
self._configuredGPIOPins['switch'] = pin
pin = periphery.CdevGPIO(
path=self.config['GPIODevice'],
line=self.config['onoffGPIOPin'],
direction="out",
inverted=self.config['invertonoffGPIOPin']
)
except Exception:
self._logger.exception(
"Exception while setting up GPIO pin {}".format(self.config['onoffGPIOPin'])
)
else:
self._configuredGPIOPins['switch'] = pin

if self.config['sensingMethod'] == 'GPIO':
self._logger.info("Using GPIO sensing to determine PSU on/off state.")
self._logger.info("Configuring GPIO for pin {}".format(self.config['senseGPIOPin']))


if not SUPPORTS_LINE_BIAS:
if self.config['senseGPIOPinPUD'] != '':
self._logger.warning("Kernel version 5.5 or greater required for GPIO bias. Using 'default'.")
bias = "default"
elif self.config['senseGPIOPinPUD'] == '':
bias = "disable"
elif self.config['senseGPIOPinPUD'] == 'PULL_UP':
bias = "pull_up"
elif self.config['senseGPIOPinPUD'] == 'PULL_DOWN':
bias = "pull_down"
else:
bias = "default"

try:
pin = periphery.CdevGPIO(path=self.config['GPIODevice'], line=self.config['senseGPIOPin'], direction='in', bias=bias)

if self.config['switchingMethod'] == 'GPIO' and self.config['senseGPIOPin'] == self.config['onoffGPIOPin']:
self._logger.info("Sensing through same pin as switching.")
self._configuredGPIOPins['sense'] = pin
except Exception:
self._logger.exception(
"Exception while setting up GPIO pin {}".format(self.config['senseGPIOPin'])
)
else:
self._logger.info("Configuring GPIO for pin {}".format(self.config['senseGPIOPin']))
if not SUPPORTS_LINE_BIAS:
if self.config['senseGPIOPinPUD'] != '':
self._logger.warning("Kernel version 5.5 or greater required for GPIO bias. Using 'default'.")
bias = "default"
elif self.config['senseGPIOPinPUD'] == '':
bias = "disable"
elif self.config['senseGPIOPinPUD'] == 'PULL_UP':
bias = "pull_up"
elif self.config['senseGPIOPinPUD'] == 'PULL_DOWN':
bias = "pull_down"
else:
bias = "default"

try:
pin = periphery.CdevGPIO(
path=self.config['GPIODevice'],
line=self.config['senseGPIOPin'],
direction='in',
bias=bias,
inverted=self.config['invertsenseGPIOPin']
)
self._configuredGPIOPins['sense'] = pin
except Exception:
self._logger.exception(
"Exception while setting up GPIO pin {}".format(self.config['senseGPIOPin'])
)


def _get_plugin_key(self, implementation):
Expand Down Expand Up @@ -248,15 +260,15 @@ def _check_psu_state(self):
self._logger.debug("Polling PSU state...")

if self.config['sensingMethod'] == 'GPIO':
r = 0
r = False
try:
r = self._configuredGPIOPins['sense'].read()
except Exception:
self._logger.exception("Exception while reading GPIO line")

self._logger.debug("Result: {}".format(r))

new_isPSUOn = r ^ self.config['invertsenseGPIOPin']
new_isPSUOn = r

self.isPSUOn = new_isPSUOn
elif self.config['sensingMethod'] == 'SYSTEM':
Expand Down Expand Up @@ -472,10 +484,8 @@ def turn_psu_on(self):
self._logger.debug("On system command returned: {}".format(r))
elif self.config['switchingMethod'] == 'GPIO':
self._logger.debug("Switching PSU On Using GPIO: {}".format(self.config['onoffGPIOPin']))
pin_output = bool(1 ^ self.config['invertonoffGPIOPin'])

try:
self._configuredGPIOPins['switch'].write(pin_output)
self._configuredGPIOPins['switch'].write(True)
except Exception :
self._logger.exception("Exception while writing GPIO line")
return
Expand Down Expand Up @@ -538,10 +548,8 @@ def turn_psu_off(self):
self._logger.debug("Off system command returned: {}".format(r))
elif self.config['switchingMethod'] == 'GPIO':
self._logger.debug("Switching PSU Off Using GPIO: {}".format(self.config['onoffGPIOPin']))
pin_output = bool(0 ^ self.config['invertonoffGPIOPin'])

try:
self._configuredGPIOPins['switch'].write(pin_output)
self._configuredGPIOPins['switch'].write(False)
except Exception:
self._logger.exception("Exception while writing GPIO line")
return
Expand Down