[avrdude-dev] [bug #31779] Add support for addressing usbtinyisp with -P option

[Grygoriy Fuchedzhy]

Follow-up Comment #5, bug #31779 (project avrdude):

 . Please don't mess with the code in main.c, at
> least not within a patch that is supposed to only
> affect the usbtiny.

done

> . For consistency, please follow the -P usb
> convention as used by other USB-based
> programmers. I.e., plain -P usb (which should be
> the default fot the usbtiny) means "use the first
> matching device found on the USB", and -P
> usb:<something else> means "use a specific
> device". For other USB-based programmers
> <something else> is the (trailing part of the)
> serial number, for the usbtiny, the idea to use
> the (OS-specific) bus/device pair has some merit.

done. Just like it was in previous patch bus is
struct usb_bus::dirname.  device is struct usb_device::filename.
User always can look how they look like using -v option.

> . When (at least) one -v option has been
> specified, make the code print out all candidate
> devices (i.e. everything where the VID/PID
> matches). That way, the user can get a list of all
> candidate devices by specifying a guaranteed to be
> invalid string for <something else>; the
> documentation mentions "-v -P usb:xxx" as an
> example. The list of candidate devices that is
> printed then should be given in a way so the
> output can be copy&pasted into the
> commandline. Examples could look like:

done.

Please review and test as I can test using linux only.
New patch is against avrdude-5.10
Thanks!


(file #23824)
    _______________________________________________________

Additional Item Attachment:

File name: 0001-added-addressing-of-usbtiny-with-P-option.patch Size:2 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?31779>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/