Merge pull request #1005 from makermelissa/libgpiod-fix

[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 2f4e4b563672578ac050f9159d69e56e5cd8ddd3..5c9083d9843c18588a5f7f1191ecdf28c7258a4e 100644 (file)
--- a/src/adafruit_blinka/microcontroller/nova/pwmout.py
+++ b/src/adafruit_blinka/microcontroller/nova/pwmout.py
@@ -1,19 +1,29 @@
-import os
-import digitalio
+# SPDX-FileCopyrightText: 2021 Melissa LeBlanc-Williams for Adafruit Industries
+#
+# SPDX-License-Identifier: MIT
+"""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,14 +48,22 @@ class PWMOut(object):
 
         """
         if PWMOut._nova is None:
 
         """
         if PWMOut._nova is None:

+            # pylint: disable=import-outside-toplevel

             from adafruit_blinka.microcontroller.nova import Connection
             from adafruit_blinka.microcontroller.nova import Connection

+
+            # pylint: enable=import-outside-toplevel
+

             PWMOut._nova = Connection.getInstance()
 
             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


@@ -64,6 +82,9 @@ 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
         PWMOut._nova.setIOpinMode(self._pwmpin, Pin.PWM)
 
         # set frequency


@@ -77,25 +98,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
-
-      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
 


@@ -142,7 +171,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.


@@ -156,7 +187,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)):


@@ -179,28 +210,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,
+        )