Enable Blackhole AICLK test #562
Open
+20
−21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pjanevskiTT
force-pushed
the
pjanevski/enable_bh_aiclk_test
branch
from
bbec183 to
950bb7a
Compare
pjanevskiTT
force-pushed
the
pjanevski/enable_bh_aiclk_test
branch
from
950bb7a to
0715487
Compare
pjanevskiTT
commented
Mar 3, 2025
| // TODO: enable this test once 80.15 fw is flashed to CI machines | ||
| // TEST(BlackholeArcMessages, BlackholeArcMessageHigherAIClock) { | ||
| // const uint32_t ms_sleep = 2000; | ||
| TEST(BlackholeArcMessages, BlackholeArcMessageHigherAIClock) { |
There was a problem hiding this comment.
Wait until #569 is merged. Use get_clock API to check the clock
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
/
Description
#548 introduced few tests to test the ARC communication, one of those being go busy/idle messages and then readouts of the Blackhole clk, but it was left disabled because 80.15 fw is not yet flashed on CI machines. This PR reenables the test and it will be merged once the new fw is released to machines.
List of the changes
Testing
CI
API Changes
/