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

[device]: Fix a bug that psuutil cannot access gpio sysfs to get PSU status #1789

Merged
merged 8 commits into from Jun 21, 2018
Merged

[device]: Fix a bug that psuutil cannot access gpio sysfs to get PSU status #1789

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7637066
Add Celestica seastone dx010 psuutil.py plugins
pphuchar Jun 9, 2018
000a63a
Fix psuutil to use dynamic sysfs gpio base.
pphuchar Jun 13, 2018
f393c4c
add exeption handler for psuutil
Jun 13, 2018
a0db28d
Merge branch 'master' into Add-psuutil-seastone2-dx010
Jun 13, 2018
2846424
remove sys.exit from psuutil
Jun 13, 2018
a5c64bf
Change logging format for psuutil
Jun 15, 2018
1c991b5
Update exception handler
Jun 19, 2018
bcc3484
Remove init psu gpio method
Jun 21, 2018
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
49 changes: 19 additions & 30 deletions device/celestica/x86_64-cel_seastone-r0/plugins/psuutil.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@
# Platform-specific PSU status interface for SONiC
#

import os.path
import os


try:
from sonic_psu.psu_base import PsuBase
Expand All @@ -18,42 +19,32 @@ def __init__(self):
PsuBase.__init__(self)
# DX010 PSU pin mapping
self.psu = [
{'base':216}, # Reserved
{'base': self.get_gpio_base()},
{'abs':27, 'power':22},
{'abs':28, 'power':25}
]

def init_psu_gpio(self, pinnum):
# export pin, input as default
gpio_base = self.psu[0]['base']
export_file = "/sys/class/gpio/export"
direction_file = '/sys/class/gpio/gpio' + str(gpio_base+pinnum) + '/direction'

try:
with open(export_file, 'w') as fd:
fd.write(str(gpio_base+pinnum))
except Exception as error:
logging.error("Unable to export gpio ", pinnum)
def get_gpio_base(self):
sys_gpio_dir = "/sys/class/gpio"
for r in os.listdir(sys_gpio_dir):
if "gpiochip" in r:
return int(r[8:],10)
return 216 #Reserve


# Get a psu status and presence
def read_psu_statuses(self, pinnum):
sys_gpio_dir = "/sys/class/gpio"
retval = 'ERR'
gpio_base = self.psu[0]['base']

gpio_dir = sys_gpio_dir + '/gpio' + str(gpio_base+pinnum)
gpio_file = gpio_dir + "/value"

# init gpio
if (not os.path.isdir(gpio_dir)):
self.init_psu_gpio(pinnum)

try:
with open(gpio_file, 'r') as fd:
retval = fd.read()
except Exception as error:
logging.error("Unable to open ", gpio_file, "file !")
except IOError:
raise IOError("Unable to open " + gpio_file + "file !")

retval = retval.rstrip('\r\n')
return retval
Expand All @@ -76,11 +67,10 @@ def get_psu_status(self, index):
"""
status = 0
psu_status = self.read_psu_statuses(self.psu[index]['power'])
if (psu_status != 'ERR'):
psu_status = int(psu_status, 10)
# Check for PSU status
if (psu_status == 1):
status = 1
psu_status = int(psu_status, 10)
# Check for PSU status
if (psu_status == 1):
status = 1

return status

Expand All @@ -93,10 +83,9 @@ def get_psu_presence(self, index):
"""
status = 0
psu_absence = self.read_psu_statuses(self.psu[index]['abs'])
if (psu_absence != 'ERR'):
psu_absence = (int(psu_absence, 10))
# Check for PSU presence
if (psu_absence == 0):
status = 1
psu_absence = (int(psu_absence, 10))
# Check for PSU presence
if (psu_absence == 0):
status = 1

return status