[avrdude-dev] Re: Good news: STK500v2 and JTAG ICE mkII code committed to CVS

[Joerg Wunsch]

As Joerg Wunsch wrote:

> I've also got a patch from Bernd Walter still in the pipeline that
> increases the write blocksize for ser_posix.c, in order to speed up
> the communication by lowering the syscall overhead.  That's going to
> be committed later on.

That one has been committed now.

> Note that for the JTAG ICE, I did change a few things in the internal
> API.  [...]
> [...], and I made the serial_recv() function return a
> -1 instead of bailing out with exit(1) upon encountering a receive
> timeout (currently only done in the Posix implementation).

> The Win32 change for this still
> has to be done,

This one still needs a volunteer knowledgable with the W32 API.
Supposedly, after calling ReadFile(), GetLastError() needs to be
called and the result examined in order to distinguish a timeout from
other errors.  If the request timed out, the error message should only
be printed if (verbose > 1) (so by default, recovered timeouts won't
be announced), and the function should return -1.

> and the traditional drivers need to be converted to
> exit(1) upon encountering a timeout (as they're now getting a -1
> returned they didn't see before in that case).

I did this now.  I'm sure though at least the STK500v2 driver might
perform a more intelligent error handling then, and retry the request
instead.  (That's what the JTAG ICE mkII code does.)  I'll leave that
to someone else to implement though, there's already more things
waiting for me.

-- 
cheers, J"org               .-.-.   --... ...--   -.. .  DL8DTL

http://www.sax.de/~joerg/                        NIC: JW11-RIPE
Never trust an operating system you don't have sources for. ;-)