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

7e94ed4442b breaks flashing GT2560 boards #1503

Closed
diresi opened this issue Sep 25, 2023 · 4 comments · Fixed by #1505
Closed

7e94ed4442b breaks flashing GT2560 boards #1503

diresi opened this issue Sep 25, 2023 · 4 comments · Fixed by #1505
Labels
bug Something isn't working

Comments

@diresi
Copy link

diresi commented Sep 25, 2023

Hi,

I have a 3d printer running a GT2560 [1] board, which is essentially an ATmega2560 with a CH340C as USB-to-Serial converter.
I found that flashing works with avrdude 6.3 but not with 7.2.

It took me a while to trace that to DTR handling that was changed in #1309.
Flashing works again when reverting commit 7e94ed4 in wiring.c.

[1]
https://www.geeetech.com/wiki/images/d/d3/Hardware_GT2560_RevA%2B.pdf

@stefanrueger
Copy link
Collaborator

stefanrueger commented Sep 25, 2023

@diresi Yes, C9 is a bit high with 1 µF, so the 50 ms dis/charge time in the usleep() calls might be a tad bit low for this board? Could you play with both values in lines 175 and 183 to see if you can get the board to reset and synchronise?

@mcuee
Copy link
Collaborator

mcuee commented Sep 26, 2023

Another way is to change C9 to something like 100nF, and then see if avrdude 7.2 version works or not.

diresi added a commit to diresi/avrdude that referenced this issue Sep 26, 2023
Geeetech GT2560 [1] use 1uF capacitors in their DTR/RESET line and 50ms
seem to be a tad to short. 80ms works well (here).

fixes avrdudes#1503

[1]
https://www.geeetech.com/wiki/images/d/d3/Hardware_GT2560_RevA%2B.pdf
diresi added a commit to diresi/avrdude that referenced this issue Sep 26, 2023
Geeetech GT2560 boards [1] use 1uF capacitors in their DTR/RESET line
and 50ms seem to be a tad too short. 80ms works well (here).

fixes avrdudes#1503

[1]
https://www.geeetech.com/wiki/images/d/d3/Hardware_GT2560_RevA%2B.pdf
@diresi
Copy link
Author

diresi commented Sep 26, 2023

That was solid advice :)
It works with 80ms discharge time, I created a pull request for that w/o reading any contribution guidelines ... please feel free to reject.

I was playing with the idea of replacing C9 already (since I broke the original SMD one when attaching probes) but then again, maybe someone else is also using that board, right?

@mcuee mcuee added the bug Something isn't working label Sep 26, 2023
@stefanrueger stefanrueger added invalid This doesn't seem right and removed bug Something isn't working labels Sep 26, 2023
@stefanrueger
Copy link
Collaborator

OK, further tests have demonstrated that the board design and too high out-of-specs Vcc led to the board being put into HVPP mode. Tinkering with usleep() has ostensibly solved the symptoms but the usleep() times were not wrong per se.

@stefanrueger stefanrueger closed this as not planned Won't fix, can't repro, duplicate, stale Sep 26, 2023
@stefanrueger stefanrueger added bug Something isn't working and removed invalid This doesn't seem right labels Sep 27, 2023
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
3 participants