v1.1.0

### Releases v1.1.0

1. Fix bug. See [Fixed count >= min comparison for servo enable](#1)
2. Clean-up all compiler warnings possible.
3. Add Table of Contents
4. Add Version String
5. Fix and Optimize old examples

Author: khoih-prog

src/ESP32FastTimerInterrupt.h

@@ -171,12 +171,8 @@ class ESP32FastTimerInterrupt
       _timerCount = (uint64_t) _frequency / frequency;
       // count up
 
-//#if (TIMER_INTERRUPT_DEBUG > 0)
-      //Serial.println("ESP32TimerInterrupt: _timerNo = " + String(_timerNo) + ", _fre = " + String(_frequency)
-      //               + ", _count = " + String((uint32_t) (_timerCount >> 32) ) + " - " + String((uint32_t) (_timerCount)));
       ISR_SERVO_LOGERROR3("ESP32FastTimerInterrupt: _timerNo =", _timerNo, ", _fre =", _frequency);
       ISR_SERVO_LOGERROR3("_count =",(uint32_t) (_timerCount >> 32), " -", (uint32_t) (_timerCount));               
-//#endif
 
       // Clock to timer (prescaler) is F_CPU / 3 = 240MHz / 3 = 80MHz
       _timer = timerBegin(_timerNo, F_CPU / (_frequency * 3), true);

src/ESP32_ISR_Servo.cpp

@@ -77,10 +77,7 @@ void IRAM_ATTR ESP32_ISR_Servo::run()
   // Reset when reaching 20000us / 10us = 2000
   if (timerCount++ >= REFRESH_INTERVAL / TIMER_INTERVAL_MICRO)
   {
-//#if (ISR_SERVO_DEBUG > 1)
-    //Serial.println("Reset count");
     ISR_SERVO_LOGDEBUG("Reset count");
-//#endif
 
     timerCount = 1;
   }
@@ -103,11 +100,7 @@ int ESP32_ISR_Servo::findFirstFreeSlot()
   {
     if (servo[servoIndex].enabled == false)
     {
-//#if (ISR_SERVO_DEBUG > 1)
-      //Serial.print("Index = ");
-      //Serial.println(servoIndex);
       ISR_SERVO_LOGDEBUG1("Index =", servoIndex);
-//#endif
 
       return servoIndex;
     }
@@ -143,12 +136,8 @@ int ESP32_ISR_Servo::setupServo(uint8_t pin, int min, int max)
 
   numServos++;
 
-//#if (ISR_SERVO_DEBUG > 0)
-  //Serial.print("Index = " + String(servoIndex) + ", count = " + String(servo[servoIndex].count));
   ISR_SERVO_LOGDEBUG3("Index =", servoIndex, ", count =", servo[servoIndex].count);
-  //Serial.println(", min = " + String(servo[servoIndex].min) + ", max = " + String(servo[servoIndex].max));
   ISR_SERVO_LOGDEBUG3("min =", servo[servoIndex].min, ", max =", servo[servoIndex].max);
-//#endif
 
   return servoIndex;
 }
@@ -172,11 +161,8 @@ bool ESP32_ISR_Servo::setPosition(unsigned servoIndex, int position)
     // It is mandatory to disable task switches during modifying shared vars
     portEXIT_CRITICAL(&timerMux);
 
-//#if (ISR_SERVO_DEBUG > 0)
-    //Serial.println("Idx = " + String(servoIndex) + ", cnt = " + String(servo[servoIndex].count) + ", pos = " + String(servo[servoIndex].position));
     ISR_SERVO_LOGERROR1("Idx =", servoIndex);
     ISR_SERVO_LOGERROR3("cnt =", servo[servoIndex].count, ", pos =",servo[servoIndex].position);
-//#endif
 
     return true;
   }
