Updated for STM32->STM32F405 PlatformDetect rename

[Adafruit_Blinka-hackapet.git] / src / adafruit_blinka / microcontroller / nova / pwmout.py

diff --git a/src/adafruit_blinka/microcontroller/nova/pwmout.py b/src/adafruit_blinka/microcontroller/nova/pwmout.py
index 57aa0e514d561a1425f052001f24b28fe8eaea23..2ff0c2594e909c7af5c0e27b21ee90e3e0baf5ed 100644 (file)
--- a/src/adafruit_blinka/microcontroller/nova/pwmout.py
+++ b/src/adafruit_blinka/microcontroller/nova/pwmout.py
@@ -1,19 +1,26 @@
-import os
-import digitalio
+"""PWMOut Class for Binho Nova"""

 
 try:
     from microcontroller.pin import pwmOuts
 except ImportError:
 
 try:
     from microcontroller.pin import pwmOuts
 except ImportError:

-    raise RuntimeError("No PWM outputs defined for this board")
+    raise RuntimeError("No PWM outputs defined for this board") from ImportError

 
 from microcontroller.pin import Pin
 
 
 from microcontroller.pin import Pin
 

+
+# pylint: disable=unnecessary-pass

 class PWMError(IOError):
     """Base class for PWM errors."""
 class PWMError(IOError):
     """Base class for PWM errors."""

+

     pass
 
 
     pass
 
 

-class PWMOut(object):
+# pylint: enable=unnecessary-pass
+
+
+class PWMOut:
+    """Pulse Width Modulation Output Class"""
+

     # Nova instance
     _nova = None
     MAX_CYCLE_LEVEL = 1024
     # Nova instance
     _nova = None
     MAX_CYCLE_LEVEL = 1024


@@ -38,26 +45,22 @@ class PWMOut(object):
 
         """
         if PWMOut._nova is None:
 
         """
         if PWMOut._nova is None:

-            if Pin._nova is not None:
-                # check if Pin already connected to Binho Nova
-                PWMOut._nova = Pin._nova
-            else:
-                from binhoHostAdapter import binhoHostAdapter
-                from binhoHostAdapter import binhoUtilities
+            # pylint: disable=import-outside-toplevel
+            from adafruit_blinka.microcontroller.nova import Connection

 
 

-                utilities = binhoUtilities.binhoUtilities()
-                devices = utilities.listAvailableDevices()
+            # pylint: enable=import-outside-toplevel

 
 

-                if len(devices) > 0:
-                    PWMOut._nova = binhoHostAdapter.binhoHostAdapter(devices[0])
-                else:
-                    raise RuntimeError('No Binho Host Adapter found!')
+            PWMOut._nova = Connection.getInstance()

 
 

+        PWMOut._nova.setOperationMode(0, "IO")

         self._pwmpin = None
         self._pwmpin = None

+        self._channel = None
+        self._enable = False

         self._open(pin, duty_cycle, frequency, variable_frequency)
 
     def __del__(self):
         self.deinit()
         self._open(pin, duty_cycle, frequency, variable_frequency)
 
     def __del__(self):
         self.deinit()

+        PWMOut._nova.close()

 
     def __enter__(self):
         return self
 
     def __enter__(self):
         return self


@@ -76,11 +79,13 @@ class PWMOut(object):
         if self._channel is None:
             raise RuntimeError("No PWM channel found for this Pin")
 
         if self._channel is None:
             raise RuntimeError("No PWM channel found for this Pin")
 

+        if variable_frequency:
+            print("Variable Frequency is not supported, continuing without it...")
+

         PWMOut._nova.setIOpinMode(self._pwmpin, Pin.PWM)
 
         # set frequency
         self.frequency = freq
         PWMOut._nova.setIOpinMode(self._pwmpin, Pin.PWM)
 
         # set frequency
         self.frequency = freq

-        PWMOut._nova.setIOpinPWMFreq(self._pwmpin, self.frequency)

         # set period
         self._period = self._get_period()
 
         # set period
         self._period = self._get_period()
 


@@ -90,32 +95,33 @@ class PWMOut(object):
         self._set_enabled(True)
 
     def deinit(self):
         self._set_enabled(True)
 
     def deinit(self):

-      try:

         """Deinit the Nova PWM."""
         """Deinit the Nova PWM."""

