diff --git a/pysystemmgr/obmc/system/__init__.py b/pysystemmgr/obmc/system/__init__.py
index 7bcea9e..2d5b752 100644
--- a/pysystemmgr/obmc/system/__init__.py
+++ b/pysystemmgr/obmc/system/__init__.py
@@ -18,9 +18,9 @@
 
 
 def convertGpio(name):
-    offset = int(''.join(list(filter(str.isdigit, name))))
+    offset = int("".join(list(filter(str.isdigit, name))))
     port = list(filter(str.isalpha, name.upper()))
-    a = ord(port[-1]) - ord('A')
+    a = ord(port[-1]) - ord("A")
     if len(port) > 1:
         a += 26
     base = a * 8 + GPIO_BASE
diff --git a/pysystemmgr/setup.py b/pysystemmgr/setup.py
index a921e9c..adfa4a2 100644
--- a/pysystemmgr/setup.py
+++ b/pysystemmgr/setup.py
@@ -1,7 +1,8 @@
 from distutils.core import setup
 
-setup(name='pysystemmgr',
-      version='1.0',
-      packages=['obmc.system'],
-      scripts=['system_manager.py'],
-      )
+setup(
+    name="pysystemmgr",
+    version="1.0",
+    packages=["obmc.system"],
+    scripts=["system_manager.py"],
+)
diff --git a/pysystemmgr/system_manager.py b/pysystemmgr/system_manager.py
index d9761b5..82ab903 100644
--- a/pysystemmgr/system_manager.py
+++ b/pysystemmgr/system_manager.py
@@ -15,15 +15,14 @@
 import obmc.inventory
 import obmc.system
 
-DBUS_NAME = 'org.openbmc.managers.System'
-OBJ_NAME = '/org/openbmc/managers/System'
-INTF_SENSOR = 'org.openbmc.SensorValue'
+DBUS_NAME = "org.openbmc.managers.System"
+OBJ_NAME = "/org/openbmc/managers/System"
+INTF_SENSOR = "org.openbmc.SensorValue"
+
 
 class SystemManager(DbusProperties, DbusObjectManager):
     def __init__(self, bus, obj_name):
-        super(SystemManager, self).__init__(
-            conn=bus,
-            object_path=obj_name)
+        super(SystemManager, self).__init__(conn=bus, object_path=obj_name)
         self.bus = bus
 
         # replace symbolic path in ID_LOOKUP
@@ -31,7 +30,8 @@
             for key in System.ID_LOOKUP[category]:
                 val = System.ID_LOOKUP[category][key]
                 new_val = val.replace(
-                    "<inventory_root>", obmc.inventory.INVENTORY_ROOT)
+                    "<inventory_root>", obmc.inventory.INVENTORY_ROOT
+                )
                 System.ID_LOOKUP[category][key] = new_val
 
         print("SystemManager Init Done")
@@ -41,75 +41,87 @@
         intf_name = INTF_SENSOR
         try:
             obj_path = System.ID_LOOKUP[category][key]
-            parts = obj_path.split('/')
-            if (parts[3] != 'sensors'):
-                print ("ERROR SystemManager: SENSOR only supported type")
+            parts = obj_path.split("/")
+            if parts[3] != "sensors":
+                print("ERROR SystemManager: SENSOR only supported type")
                 intf_name = ""
         except Exception as e:
-            print ("ERROR SystemManager: "+str(e)+" not found in lookup")
+            print("ERROR SystemManager: " + str(e) + " not found in lookup")
 
         return [obj_path, intf_name]
 
-    @dbus.service.method(DBUS_NAME, in_signature='ss', out_signature='(ss)')
+    @dbus.service.method(DBUS_NAME, in_signature="ss", out_signature="(ss)")
     def getObjectFromId(self, category, key):
         return self.doObjectLookup(category, key)
 
-    @dbus.service.method(DBUS_NAME, in_signature='sy', out_signature='(ss)')
+    @dbus.service.method(DBUS_NAME, in_signature="sy", out_signature="(ss)")
     def getObjectFromByteId(self, category, key):
         byte = int(key)
         return self.doObjectLookup(category, byte)
 
-    @dbus.service.method(DBUS_NAME, in_signature='s', out_signature='sis')
+    @dbus.service.method(DBUS_NAME, in_signature="s", out_signature="sis")
     def gpioInit(self, name):
