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

Dragon JTAG fix #979

Merged
merged 1 commit into from
Jun 26, 2022
Merged

Dragon JTAG fix #979

merged 1 commit into from
Jun 26, 2022

Conversation

MCUdude
Copy link
Collaborator

@MCUdude MCUdude commented May 29, 2022

This simple fix resolves the issue where a connected AVR JTAG target doesn't start its program after programming has occurred. Fix has been tested on real hardware.

Closes #366 and #377

@dl8dtl
Copy link
Contributor

dl8dtl commented May 30, 2022

What confuses me a bit about that PR is the entire addition of an SPI programmer to the code.

If I see it right, the actual fix would be a one-liner, doesn't it?

@MCUdude
Copy link
Collaborator Author

MCUdude commented May 30, 2022

Whoa, sorry! It is supposed to be a oneliner. I've accidentally added some linuxspi as well. I'll fix it right away!

@MCUdude
Copy link
Collaborator Author

MCUdude commented Jun 15, 2022

I think this is safe to merge

@dl8dtl
Copy link
Contributor

dl8dtl commented Jun 26, 2022

I can confirm it fixes the Dragon issue.
On the JTAGICEmkII, the firmware eventually starts even without the patch, but with the patch applied, it's a lot faster.

@dl8dtl dl8dtl merged commit 55dddc5 into avrdudes:main Jun 26, 2022
freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this pull request Nov 26, 2022
…into 7.0

At some point after 5.11 avrdude started leaving the target stopped
upon completion of a JTAG transaction. I filed a github issue about
this issue:

    avrdudes/avrdude#366

This is super annoying because, for example, the target does not
boot after JTAG programming. Others were seeing this and shortly
after 7.0 was released, MCUdude created a PR which has since been
merged:

    avrdudes/avrdude#979

Apply this patch.

PR:		266186
Approved by:	joerg (maintainer timeout, 12 weeks)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug #43898] atmega644p remains stopped after JTAG transaction
3 participants