Lauflicht überspringt LEDs - Anfängerfrage

[Benedikt K. (benek)]

Hallo Forum,
ich bin gerade dabei mich in AVR GCC einzuarbeiten. Bis jetzt habe ich 
auch nur LEDs zum Blinken gebracht und Taster abgefragt.
Das Programm, das ich heute für ein Lauflicht geschrieben habe, 
funktioniert aber nicht bzw. ich finde den Fehler nicht. Vielleicht hat 
ja gerade jemand Zeit über meinen Code zu schauen :).

#define F_CPU 16000000L

#include <avr/io.h>
#include <util/delay.h>


int main(void)
{
  
  DDRB = (1 << DDB0) | (1 << DDB1) | (1 << DDB2) | (1 << DDB3) | (1 << DDB4) | (1 << DDB5);
  PORTB = 0x00;
  
    while(1)
    {
    for(int i = 0; i <= 5; i++){
      PORTB = (1 << PORTB + i);
      _delay_ms(100);
      PORTB &= ~(1 << PORTB + i);
      _delay_ms(100);
    }
    }
}

[Frank M. (ukw) (Moderator)]

Benedikt K. schrieb:

>       PORTB = (1 << PORTB + i);
>       ...
>       PORTB &= ~(1 << PORTB + i);

Richtig:

        PORTB |= (1 << i);
        ...
        PORTB &= ~(1 << i);

Gruß,

Frank

[Benedikt K. (benek)]

Frank M. schrieb:
> Benedikt K. schrieb:
>
>>       PORTB = (1 << PORTB + i);
>>       ...
>>       PORTB &= ~(1 << PORTB + i);
>
> Richtig:
>
>         PORTB |= (1 << i);
>         ...
>         PORTB &= ~(1 << i);
>
> Gruß,
>
> Frank

Vielen Dank. So funktioniert es jetzt auch :)

MfG Benedikt

[Fabian O. (xfr)]

Du willst in der Schleife LED i an- bzw. ausschalten. Das geht mit:

PORTB |= (1 << i)

bzw.

PORTB &= ~(1 << i)

Den aktuellen Wert vom Port brauchst Du auf der rechten Seite nicht.

[Frank M. (ukw) (Moderator)]

Benedikt K. schrieb:
> Vielen Dank. So funktioniert es jetzt auch :)

Du weisst, dass der Ausdruck "PORTB" rechts vom Schiebeoperator "<<" 
kompletter Unsinn war?

[Benedikt K. (benek)]

Frank M. schrieb:
> Benedikt K. schrieb:
>> Vielen Dank. So funktioniert es jetzt auch :)
>
> Du weisst, dass der Ausdruck "PORTB" rechts vom Schiebeoperator "<<"
> kompletter Unsinn war?

Jetzt weis ich es...