Beginner Problem, LED steuern?

[Hubert.G]

Die Led sind schon richtig so, nur die Ansteuerung ist verkehrt.
Du beginnst mit

PORTC = 0b0100000; // Rote LED
Damit setzt du PC5 auf 1 (5V) die Led ist aber auch gegen 5V geschaltet und leuchtet daher nicht.
Infolge läuft deine Ansteuerung verkehrt.
Übersichtlicher ist auch die Schreibweise wie sie "schorsch_76" schon gezeigt hat:

PORTC |= (1<<5); // setzt das 5te Bit und lässt alle anderen so wie sie sind
PORTC &= ~(1<<5); // löscht das Bit 5 und lässt alle anderen so wie sie sind (Hier leuchtet die Led)

[micron]

Hallo,
so, hab jetzt den code überarbeitet von mikrocontroller.net nochmal das tutorial angeschaut, das funktioniert jetzt so wie gedacht.
Das Problem war meine falsche ansteuerung wie Hubert es gesagt, hab das falsch verstanden -.-.

Hier der code mit Kommentar, falls der falsch ist bitte nimmt euch etwas Zeit nochmal und sagt was ich falsch beschrieben habe, so hab ich das jedenfalls verstanden:

int main()
{
	
	//DDRC |= (1<<PC5);
	//DDRC |= (1<<PC2);
	//DDRC |= (1<<PC1);
	//
	DDRC |= (1<<PC5)|(1<<PC2)|(1<<PC1); // Pins 5, 2 und 1 auf Ausgang setzen, jetzt leuchten alle LEDs
	
	while(1)
	{
		//TODO:: Please write your application code
                // Rote LED brennt
		PORTC |= (1 << PC1); // Grüne und 
		PORTC |= (1 << PC2);// gelbe deaktivieren
		_delay_ms(1000); // 1sec warten
		PORTC |= (1 << PC5); // Bit PC5 setzen->rote LED aus
		PORTC &= ~(1 << PC2); // Bit 2 auf Port 2 löschen die gelbe LED brennt
		_delay_ms(1000);
   		PORTC |= (1 << PC2); // das selbe wieder mit Port PC2, Bit setzen gelbe LED ist aus
		PORTC &= ~(1 << PC1); // Bit löschen grüne LED brennt
		_delay_ms(1000);
		PORTC &= ~(1 << PC5); // rot wieder aktivieren
	}
	return 0;
}

Ich hoffe ich hab das jetzt richtig verstanden, wenn nicht muss die Docu noch mal her

Besten Dank
micron

[Hubert.G]

Ich denke mal du hast es richtig verstanden. Ein kleiner Schönheitsfehler ist aber noch drinnen.

    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.