patchAVR Downloader/UploaDEr - Patches: patch #4539, Ability to control the bit clock...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #4539: Ability to control the bit clock (usleep) delay for ppi interface

Submitter:  Neil Davey <npdavey>
Submitted:  Sat 22 Oct 2005 05:02:35 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  joerg_wunsch Open/Closed:  Closed

Jump to the original submission

Thu 17 Aug 2006 03:07:03 PM UTC, comment #6: 

I just committed a slightly modified version of your patch to
CVS.

Thank you very much for the submission!

Joerg Wunsch <joerg_wunsch>
Group administrator
Wed 16 Aug 2006 08:35:24 PM UTC, comment #5: 

Nevermind.  I manged it to unpatch the white-space changes, and
I also added a spin-loop delay implementation that allows for
short delays well below what usleep() can usually deliver,
including some kind of self-calibration for Unix-style systems.
I also added documentation.

Will commit that to CVS soon.

Joerg Wunsch <joerg_wunsch>
Group administrator
Tue 15 Aug 2006 01:20:56 AM UTC, comment #4: 

Sorry about the whitespace... Will fix the patch..

Neil Davey <npdavey>
Mon 14 Aug 2006 10:46:47 PM UTC, comment #3: 

Btw., on my FreeBSD, it just gets dog slow (yes, really) when
I specify any -i value greater than 1.  Good enough to recover
from an accidentally set 128 kHz oscillator fuse, but not usable
at all otherwise. :-(

Maybe usleep() is really too heavy-weight here, and we'd better
resort to a spin-loop.

Joerg Wunsch <joerg_wunsch>
Group administrator
Mon 14 Aug 2006 10:25:40 PM UTC, comment #2: 

I'm about to apply the patch, and already adapted it to the
recent changes (mainly caused by the added support for serial
bit-bang adapters).

Now I had to find that you also removed all trailing whitespace
in your patch.  Cosmetic changes like this one are not supposed
to be done in the same run as true code changes (as they might
hide the actual code changes).  Would you please care to
resubmit the patch without the gratitous whitespace changes?

I think the feature is really a useful one!

Joerg Wunsch <joerg_wunsch>
Group administrator
Mon 24 Oct 2005 03:33:04 AM UTC, comment #1: 

I should also add that the usleep delay is only called if the delay is greater than 1.

The syntax for controlling the delay is -i <delay>

Neil Davey <npdavey>
Sat 22 Oct 2005 05:02:35 AM UTC, original submission:  

I've added a command line argument that is used to control the bit clock for parallel port programming interfaces.
The parameter passed on the command line is passed directly to usleep.

This patch is useful if you use the internal 128kHz clock as you main source

This was compiled under cygwin, and used on Win2K using the bsd style hardware programmer.

Neil Davey <npdavey>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #8101:  avrdude-5.0-npd_ispdelay_2005-10-20.patch.bz2 added by npdavey (5KiB - application/x-bzip2 - ISP bit clock delay control patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 5 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-08-17 joerg_wunsch StatusIn Progress Done
    Open/ClosedOpen Closed
2006-08-14 joerg_wunsch StatusNone In Progress
    Assigned toNone joerg_wunsch
2005-10-22 npdavey Attached File- Added avrdude-5.0-npd_ispdelay_2005-10-20.patch.bz2, #5352

Back to the top

Powered by Savane 3.13-aa77.
Corresponding source code