Skip to content

Commit 16fe8c3

Browse files
nordic-baminordic-piks
authored andcommitted
tests: drivers: Relax requirements for SPIM & TWIM latency tests
The assumed limits were too strict. Signed-off-by: Bartosz Miller <[email protected]>
1 parent 80fe797 commit 16fe8c3

File tree

2 files changed

+8
-8
lines changed
  • tests/drivers

2 files changed

+8
-8
lines changed

tests/drivers/i2c/i2c_latency/src/main.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,11 +131,11 @@ static void assess_measurement_result(uint64_t timer_value_us,
131131
uint64_t maximal_allowed_transmission_time_us;
132132

133133
if (buffer_size == 1) {
134-
/* 400% */
135-
maximal_allowed_transmission_time_us = (1 + 4) * theoretical_transmission_time_us;
134+
/* 500% */
135+
maximal_allowed_transmission_time_us = (1 + 5) * theoretical_transmission_time_us;
136136
} else {
137-
/* 100% */
138-
maximal_allowed_transmission_time_us = (1 + 1) * theoretical_transmission_time_us;
137+
/* 200% */
138+
maximal_allowed_transmission_time_us = (1 + 2) * theoretical_transmission_time_us;
139139
}
140140

141141
zassert_true(timer_value_us < maximal_allowed_transmission_time_us,

tests/drivers/spi/spi_latency/src/main.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,14 +77,14 @@ static void assess_measurement_result(uint64_t timer_value_us,
7777
uint64_t maximal_allowed_transmission_time_us;
7878

7979
if (buffer_size == 1) {
80-
/* 3000%
80+
/* 4000%
8181
* Issue with latency for 1B buffer is reported
8282
* limit will be adjusted after fix
8383
*/
84-
maximal_allowed_transmission_time_us = (1 + 30) * theoretical_transmission_time_us;
84+
maximal_allowed_transmission_time_us = (1 + 40) * theoretical_transmission_time_us;
8585
} else {
86-
/* 100% */
87-
maximal_allowed_transmission_time_us = (1 + 1) * theoretical_transmission_time_us;
86+
/* 200% */
87+
maximal_allowed_transmission_time_us = (1 + 2) * theoretical_transmission_time_us;
8888
}
8989
zassert_true(timer_value_us < maximal_allowed_transmission_time_us,
9090
"Measured transmission call latency is over the specified limit %llu > %llu",

0 commit comments

Comments
 (0)