Ich denke mal du hast es richtig verstanden. Ein kleiner Schönheitsfehler ist aber noch drinnen.
Code:
    while(1)     {         //TODO:: Please write your application code                 // Rote LED brennt         PORTC |= (1 << PC1); // Grüne und           "Diese beiden Zeilen gehören vor das          PORTC |= (1 << PC2);// gelbe deaktivieren   "while(1)        
 _delay_ms(1000); // 1sec warten
PC1 und PC2 musst du nur einmal aktivieren, gehört daher vor das while(1)

Da ist wohl irgendwie die Formatierung davongelaufen.