Why do you invert the 'flag' variable in the ISR? First time through it will work, but nowhere in your code is it reset!

Why not just say:


timer0_isr:
{
flag = 1;
TR0 = 0; //stop the timer
}


main code:

if (flag)
{
flag = 0;
//do other stuff
}