-        gpio_path = ''
+        gpio_path = ""
         gpio_num = -1
-        r = ['', gpio_num, '']
+        r = ["", gpio_num, ""]
         if name not in System.GPIO_CONFIG:
             # TODO: Better error handling
-            msg = "ERROR: "+name+" not found in GPIO config table"
+            msg = "ERROR: " + name + " not found in GPIO config table"
             print(msg)
             raise Exception(msg)
         else:
 
             gpio_num = -1
             gpio = System.GPIO_CONFIG[name]
-            if 'gpio_num' in System.GPIO_CONFIG[name]:
-                gpio_num = gpio['gpio_num']
+            if "gpio_num" in System.GPIO_CONFIG[name]:
+                gpio_num = gpio["gpio_num"]
             else:
-                if 'gpio_pin' in System.GPIO_CONFIG[name]:
-                    gpio_num = obmc.system.convertGpio(gpio['gpio_pin'])
+                if "gpio_pin" in System.GPIO_CONFIG[name]:
+                    gpio_num = obmc.system.convertGpio(gpio["gpio_pin"])
                 else:
-                    msg = "ERROR: SystemManager - GPIO lookup failed for "+name
+                    msg = (
+                        "ERROR: SystemManager - GPIO lookup failed for " + name
+                    )
                     print(msg)
                     raise Exception(msg)
 
-            if (gpio_num != -1):
-                r = [obmc.enums.GPIO_DEV, gpio_num, gpio['direction']]
+            if gpio_num != -1:
+                r = [obmc.enums.GPIO_DEV, gpio_num, gpio["direction"]]
         return r
 
-    @dbus.service.method(DBUS_NAME, in_signature='',
-                         out_signature='ssa(sb)a(sb)a(sbb)ssssa(sb)')
+    @dbus.service.method(
+        DBUS_NAME, in_signature="", out_signature="ssa(sb)a(sb)a(sbb)ssssa(sb)"
+    )
     def getGpioConfiguration(self):
-        power_config = System.GPIO_CONFIGS.get('power_config', {})
-        power_good_in = power_config.get('power_good_in', '')
-        latch_out = power_config.get('latch_out', '')
-        power_up_outs = power_config.get('power_up_outs', [])
-        reset_outs = power_config.get('reset_outs', [])
-        pci_reset_outs = power_config.get('pci_reset_outs', [])
-        hostctl_config = System.GPIO_CONFIGS.get('hostctl_config', {})
-        fsi_data = hostctl_config.get('fsi_data', '')
-        fsi_clk = hostctl_config.get('fsi_clk', '')
-        fsi_enable = hostctl_config.get('fsi_enable', '')
-        cronus_sel = hostctl_config.get('cronus_sel', '')
-        optionals = hostctl_config.get('optionals', [])
-        r = [power_good_in, latch_out, power_up_outs, reset_outs,
-             pci_reset_outs, fsi_data, fsi_clk, fsi_enable, cronus_sel,
-             optionals]
+        power_config = System.GPIO_CONFIGS.get("power_config", {})
+        power_good_in = power_config.get("power_good_in", "")
+        latch_out = power_config.get("latch_out", "")
+        power_up_outs = power_config.get("power_up_outs", [])
+        reset_outs = power_config.get("reset_outs", [])
+        pci_reset_outs = power_config.get("pci_reset_outs", [])
+        hostctl_config = System.GPIO_CONFIGS.get("hostctl_config", {})
+        fsi_data = hostctl_config.get("fsi_data", "")
+        fsi_clk = hostctl_config.get("fsi_clk", "")
+        fsi_enable = hostctl_config.get("fsi_enable", "")
+        cronus_sel = hostctl_config.get("cronus_sel", "")
+        optionals = hostctl_config.get("optionals", [])
+        r = [
+            power_good_in,
+            latch_out,
+            power_up_outs,
+            reset_outs,
+            pci_reset_outs,
+            fsi_data,
+            fsi_clk,
+            fsi_enable,
+            cronus_sel,
+            optionals,
+        ]
         print("Power GPIO config: " + str(r))
         return r
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     dbus.mainloop.glib.DBusGMainLoop(set_as_default=True)
     bus = get_dbus()
     obj = SystemManager(bus, OBJ_NAME)