Hallo,
so auf den ersten Blick hätte ich zwei Anmerkungen,
  • Wie schauts mit einem Prescaler aus, wenn kein Wert gesetzt wird ist er 0, und somit der Timer aus.
  • Du schreibts nur nach OCR1AL, das ist aber ein 16-Bit Register, da sollte man trotzdem OCR1A angeben, dann macht der Compiler den Rest mit High und Low-Register.