stmhal: Fix edge case for timer PWM of 100%.

dpgeorge · dpgeorge · commit f042d7a4d719 · 2014-09-29T14:15:01.000+01:00
Also improve precision of calculating PWM percent in integer mode.
Also update teensy with edge case fix.
diff --git a/stmhal/timer.c b/stmhal/timer.c
@@ -326,19 +326,18 @@ STATIC uint32_t compute_pwm_value_from_percent(uint32_t period, mp_obj_t percent
 STATIC mp_obj_t compute_percent_from_pwm_value(uint32_t period, uint32_t cmp) {
     #if MICROPY_PY_BUILTINS_FLOAT
     float percent;
-    if (cmp > period) {
+    if (cmp >= period) {
         percent = 100.0;
     } else {
         percent = (float)cmp * 100.0 / ((float)period);
     }
     return mp_obj_new_float(percent);
     #else
     mp_int_t percent;
-    if (cmp > period) {
+    if (cmp >= period) {
         percent = 100;
-    } else if (period > MAX_PERIOD_DIV_100) {
-        // We divide the top and bottom by 128, and then do the math.
-        percent = (cmp / 128) * 100 / (period / 128);
+    } else if (cmp > MAX_PERIOD_DIV_100) {
+        percent = cmp / (period / 100);
     } else {
         percent = cmp * 100 / period;
     }
diff --git a/teensy/timer.c b/teensy/timer.c
@@ -175,15 +175,15 @@ STATIC uint32_t compute_pwm_value_from_percent(uint32_t period, mp_obj_t percent
 STATIC mp_obj_t compute_percent_from_pwm_value(uint32_t period, uint32_t cmp) {
     #if MICROPY_PY_BUILTINS_FLOAT
     float percent = (float)cmp * 100.0 / (float)period;
-    if (cmp > period) {
+    if (cmp >= period) {
         percent = 100.0;
     } else {
         percent = (float)cmp * 100.0 / (float)period;
     }
     return mp_obj_new_float(percent);
     #else
     mp_int_t percent;
-    if (cmp > period) {
+    if (cmp >= period) {
         percent = 100;
     } else {
         percent = cmp * 100 / period;
