Merge pull request #1 from raphweb/master

khoih-prog · web-flow · commit 779e30b83026 · 2021-01-02T19:57:55.000-05:00
Fixed count &gt;= min comparison for servo enable.
diff --git a/src/ESP32_ISR_Servo.cpp b/src/ESP32_ISR_Servo.cpp
@@ -322,7 +322,7 @@ bool ESP32_ISR_Servo::enable(unsigned servoIndex)
     return false;
   }
 
-  if ( servo[servoIndex].count >= servo[servoIndex].min )
+  if ( servo[servoIndex].count >= servo[servoIndex].min / TIMER_INTERVAL_MICRO )
     servo[servoIndex].enabled = true;
 
   // ESP32 is a multi core / multi processing chip.
@@ -355,7 +355,7 @@ void ESP32_ISR_Servo::enableAll()
 
   for (int servoIndex = 0; servoIndex < MAX_SERVOS; servoIndex++)
   {
-    if ( (servo[servoIndex].count >= servo[servoIndex].min ) && !servo[servoIndex].enabled && (servo[servoIndex].pin <= ESP32_MAX_PIN) )
+    if ( (servo[servoIndex].count >= servo[servoIndex].min / TIMER_INTERVAL_MICRO ) && !servo[servoIndex].enabled && (servo[servoIndex].pin <= ESP32_MAX_PIN) )
     {
       servo[servoIndex].enabled = true;
     }

Original file line number	Diff line number	Diff line change
`@@ -322,7 +322,7 @@ bool ESP32_ISR_Servo::enable(unsigned servoIndex)`
`322`	`322`	`return false;`
`323`	`323`	`}`
`324`	`324`
`325`		`- if ( servo[servoIndex].count >= servo[servoIndex].min )`
	`325`	`+ if ( servo[servoIndex].count >= servo[servoIndex].min / TIMER_INTERVAL_MICRO )`
`326`	`326`	`servo[servoIndex].enabled = true;`
`327`	`327`
`328`	`328`	`// ESP32 is a multi core / multi processing chip.`
`@@ -355,7 +355,7 @@ void ESP32_ISR_Servo::enableAll()`
`355`	`355`
`356`	`356`	`for (int servoIndex = 0; servoIndex < MAX_SERVOS; servoIndex++)`
`357`	`357`	`{`
`358`		`- if ( (servo[servoIndex].count >= servo[servoIndex].min ) && !servo[servoIndex].enabled && (servo[servoIndex].pin <= ESP32_MAX_PIN) )`
	`358`	`+ if ( (servo[servoIndex].count >= servo[servoIndex].min / TIMER_INTERVAL_MICRO ) && !servo[servoIndex].enabled && (servo[servoIndex].pin <= ESP32_MAX_PIN) )`
`359`	`359`	`{`
`360`	`360`	`servo[servoIndex].enabled = true;`
`361`	`361`	`}`