Massive pylinting session and added Github Actions

[Adafruit_Blinka-hackapet.git] / src / adafruit_blinka / microcontroller / tegra / t186 / pin.py

diff --git a/src/adafruit_blinka/microcontroller/tegra/t186/pin.py b/src/adafruit_blinka/microcontroller/tegra/t186/pin.py
index f033b5c4149fbbf089ade427b5894491374bdcf7..ba06449636cfad3e22dcbe6a644636ba9a3db0a6 100644 (file)
--- a/src/adafruit_blinka/microcontroller/tegra/t186/pin.py
+++ b/src/adafruit_blinka/microcontroller/tegra/t186/pin.py
@@ -1,11 +1,15 @@
-import sys
+"""Tegra T186 pin names"""
+

 import atexit
 import Jetson.GPIO as GPIO
 import atexit
 import Jetson.GPIO as GPIO

+

 GPIO.setmode(GPIO.TEGRA_SOC)
 GPIO.setmode(GPIO.TEGRA_SOC)

-GPIO.setwarnings(False)   # shh!
+GPIO.setwarnings(False)  # shh!
+

 
 

-# Pins dont exist in CPython so...lets make our own!

 class Pin:
 class Pin:

+    """Pins dont exist in CPython so...lets make our own!"""
+

     IN = 0
     OUT = 1
     LOW = 0
     IN = 0
     OUT = 1
     LOW = 0


@@ -28,7 +32,8 @@ class Pin:
         return self.id == other
 
     def init(self, mode=IN, pull=None):
         return self.id == other
 
     def init(self, mode=IN, pull=None):

-        if mode != None:
+        """Initialize the Pin"""
+        if mode is not None:

             if mode == self.IN:
                 self._mode = self.IN
                 GPIO.setup(self.id, GPIO.IN)
             if mode == self.IN:
                 self._mode = self.IN
                 GPIO.setup(self.id, GPIO.IN)


@@ -37,7 +42,7 @@ class Pin:
                 GPIO.setup(self.id, GPIO.OUT)
             else:
                 raise RuntimeError("Invalid mode for pin: %s" % self.id)
                 GPIO.setup(self.id, GPIO.OUT)
             else:
                 raise RuntimeError("Invalid mode for pin: %s" % self.id)

-        if pull != None:
+        if pull is not None:

             if self._mode != self.IN:
                 raise RuntimeError("Cannot set pull resistor on output")
             if pull == self.PULL_UP:
             if self._mode != self.IN:
                 raise RuntimeError("Cannot set pull resistor on output")
             if pull == self.PULL_UP:


@@ -48,54 +53,61 @@ class Pin:
                 raise RuntimeError("Invalid pull for pin: %s" % self.id)
 
     def value(self, val=None):
                 raise RuntimeError("Invalid pull for pin: %s" % self.id)
 
     def value(self, val=None):

-        if val != None:
+        """Set or return the Pin Value"""
+        if val is not None:

             if val == self.LOW:
                 self._value = val
                 GPIO.output(self.id, val)
             if val == self.LOW:
                 self._value = val
                 GPIO.output(self.id, val)

-            elif val == self.HIGH:
+                return None
+            if val == self.HIGH:

                 self._value = val
                 GPIO.output(self.id, val)
                 self._value = val
                 GPIO.output(self.id, val)

-            else:
-                raise RuntimeError("Invalid value for pin")
-        else:
-            return GPIO.input(self.id)
+                return None
+            raise RuntimeError("Invalid value for pin")
+        return GPIO.input(self.id)

 
 

+    # pylint: disable=no-method-argument

     @atexit.register
     def cleanup():
     @atexit.register
     def cleanup():

+        """Clean up pins"""

         print("Exiting... \nCleaning up pins")
         GPIO.cleanup()
 
         print("Exiting... \nCleaning up pins")
         GPIO.cleanup()
 

+    # pylint: enable=no-method-argument
+
+

 # Cannot be used as GPIO
 # Cannot be used as GPIO

-SDA = Pin('GPIO_SEN9')
-SCL = Pin('GPIO_SEN8')
-SDA_1 = Pin('GEN1_I2C_SDA')
-SCL_1 = Pin('GEN1_I2C_SCL')
-
-J04 = Pin('AUD_MCLK')
-J06 = Pin('GPIO_AUD1')
-AA02 = Pin('CAN_GPIO2')
-N06 = Pin('GPIO_CAM7')
-N04 = Pin('GPIO_CAM5')
-N05 = Pin('GPIO_CAM6')
-N03 = Pin('GPIO_CAM4')
-AA01 = Pin('CAN_GPIO1')
-I05 = Pin('GPIO_PQ5')
-T03 = Pin('UART1_CTS')
-T02 = Pin('UART1_RTS')
-J00 = Pin('DAP1_SCLK')
-J03 = Pin('DAP1_FS')
-J02 = Pin('DAP1_DIN')
-J01 = Pin('DAP1_DOUT')
-P17 = Pin('GPIO_EXP_P17')
-AA00 = Pin('CAN0_GPIO0')
-Y01 = Pin('GPIO_MDM2')
-P16 = Pin('GPIO_EXP_P16')
-I04 = Pin('GPIO_PQ4')
-J05 = Pin('GPIO_AUD0')
+SDA = Pin("GPIO_SEN9")
+SCL = Pin("GPIO_SEN8")
+SDA_1 = Pin("GEN1_I2C_SDA")
+SCL_1 = Pin("GEN1_I2C_SCL")
+
+J04 = Pin("AUD_MCLK")
+J06 = Pin("GPIO_AUD1")
+AA02 = Pin("CAN_GPIO2")
+N06 = Pin("GPIO_CAM7")
+N04 = Pin("GPIO_CAM5")
+N05 = Pin("GPIO_CAM6")
+N03 = Pin("GPIO_CAM4")
+AA01 = Pin("CAN_GPIO1")
+I05 = Pin("GPIO_PQ5")
+T03 = Pin("UART1_CTS")
+T02 = Pin("UART1_RTS")
+J00 = Pin("DAP1_SCLK")
+J03 = Pin("DAP1_FS")
+J02 = Pin("DAP1_DIN")
+J01 = Pin("DAP1_DOUT")
+P17 = Pin("GPIO_EXP_P17")
+AA00 = Pin("CAN0_GPIO0")
+Y01 = Pin("GPIO_MDM2")
+P16 = Pin("GPIO_EXP_P16")
+I04 = Pin("GPIO_PQ4")
+J05 = Pin("GPIO_AUD0")

 
 i2cPorts = (
 
 i2cPorts = (

-    (1, SCL, SDA), (0, SCL_1, SDA_1),
+    (1, SCL, SDA),
+    (0, SCL_1, SDA_1),

 )
 
 # ordered as spiId, sckId, mosiId, misoId
 )
 
 # ordered as spiId, sckId, mosiId, misoId

-spiPorts = ((3, N03, N05, N04), )
+spiPorts = ((3, N03, N05, N04),)