[earlgrey_es,dv] fix closed source regression faillure #19474
Merged
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
jdonjdon
requested review from
matutem,
jadephilipoom,
a-will,
moidx and
msfschaffner
and removed request for
a team,
matutem and
jadephilipoom
a-will
approved these changes
Aug 17, 2023
Add wait fetch enable before any generic sequence begin for stub cpu test. Signed-off-by: Jaedon Kim <[email protected]>
pwrmgr_normal_sleep_all_reset_reqs_test.c shows spurious ping timeout issues. Update timeout value to 256 ( recommended by design engineer). Signed-off-by: Jaedon Kim <[email protected]>
dai_wr has multiple csr access including backdoor access. In parallel_lc_esc vseq, `wait_csr_no_outstanding` is not good enough because it can terminate test thread while dai write operation is in progress. Update `rand_wait_csr_no_outstanding` s.t wait for no outstanding dai write operation. Signed-off-by: Jaedon Kim <[email protected]>
rw evict test requires the same page write (illegal). In flash, this causes all data approaches to zero and create false ecc error. To avoid converging zero, write patterned data instead of random. Signed-off-by: Jaedon Kim <[email protected]>
If fatal alert is set during multiple program operation, flash can be disabled by fatal alert in the middle of multiple program operation. To avoid test lock up, add parallel thread to terminate program operation upon fatal alert is observed. Signed-off-by: Jaedon Kim <[email protected]>
Signed-off-by: Jaedon Kim <[email protected]>
Update scoreboard `erase_bank` task align with opensource flash model bank_erase behavior Signed-off-by: Jaedon Kim <[email protected]>
`flash_ctrl_mp_regions` test requires 32 consequtive recover alerts. When async frequency of alert agent is too slow, receiver alert agent misses alerts and causes test fail. Adjust frequency range of receiver alert agent just for this test. Signed-off-by: Jaedon Kim <[email protected]>
Same issue as lowRISC#19268 has been observed. Hardening error injection. Signed-off-by: Jaedon Kim <[email protected]>
sha-ron
approved these changes
Aug 18, 2023
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.
Following PRs' from mater branch
#19415
#19430
#19431
#19450