Opened 9 years ago

Closed 9 years ago

#47816 closed defect (invalid)

avrdude: fix ftdi_syncbb to bring Programming Enable time delay into spec

Reported by: beck.dac@… Owned by: g5pw (Aljaž Srebrnič)
Priority: Normal Milestone:
Component: ports Version: 2.3.3
Keywords: haspatch Cc: beck.dac@…
Port: avrdude

Description

When using the FTDI bit banging programmer, chips will reply to signature requests and even program with -D (disable chip erase). However, when -D is left out and the default action -e (chip erase) occurs, the chip will not reply to a subsequent program enable in time.

This happens because the datasheet specified delay (20ms) has not been observed. The current version of avrdude 6.1 only waits for 5 ms. For chips with slow oscillators this is a problem.

The attached patch (patch-245r.diff) to be applied to the source corrects this problem as described in this post from avrdude-dev: https://lists.nongnu.org/archive/html/avrdude-dev/2013-09/msg00160.html

I have also included a Portfile patch (Portfile-avrdude.diff) that applies the attached patch.

Attachments (2)

Portfile-avrdude.diff (395 bytes) - added by beck.dac@… 9 years ago.
Portfile patch that applies the source code patch
patch-ft245r.diff (1.0 KB) - added by beck.dac@… 9 years ago.
Source code patch that corrects the out of spec delay

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by beck.dac@…

Attachment: Portfile-avrdude.diff added

Portfile patch that applies the source code patch

Changed 9 years ago by beck.dac@…

Attachment: patch-ft245r.diff added

Source code patch that corrects the out of spec delay

comment:1 Changed 9 years ago by beck.dac@…

Cc: beck.dac@… added

Cc Me!

comment:2 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: beck.dac@… removed
Keywords: haspatch added; avrdude ftdi removed
Owner: changed from macports-tickets@… to g5pw@…
Summary: patch avrdude for ftdi_syncbb fix to bring Programming Enable time delay into specavrdude: fix ftdi_syncbb to bring Programming Enable time delay into spec

If this changes runtime behavior, the revision must be increased.

The mailing list post you reference is from 2013. Why haven't the developers released a new version fixing this problem?

comment:3 in reply to:  2 Changed 9 years ago by beck.dac@…

Thanks for looking at this...

Replying to ryandesign@…:

If this changes runtime behavior, the revision must be increased.

I've not submitted to macports before so I'm not sure I know what you mean. I understand the words, but I don't know if you mean the macports revision or avrdude...

The mailing list post you reference is from 2013. Why haven't the developers released a new version fixing this problem?

It was briefly fixed in an earlier version but it appears there was a reversion at 6.1. My guess is that since there have been no new releases since 6.1 this patch has not been applied.

Last edited 9 years ago by beck.dac@… (previous) (diff)

comment:4 Changed 9 years ago by beck.dac@…

Cc: beck.dac@… added
Last edited 9 years ago by beck.dac@… (previous) (diff)

comment:5 Changed 9 years ago by g5pw (Aljaž Srebrnič)

Resolution: invalid
Status: newclosed

I'm sorry, I'm not very familiar with that programmer, so I wouldn't know if the code fixes and does not have side effects on other hardware (as per this response). You should submit this patch upstream.

Note: See TracTickets for help on using tickets.