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

Added macro with asynchronous reset and synchronous clear. #214

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

yvantor
Copy link
Contributor

@yvantor yvantor commented Mar 5, 2024

No description provided.

@yvantor yvantor requested a review from niwis March 5, 2024 15:47
@yvantor
Copy link
Contributor Author

yvantor commented Mar 5, 2024

@niwis not putting this in draft state as I think it will not need much more than this, but I think it might make sense to wait for the activity on CVA6 to complete before evaluating a merge.

Copy link
Collaborator

@niwis niwis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yvantor thanks a lot, LGTM. I'm about to make a new release, so I think we can merge this already and follow up if needed.

@niwis niwis merged commit 50ba9de into master Mar 7, 2024
5 checks passed
@niwis niwis deleted the yt/synch-clear branch March 7, 2024 09: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.

2 participants