Skip to content

Commit f7786bb

Browse files
henrikbrixandersennashif
authored andcommitted
tests: drivers: can: api: relax CAN sample point accuracy requirements
Relax the CAN sample point location accuracy requirements in the CAN API test suites by taking CONFIG_CAN_SAMPLE_POINT_MARGIN into account. Fixes: #100615 Signed-off-by: Henrik Brix Andersen <[email protected]>
1 parent 7830a76 commit f7786bb

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

tests/drivers/can/api/src/canfd.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -528,7 +528,8 @@ ZTEST_USER(canfd, test_set_timing_data_while_started)
528528
int err;
529529

530530
err = can_calc_timing_data(can_dev, &timing, TEST_BITRATE_3, TEST_SAMPLE_POINT);
531-
zassert_ok(err, "failed to calculate data timing (err %d)", err);
531+
zassert_true(err >= 0, "failed to calculate data timing (err %d)", err);
532+
zassert_true(err <= CONFIG_CAN_SAMPLE_POINT_MARGIN, "sample point error %d too large", err);
532533

533534
err = can_set_timing_data(can_dev, &timing);
534535
zassert_not_equal(err, 0, "changed data timing while started");

tests/drivers/can/api/src/classic.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1393,7 +1393,8 @@ ZTEST_USER(can_classic, test_set_timing_while_started)
13931393
int err;
13941394

13951395
err = can_calc_timing(can_dev, &timing, TEST_BITRATE_1, TEST_SAMPLE_POINT);
1396-
zassert_ok(err, "failed to calculate timing (err %d)", err);
1396+
zassert_true(err >= 0, "failed to calculate timing (err %d)", err);
1397+
zassert_true(err <= CONFIG_CAN_SAMPLE_POINT_MARGIN, "sample point error %d too large", err);
13971398

13981399
err = can_set_timing(can_dev, &timing);
13991400
zassert_not_equal(err, 0, "changed timing while started");

0 commit comments

Comments
 (0)