@@ -194,11 +180,8 @@ int ESP32_ISR_Servo::getPosition(unsigned servoIndex)
   // Updates interval of existing specified servo
   if ( servo[servoIndex].enabled && (servo[servoIndex].pin <= ESP32_MAX_PIN) )
   {
-//#if (ISR_SERVO_DEBUG > 0)
-    //Serial.println("Idx = " + String(servoIndex) + ", cnt = " + String(servo[servoIndex].count) + ", pos = " + String(servo[servoIndex].position));
     ISR_SERVO_LOGERROR1("Idx =", servoIndex);
     ISR_SERVO_LOGERROR3("cnt =", servo[servoIndex].count, ", pos =",servo[servoIndex].position);
-//#endif
 
     return (servo[servoIndex].position);
   }
@@ -236,11 +219,8 @@ bool ESP32_ISR_Servo::setPulseWidth(unsigned servoIndex, unsigned int pulseWidth
     // It is mandatory to disable task switches during modifying shared vars
     portEXIT_CRITICAL(&timerMux);
 
-//#if (ISR_SERVO_DEBUG > 0)
-    //Serial.println("Idx = " + String(servoIndex) + ", cnt = " + String(servo[servoIndex].count) + ", pos = " + String(servo[servoIndex].position));
     ISR_SERVO_LOGERROR1("Idx =", servoIndex);
     ISR_SERVO_LOGERROR3("cnt =", servo[servoIndex].count, ", pos =",servo[servoIndex].position);
-//#endif
 
     return true;
   }
@@ -258,11 +238,8 @@ unsigned int ESP32_ISR_Servo::getPulseWidth(unsigned servoIndex)
   // Updates interval of existing specified servo
   if ( servo[servoIndex].enabled && (servo[servoIndex].pin <= ESP32_MAX_PIN) )
   {
-//#if (ISR_SERVO_DEBUG > 0)
-    //Serial.println("Idx = " + String(servoIndex) + ", cnt = " + String(servo[servoIndex].count) + ", pos = " + String(servo[servoIndex].position));
     ISR_SERVO_LOGERROR1("Idx =", servoIndex);
     ISR_SERVO_LOGERROR3("cnt =", servo[servoIndex].count, ", pos =",servo[servoIndex].position);
-//#endif
 
     return (servo[servoIndex].count * TIMER_INTERVAL_MICRO );
   }

src/ESP32_ISR_Servo.h

@@ -161,21 +161,12 @@ class ESP32_ISR_Servo
 
       // Interval in microsecs
       if ( ESP32_ITimer && ESP32_ITimer->attachInterruptInterval(TIMER_INTERVAL_MICRO, (timer_callback) ESP32_ISR_Servo_Handler ) )
-        //if ( ESP32_ITimer.attachInterruptInterval(TIMER_INTERVAL_MICRO, (timer_callback) ESP32_ISR_Servo_Handler ) )
       {
-//#if (ISR_SERVO_DEBUG > 0)
-        //Serial.println("Starting  ITimer OK");
         ISR_SERVO_LOGERROR("Starting  ITimer OK");
-//#endif
       }
       else
       {
-//#if (ISR_SERVO_DEBUG > 0)
-        // Can't set ESP32_ITimer correctly. Select another freq. or interval
-        //Serial.println("Fail setup ESP32_ITimer");
-        ISR_SERVO_LOGERROR("Fail setup ESP32_ITimer");
-//#endif
-      }
+        ISR_SERVO_LOGERROR("Fail setup ESP32_ITimer");      }
 
       for (int servoIndex = 0; servoIndex < MAX_SERVOS; servoIndex++)
       {
@@ -203,8 +194,8 @@ class ESP32_ISR_Servo
       unsigned long count;                // In microsecs
       int           position;             // In degrees
       bool          enabled;              // true if enabled
-      int16_t       min;
-      int16_t       max;
+      uint16_t      min;
+      uint16_t      max;
     } servo_t;
 
     volatile servo_t servo[MAX_SERVOS];