bugAVR Downloader/UploaDEr - Bugs: bug #23685, avrdude uses parallel device for...

 
 

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

bug #23685: avrdude uses parallel device for "ponyser" programmer

Submitted by:  Bernhard Walle <bwalle>
Submitted on:  Mon 23 Jun 2008 06:59:27 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: FixedPrivacy: Public
Assigned to: Joerg Wunsch <joerg_wunsch>Originator Name: Bernhard Walle
Open/Closed: ClosedRelease: None
Programmer hardware: Device type: 

Mon 30 Jan 2012 05:14:36 PM UTC, comment #3:

I implemented a connection_type parameter that is tied to
each programmer definition in avrdude.conf. While this
information is partially redundant (e.g. you have "type =
par;", followed by "connection_type = parallel;"), it
explicitly spells out the connection type for each programmer,
centralizing the setting. The programmer implementation can
sometimes handle multiple transports, e.g. stk500v2 could be
either serial or USB by default, so I think mentioning the
connection method in avrdude.conf is the most obvious place.

(Committed in SVN revision 1047.)

Joerg Wunsch <joerg_wunsch>
Project AdministratorIn charge of this item.
Mon 02 Jan 2012 05:15:12 PM UTC, comment #2:

I would solve the problem by introducing another method in the programmer data structure.

The provided function has to return the correct value.
If port is given it returns port otherwise it returns the serial or parallel value.

An USB programmer could also return a string like "usb" or anything else. When we define that the function is called after pgm->setup() and the returned value is not used after pgm->teardown(), the programmer also could create in its private data a dynamic created string. (For example when programmers check the usb ports to find the right device, they could show to used port here.)

PS:
Maybe we also should introduce a default_usb value (and add it to parameter list above) as new programmers are likely to be connected by usb (or a serial emulation).

Rene Liebscher <rliebscher>
Project Member
Tue 04 Nov 2008 11:16:18 AM UTC, comment #1:

Your patch just extends an evil hack that is already present
in the sources.

What's IMHO needed but requires a bit more of work is a
framework where each programmer type can select the interface
class it would like to use by default (preferrably through
avrdude.conf itself). That way, an AVR Dragon and an AVRISP
mkII could default to USB, while the old AVRISP, or your serbb
programmer default to the serial port, and an STK200 to the
parallel port.

Joerg Wunsch <joerg_wunsch>
Project AdministratorIn charge of this item.
Mon 23 Jun 2008 06:59:27 PM UTC, original submission:

When I specify

default_programmer = "pony-stk200";
default_parallel = "/dev/parport0";
default_serial = "/dev/ttyS0";

in my ~/.avrduderc, I get following error message:

/dev/parport0: tcgetattr: Invalid argument
ioctl("TIOCxBRK"): Invalid argument

That implies that the default_parallel is used even if the programmer needs a serial device. The attached patch should fix the problem.

Bernhard Walle <bwalle>

 

Attached Files
file #15918:  avrdude-use-serial-device-for-serbb added by bwalle (796B - application/octet-stream - Fix without printf() :-))
file #15917:  avrdude-use-serial-device-for-serbb added by bwalle (855B - application/octet-stream - Fix for the problem)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rliebscher (Posted a comment)
  • -unavailable- added by joerg_wunsch (Posted a comment)
  • -unavailable- added by bwalle (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 30 Jan 2012 05:14:36 PM UTCjoerg_wunschStatusPostponed=>Fixed
      Open/ClosedOpen=>Closed
    Tue 04 Nov 2008 11:16:18 AM UTCjoerg_wunschAssigned toNone=>joerg_wunsch
    Tue 04 Nov 2008 11:16:17 AM UTCjoerg_wunschStatusNone=>Postponed
    Mon 23 Jun 2008 07:02:13 PM UTCbwalleAttached File-=>Added avrdude-use-serial-device-for-serbb, #15918
    Mon 23 Jun 2008 06:59:27 PM UTCbwalleAttached File-=>Added avrdude-use-serial-device-for-serbb, #15917

    Back to the top


    Powered by Savane 3.1-cleanup1