[avrdude-dev] [patch #8380] adds 500k 1M 2M baud to ser_posix.c

[Joerg Wunsch]

Follow-up Comment #8, patch #8380 (project avrdude):

Thank you for the patch!

Some remarks:

. I don't think you have to special-case OSX in any way.  It does
  already work the way it is now, just the same like *BSD (which it is
  derived from in that respect); the Bxxxx names are just aliases for
  number xxxx there, so cfsetispeed() and cfsetospeed() can be fed the
  raw baudrate numbers as well as the Bxxxx defines.

. Your patch for Linux is probably fine, though I cannot test it
  (lacking any device with a non-standard baudrate).

. Please don't add unrelated changes to a patchfile (indentation
  changes in line 96 ff.).


    _______________________________________________________

Reply to this item at:

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

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