Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[earlgrey_es] Cherry-pick - fix phy_host_gran_err test #19707

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

eshapira
Copy link
Contributor

@eshapira eshapira commented Sep 20, 2023

This commit already exists in the master branch, this PR will merge it to earlgrey_es branch.

Replace fatal_err.max with cfg param shared between closed source.

Signed-off-by: Jaedon Kim <[email protected]>
@eshapira eshapira requested a review from a team as a code owner September 20, 2023 06:50
@eshapira eshapira requested review from rswarbrick, sha-ron and jdonjdon and removed request for a team and rswarbrick September 20, 2023 06:50
@jdonjdon
Copy link
Contributor

jdonjdon commented Sep 20, 2023

@eshapira
This time it goes smooth heh

@eshapira
Copy link
Contributor Author

@eshapira This time it goes smooth heh

Yes, hopefully now I have the correct way to do it😀

@eshapira
Copy link
Contributor Author

@jdonjdon Can you please take a look on the failing CIs? It's weird that the CIs are failing on this commit after they passed when we committed it to the master branch.

@eshapira eshapira merged commit 25e2bb0 into lowRISC:earlgrey_es Sep 20, 2023
23 checks passed
@eshapira
Copy link
Contributor Author

Thanks!

@eshapira eshapira deleted the earlgrey_es branch October 4, 2023 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants