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

Add software auto reset to Atmel and stm32 #794

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Kevincoooool
Copy link

The target chip can auto reset after download with this pr

@mathias-arm mathias-arm requested a review from flit April 8, 2021 17:15
@mathias-arm mathias-arm requested a review from 0xc0170 April 23, 2021 20:19
@mathias-arm
Copy link
Collaborator

This looks like a layering violation. You should look at why target_set_state(RESET_PROGRAM) (or possibly swd_set_target_state_hw(RESET_PROGRAM)) does not work in your case. Not sure if there is a generic (or HIC-specific) way to detect that hardware reset does not work and fall back to software reset.

@cederom
Copy link
Contributor

cederom commented Apr 24, 2021

There was an idea to unify reset once #371

@Kevincoooool
Copy link
Author

Hello, because my hardware does not set aside RST pin for DAPLINK, so I can only use SWD to perform a software reset. This PR has been tested to work normally and there is no other impact

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants