final rework for issue #250. all tests passed

gin66 · May 1, 2024 · 3990aa4 · 3990aa4
1 parent ff281d0
commit 3990aa4
Show file tree

Hide file tree

Showing 9 changed files with 26 additions and 22 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@ TODO:
 - rename RampConstAcceleration to e.g. RampControl
 - merge the two esp32 rmt drivers as soon as esp32c3 works
 
+pre-0.30.13:1
+- avr: rework Stepper-ISR routine. It should now be robust against interrupt blockage in regard to steps lost. If interrupt blockage is too long, then 4ms paus could occur between two steps.
+- avr: Interrupt blockage of 30us tested and passed
+
 0.30.12:
 - esp32: fix deprecation warning for `rmt_memory_rw_rst()`
 - esp32: add build test for platform espressif v6.6.0 with arduino core (#251)

diff --git a/extras/tests/simavr_based/test_issue208/expect.txt b/extras/tests/simavr_based/test_issue208/expect.txt
@@ -2,17 +2,17 @@
     DirB:        1*L->H,        0*H->L
  EnableA:        0*L->H,        0*H->L
  EnableB:        0*L->H,        0*H->L
-   StepA:   235907*L->H,   235907*H->L, Max High=10us Total High=927140us
+   StepA:   235908*L->H,   235908*H->L, Max High=11us Total High=927312us
    StepB:        2*L->H,        2*H->L, Max High=12us Total High=17us
-Position[A]=37463
+Position[A]=37462
 
 Position[B]=2
 
-Time in FillISR  max=768 us, total=1213765 us
+Time in FillISR  max=767 us, total=1211908 us
 
-Time in StepA  max=10 us, total=927140 us
+Time in StepA  max=11 us, total=927312 us
 
 Time in StepB  max=12 us, total=17 us
 
-Time in StepISR  max=6 us, total=912200 us
+Time in StepISR  max=6 us, total=897842 us
 
diff --git a/...r_based/off_test_issue250_30us/.gitignore → ...imavr_based/test_issue250_30us/.gitignore b/...r_based/off_test_issue250_30us/.gitignore → ...imavr_based/test_issue250_30us/.gitignore
diff --git a/...avr_based/off_test_issue250_30us/Makefile → .../simavr_based/test_issue250_30us/Makefile b/...avr_based/off_test_issue250_30us/Makefile → .../simavr_based/test_issue250_30us/Makefile
diff --git a/...r_based/off_test_issue250_30us/expect.txt → ...imavr_based/test_issue250_30us/expect.txt b/...r_based/off_test_issue250_30us/expect.txt → ...imavr_based/test_issue250_30us/expect.txt
diff --git a/...sed/off_test_issue250_30us/platformio.ini → ...r_based/test_issue250_30us/platformio.ini b/...sed/off_test_issue250_30us/platformio.ini → ...r_based/test_issue250_30us/platformio.ini
diff --git a/extras/tests/simavr_based/test_sd_04_timing_2560/expect.txt b/extras/tests/simavr_based/test_sd_04_timing_2560/expect.txt
@@ -4,9 +4,9 @@
  EnableA:        1*L->H,        1*H->L
  EnableB:        2*L->H,        1*H->L
  EnableC:        2*L->H,        1*H->L
-   StepA:    64000*L->H,    64000*H->L, Max High=26us Total High=479601us
-   StepB:    64000*L->H,    64000*H->L, Max High=30us Total High=368752us
-   StepC:    64000*L->H,    64000*H->L, Max High=36us Total High=379262us
+   StepA:    64000*L->H,    64000*H->L, Max High=29us Total High=491914us
+   StepB:    64000*L->H,    64000*H->L, Max High=31us Total High=363617us
+   StepC:    64000*L->H,    64000*H->L, Max High=41us Total High=374085us
 Position[A]=64000
 
 Position[B]=64000
@@ -19,11 +19,11 @@ Time in EnableB  max=246080 us, total=246080 us
 
 Time in EnableC  max=254234 us, total=254234 us
 
-Time in FillISR  max=2805 us, total=2111191 us
+Time in FillISR  max=2697 us, total=2102378 us
 
-Time in StepA  max=26 us, total=479601 us
+Time in StepA  max=29 us, total=491914 us
 
-Time in StepB  max=30 us, total=368752 us
+Time in StepB  max=31 us, total=363617 us
 
-Time in StepC  max=36 us, total=379262 us
+Time in StepC  max=41 us, total=374085 us
 
diff --git a/extras/tests/simavr_based/test_sd_04_timing_328p/expect.txt b/extras/tests/simavr_based/test_sd_04_timing_328p/expect.txt
@@ -2,8 +2,8 @@
     DirB:        1*L->H,        0*H->L
  EnableA:        1*L->H,        1*H->L
  EnableB:        2*L->H,        1*H->L
-   StepA:     1000*L->H,     1000*H->L, Max High=14us Total High=4334us
-   StepB:     1000*L->H,     1000*H->L, Max High=14us Total High=4915us
+   StepA:     1000*L->H,     1000*H->L, Max High=14us Total High=4311us
+   StepB:     1000*L->H,     1000*H->L, Max High=15us Total High=4977us
 Position[A]=1000
 
 Position[B]=1000
@@ -12,11 +12,11 @@ Time in EnableA  max=225398 us, total=225398 us
 
 Time in EnableB  max=238115 us, total=238115 us
 
-Time in FillISR  max=2640 us, total=47588 us
+Time in FillISR  max=2639 us, total=47567 us
 
-Time in StepA  max=14 us, total=4334 us
+Time in StepA  max=14 us, total=4311 us
 
-Time in StepB  max=14 us, total=4915 us
+Time in StepB  max=15 us, total=4977 us
 
-Time in StepISR  max=6 us, total=8679 us
+Time in StepISR  max=6 us, total=8566 us
 
diff --git a/extras/tests/simavr_based/test_sd_04_timing_328p_37k/expect.txt b/extras/tests/simavr_based/test_sd_04_timing_328p_37k/expect.txt
@@ -2,15 +2,15 @@
     DirB:        1*L->H,        0*H->L
  EnableA:        1*L->H,        1*H->L
  EnableB:        1*L->H,        0*H->L
-   StepA:     1000*L->H,     1000*H->L, Max High=11us Total High=3969us
+   StepA:     1000*L->H,     1000*H->L, Max High=10us Total High=3965us
    StepB:        0*L->H,        0*H->L, Max High=0us Total High=0us
 Position[A]=1000
 
 Time in EnableA  max=225399 us, total=225399 us
 
-Time in FillISR  max=2013 us, total=27494 us
+Time in FillISR  max=2013 us, total=27460 us
 
-Time in StepA  max=11 us, total=3969 us
+Time in StepA  max=10 us, total=3965 us
 
-Time in StepISR  max=5 us, total=4031 us
+Time in StepISR  max=5 us, total=3975 us