AVR Freaks

Which is a strong reason NOT to use PROGMEM. Unless using C++ (where it is not available) then __flash is better in almost every respect. Instead of:

const PROGMEM uint8_t DriveTech[5][3] = {
    {7, 8, 9},
    {5, 18, 23}
    etc.
};

PORTB = pgm_read_byte(&DriveTech[2][1]);

you really could just use:

const __flash uint8_t DriveTech[5][3] = {
    {7, 8, 9},
    {5, 18, 23}
    etc.
};

PORTB = DriveTech[2][1];

Because "__flash" is a modifier that is inherent in the data type, when the compiler comes to read the data it already knows this has to be an LPM not an LDS read. No more need for any pgm_read_XXX(&...) nonsense any more.