RE: [avr-gcc-list] problems with timer0 on atmega128

[Artur Lipowski]

Assuming (rather slow) 4Mhz crystal your interrupt will be called about 15
times per second, and your first internal delay loops will take about 0.25
microsecond. You should use scope or be very fast to detect blinks 8-)
You should clear timer on compare match or use apopriate timer settings or
use 16 bit timer for longer periods ("one shot" work).
You start interrupts before timer is set up so no repeatable behaviour of
the timer can be assumed.

Delay loops are not needed at all, just use/reprogram timer(s) for all
delays.
Do not use outb.
Do not use crypric values when setting control registers eg outb(TCCR0, 5),
there are defined names for all bits.
When you declare variable static you not need to initialize it to 0.
Do not use static when not needed (eg. for max and count) to allow compiler
optimization (especially in size).
Insted of setting whole ports just define pins connected to leds and change
only these bits in IO ports.

Regards
--
Artur Lipowski

avr-gcc-list at http://avr1.org