From 3990aa44e244ea18f566ddad58c6f9a50ea2427b Mon Sep 17 00:00:00 2001 From: gin66 <5549373+gin66@users.noreply.github.com> Date: Wed, 1 May 2024 01:11:46 +0000 Subject: [PATCH] final rework for issue #250. all tests passed --- CHANGELOG.md | 4 ++++ extras/tests/simavr_based/test_issue208/expect.txt | 10 +++++----- .../.gitignore | 0 .../Makefile | 0 .../expect.txt | 0 .../platformio.ini | 0 .../simavr_based/test_sd_04_timing_2560/expect.txt | 14 +++++++------- .../simavr_based/test_sd_04_timing_328p/expect.txt | 12 ++++++------ .../test_sd_04_timing_328p_37k/expect.txt | 8 ++++---- 9 files changed, 26 insertions(+), 22 deletions(-) rename extras/tests/simavr_based/{off_test_issue250_30us => test_issue250_30us}/.gitignore (100%) rename extras/tests/simavr_based/{off_test_issue250_30us => test_issue250_30us}/Makefile (100%) rename extras/tests/simavr_based/{off_test_issue250_30us => test_issue250_30us}/expect.txt (100%) rename extras/tests/simavr_based/{off_test_issue250_30us => test_issue250_30us}/platformio.ini (100%) diff --git a/CHANGELOG.md b/CHANGELOG.md index d5af3994..d9408222 100644 --- 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 index eac920c6..6cd81dbf 100644 --- 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/extras/tests/simavr_based/off_test_issue250_30us/.gitignore b/extras/tests/simavr_based/test_issue250_30us/.gitignore similarity index 100% rename from extras/tests/simavr_based/off_test_issue250_30us/.gitignore rename to extras/tests/simavr_based/test_issue250_30us/.gitignore diff --git a/extras/tests/simavr_based/off_test_issue250_30us/Makefile b/extras/tests/simavr_based/test_issue250_30us/Makefile similarity index 100% rename from extras/tests/simavr_based/off_test_issue250_30us/Makefile rename to extras/tests/simavr_based/test_issue250_30us/Makefile diff --git a/extras/tests/simavr_based/off_test_issue250_30us/expect.txt b/extras/tests/simavr_based/test_issue250_30us/expect.txt similarity index 100% rename from extras/tests/simavr_based/off_test_issue250_30us/expect.txt rename to extras/tests/simavr_based/test_issue250_30us/expect.txt diff --git a/extras/tests/simavr_based/off_test_issue250_30us/platformio.ini b/extras/tests/simavr_based/test_issue250_30us/platformio.ini similarity index 100% rename from extras/tests/simavr_based/off_test_issue250_30us/platformio.ini rename to extras/tests/simavr_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 index 559ecba9..0e01c2fa 100644 --- 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 index 573eddea..4bdba4a8 100644 --- 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 index e6e67ac0..d9d2b02b 100644 --- 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