[avrdude-dev] [patch #6866] bug #26703: [Feature Request] Support device reset for 'arduino' programmer type

[Michael Dreher]

Follow-up Comment #6, patch #6866 (project avrdude):

Hi Michal,

there is a little typo in ser_win32.c. The filedescriptor parameter is named
"fdp" instead of "fd" which prohibts compiling it.

original:
static int ser_set_dtr_rts(union filedescriptor *fdp, int is_on)

changed:
static int ser_set_dtr_rts(union filedescriptor *fd, int is_on)

The current CVS version (845) does not reset DTR when closing the serial port
under Windows. This leaves the DTR signal at the WT-11 bluetooth chip at a
high level. Other serial port hardware+drivers+OS combinations like the FTDI
chip on the Duemilanove behave differently.

Please add a CLEARDTR and CLEARRTS when closing the port.

Another potential problem is, that ser_setspeed() will always reset DTR and
RTS, but it should keep their values. So the result depends on the order
between calling ser_setspeed() and set_set_dtr_rts(). That was something I
also forgot in my patch.

  Michael



    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?6866>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/