Follow-up Comment #1, patch #7729 (project avrdude):

Hi,

I had a quick look and it looks fine to me. I have just one question. You
changed:
        if (pgm->pinno[PIN_AVR_RESET] < 4 || pgm->pinno[PIN_AVR_RESET] == 0) {
to:
        if (pgm->pinno[PIN_AVR_RESET] < 5 || pgm->pinno[PIN_AVR_RESET] == 0) {
and I don't understand why. Using pin 4 (TMS/CS) as reset-pin is a valid
choice and indeed the default. Four lines below pgm->pinno[PIN_AVR_RESET]
actually gets set to 4.

Cheers,
Hannes

    _______________________________________________________

Reply to this item at:

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

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


_______________________________________________
avrdude-dev mailing list
avrdude-dev@nongnu.org
https://lists.nongnu.org/mailman/listinfo/avrdude-dev

Reply via email to