Is the first set correct for standard UNO? I don't remember ISP code being this large or complicated but the last time I looked at it was before version 1.0 so I'd imagine a lot has changed since then. I am having a bit of trouble understanding the purpose of current style and old style, was there an Arduino board that has UNO form factor but used different pinouts?
The reason for the second set of pins is described in the comment:
// HOODLOADER2 means running sketches on the atmega16u2
// serial converter chips on Uno or Mega boards.
// We must use pins that are broken out:
#else
#define RESET 4
#define LED_HB 7
#define LED_ERR 6
#define LED_PMODE 5
Since the code could also run on an atmega16u2 with less usable pins.
Thanks for the info. I don't have 16u2 based Arduino for programming, just for USB to serial on Arduino boards. The first set of numbers will be used to finalize my design.
I'll keep this in mind if I ever needed high voltage programming. Right now I have ATTiny 44/84 and ATTiny 45/85 shield (no LEDs as they are tiny board) and am currently doing 324/644/1284 which is a huge chip requiring bigger PCB so I ended up with extra space that I could slip in a few LEDs for status. Is this the biggest DIP chip of all AVR chip? ATMega2560 does not come in DIP AFAIK.