Flashing fails if debug interface does not have reset signal

[afeinman-snap]

The change in v0.40,0 introduced in a396354

pyOCD/pyocd/subcommands/load_cmd.py

Lines 166 to 168 in cc1ae68

	# Reset the target after programming unless --no-reset was specified.
	if not self._args.no_reset:
	session.target.reset(Target.ResetType.NSRST)

breaks flashing of devices that do not support hardware reset. It uses n_srst reset unconditionally, which fails on 3-wire setups (SWD/SWCLK/GND), where a software reset (default) should be used.

I propose to either:

• use a software reset which is a default for reset command
  or better yet
• add reset type option to the load command

[afeinman-snap]

Possible workaround:

pyocd load --no-reset .....
pyocd reset ...

[TeoMahnic]

Hi @afeinman-snap, thank you for reporting this issue, sharing a workaround, and submitting a PR.

PR #1843 will most likely need some refinement before it’s ready to be merged.

[afeinman-snap]

PR #1843 will most likely need some refinement before it’s ready to be merged.
I'm fine with either refining it if you provide some guidance, or waiting for the next release since it seems you already have plans for addressing this. Thank you for the quick response. Appreciate the work you are doing

[TeoMahnic]

Hi @afeinman-snap,
I will be closing this issue and the #1843, as it was superseded by #1861.

The newly added pre-reset and post-reset session options will be included in the next release. They will also restore the default load sub-command reset behavior from versions prior to v0.40.

Please feel free to re-open this issue or open a new one if you continue to experience flashing issues.