commit 7a4d77a86e98f02839213ce040c1d8f2f6be31cc
author Brad Bishop <bradleyb@fuzziesquirrel.com> Thu Sep 08 23:26:17 2016 -0400
committer Patrick Williams <patrick@stwcx.xyz> Mon Sep 12 18:47:24 2016 +0000
tree 1bbb7e8291400a6b9b82271ec398c8e1720b74ac
parent cf98415545b993e4a1e24cbea7ad076a6b5ebb1d

chassis: pep8 fixes

Change-Id: I6dc52616ce30660f2e6992f6e1a52d44dfec1cd7
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

pychassisctl/chassis_control.py

diff --git a/pychassisctl/chassis_control.py b/pychassisctl/chassis_control.py
index 373d876..20532c9 100644
--- a/pychassisctl/chassis_control.py
+++ b/pychassisctl/chassis_control.py
@@ -20,7 +20,7 @@
 
 class ChassisControlObject(DbusProperties, DbusObjectManager):
     def getUuid(self):
-        uuid = "";
+        uuid = ""
         try:
             with open(MACHINE_ID) as f:
                 data = f.readline().rstrip('\n')
@@ -95,13 +95,11 @@
         bus.add_signal_receiver(self.SystemStateHandler,
                                 signal_name="GotoSystemState")
 
-
     def getInterface(self, name):
         o = self.dbus_objects[name]
         obj = bus.get_object(o['bus_name'], o['object_name'], introspect=False)
         return dbus.Interface(obj, o['interface_name'])
 
-
     @dbus.service.method(DBUS_NAME,
                          in_signature='', out_signature='')
     def setIdentify(self):
@@ -152,7 +150,7 @@
     def reboot(self):
         print "Rebooting"
         if self.getPowerState() == POWER_OFF:
-            self.powerOn();
+            self.powerOn()
         else:
             self.Set(DBUS_NAME, "reboot", 1)
             self.powerOff()
@@ -163,7 +161,7 @@
     def softReboot(self):
         print "Soft Rebooting"
         if self.getPowerState() == POWER_OFF:
-            self.powerOn();
+            self.powerOn()
         else:
             self.Set(DBUS_NAME, "reboot", 1)
             self.softPowerOff()
@@ -178,8 +176,7 @@
     ## Signal handler
 
     def SystemStateHandler(self, state_name):
-        if (
-                state_name == "HOST_POWERED_OFF" or state_name == "HOST_POWERED_ON"):
+        if state_name in ["HOST_POWERED_OFF", "HOST_POWERED_ON"]:
             intf = self.getInterface('settings')
             intf.Set("org.openbmc.settings.Host", "system_state", state_name)
 
@@ -193,14 +190,14 @@
         if state == POWER_OFF:
             self.powerOn()
         elif state == POWER_ON:
-            self.softPowerOff();
+            self.softPowerOff()
 
     def long_power_button_signal_handler(self):
         print "Long-press button, hard power off"
-        self.powerOff();
+        self.powerOff()
 
     def softreset_button_signal_handler(self):
-        self.softReboot();
+        self.softReboot()
 
     def host_watchdog_signal_handler(self):
         print "Watchdog Error, Hard Rebooting"
@@ -239,4 +236,3 @@
 
     print "Running ChassisControlService"
     mainloop.run()
-