Cleanup to make pylint happy

pylintrc

@@ -155,7 +155,6 @@ disable=raw-checker-failed,
         missing-function-docstring,
         missing-class-docstring,
         missing-module-docstring,
-        too-many-positional-arguments
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option

sa818.py

@@ -78,10 +78,7 @@ def __init__(self, port=None, baud=DEFAULT_BAUDRATE):
         # Send initialization command and check the reply
         self.send(self.INIT)
         reply = self.readline()
-        if reply == "+DMOCONNECT:0":
-          # if expected response, proceed
-          break
-        else:
+        if reply != "+DMOCONNECT:0":
           # if unexpected response, try another port
           logger.debug("Port %s not SA818: %s", _port, reply)
           self.serial.close()
@@ -133,6 +130,7 @@ def version(self):
     return version
 
   def set_radio(self, frequency, offset, bw, squelch, ctcss, dcs, tail):
+    # pylint: disable=too-many-locals
     tone = ctcss if ctcss else dcs
     if tone:                # 0000 = No ctcss or dcs tone
       tx_tone, rx_tone = tone