In der TIMER1_OVF_vect cli() und sei() jeweils vor bzw. nach den ifs.

Ich weiss nicht, ob die _BV Makros in den neuen versionen des avr_gcc noch verwendet werden. Aber Du solltest die auf jeden fall einheitlich benutzen (mal hast Du _BV verwendet, ein anderes mal verwendest Du normale shifts).