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

Debugger: Slightly relax the RISC-V restriction when handling of restart request. #1569

Merged
merged 3 commits into from
Mar 19, 2023

Conversation

noppej
Copy link
Contributor

@noppej noppej commented Mar 15, 2023

With this PR, we now allow restart, but will not re-flash, so that we avoid the issue #1519, while still allowing users to restart their target during debugging.

cc: @MabezDev please confirm that this is what you requested :)

@noppej noppej requested a review from Tiwalun March 15, 2023 16:39
Copy link
Member

@MabezDev MabezDev left a comment

Choose a reason for hiding this comment

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

Perfect, thank you!

@noppej noppej enabled auto-merge March 15, 2023 21:28
@noppej noppej added this pull request to the merge queue Mar 19, 2023
@noppej noppej merged commit 2c2f023 into master Mar 19, 2023
@noppej noppej deleted the riscv_restart branch March 19, 2023 15:24
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.

3 participants