-        if self._channel is not None:
-            #self.duty_cycle = 0
-            self._set_enabled(False) # make to disable before unexport
-            """
-            try:
-                #unexport_path = os.path.join(channel_path, self._unexport_path)
-                with open(os.path.join(channel_path, self._unexport_path), "w") as f_unexport:
-                    f_unexport.write("%d\n" % self._pwmpin)
-            except IOError as e:
-                raise PWMError(e.errno, "Unexporting PWM pin: " + e.strerror)
-                """
-      except Exception as e:
-          # due to a race condition for which I have not yet been
-          # able to find the root cause, deinit() often fails
-          # but it does not effect future usage of the pwm pin
-          print("warning: failed to deinitialize pwm pin {0}:{1} due to: {2}\n".format(self._channel, self._pwmpin, type(e).__name__))
-      finally:
-          self._channel = None
-          self._pwmpin = None
+        # pylint: disable=broad-except
+        try:
+            if self._channel is not None:
+                # self.duty_cycle = 0
+                self._set_enabled(False)  # make to disable before unexport
+
+        except Exception as e:
+            # due to a race condition for which I have not yet been
+            # able to find the root cause, deinit() often fails
+            # but it does not effect future usage of the pwm pin
+            print(
+                "warning: failed to deinitialize pwm pin {0}:{1} due to: {2}\n".format(
+                    self._channel, self._pwmpin, type(e).__name__
+                )
+            )
+        finally:
+            self._channel = None
+            self._pwmpin = None
+        # pylint: enable=broad-except

 
     def _is_deinited(self):
         if self._pwmpin is None:
 
     def _is_deinited(self):
         if self._pwmpin is None:

-            raise ValueError("Object has been deinitialize and can no longer "
-                             "be used. Create a new object.")
+            raise ValueError(
+                "Object has been deinitialize and can no longer "
+                "be used. Create a new object."
+            )

 
     # Mutable properties
 
 
     # Mutable properties
 


@@ -162,7 +168,9 @@ class PWMOut(object):
         duty_cycle = duty_cycle * PWMOut.MAX_CYCLE_LEVEL
 
         # Set duty cycle
         duty_cycle = duty_cycle * PWMOut.MAX_CYCLE_LEVEL
 
         # Set duty cycle

+        # pylint: disable=protected-access

         Pin._nova.setIOpinValue(self._pwmpin, duty_cycle)
         Pin._nova.setIOpinValue(self._pwmpin, duty_cycle)

+        # pylint: enable=protected-access

 
     duty_cycle = property(_get_duty_cycle, _set_duty_cycle)
     """Get or set the PWM's output duty cycle as a ratio from 0.0 to 1.0.
 
     duty_cycle = property(_get_duty_cycle, _set_duty_cycle)
     """Get or set the PWM's output duty cycle as a ratio from 0.0 to 1.0.


@@ -176,7 +184,7 @@ class PWMOut(object):
     """
 
     def _get_frequency(self):
     """
 
     def _get_frequency(self):

-        return int(PWMOut._nova.getIOpinPWMFreq(self._pwmpin).split('PWMFREQ ')[1])
+        return int(PWMOut._nova.getIOpinPWMFreq(self._pwmpin).split("PWMFREQ ")[1])

 
     def _set_frequency(self, frequency):
         if not isinstance(frequency, (int, float)):
 
     def _set_frequency(self, frequency):
         if not isinstance(frequency, (int, float)):


@@ -199,28 +207,32 @@ class PWMOut(object):
 
         if enabled == "1":
             return True
 
         if enabled == "1":
             return True

-        elif enabled == "0":
+        if enabled == "0":

             return False
 
             return False
 

-        raise PWMError(None, "Unknown enabled value: \"%s\"" % enabled)
+        raise PWMError(None, 'Unknown enabled value: "%s"' % enabled)

 
     def _set_enabled(self, value):
 
     def _set_enabled(self, value):

+        """Get or set the PWM's output enabled state.
+
+        Raises:
+            PWMError: if an I/O or OS error occurs.
+            TypeError: if value type is not bool.
+
+        :type: bool
+        """

         if not isinstance(value, bool):
             raise TypeError("Invalid enabled type, should be string.")
         self._enable = value
         if not self._enable:
             self._set_duty_cycle(0.0)
         if not isinstance(value, bool):
             raise TypeError("Invalid enabled type, should be string.")
         self._enable = value
         if not self._enable:
             self._set_duty_cycle(0.0)

-    """Get or set the PWM's output enabled state.
-
-    Raises:
-        PWMError: if an I/O or OS error occurs.
-        TypeError: if value type is not bool.
-
-    :type: bool
-    """

 
     # String representation
 
     def __str__(self):
 
     # String representation
 
     def __str__(self):

-        return "PWM%d, pin %s (freq=%f Hz, duty_cycle=%f%%)" % \
-            (self._pin, self._pin, self.frequency, self.duty_cycle * 100,)
+        return "PWM%d, pin %s (freq=%f Hz, duty_cycle=%f%%)" % (
+            self._pin,
+            self._pin,
+            self.frequency,
+            self.duty_cycle * 100,
+        )