Code:
    timer_compare_new=  ICR1L; 
    timer_compare_new= ( ICR1H<< 8);
Ist es Absicht, dass du den Wert aus ICR1L einfach "wegschmeißt" und nur den Wert aus ICR1H verwendest? Ich vermute mal nein, denn sonst würde es dich ja nicht wundern, dass du als Ergebnis nur 0 oder 256 bekommst.

Warum machst du es dir nicht etwas einfacher, und verwendest "timer_compare_new= ICR1;".