Verzögerung bei Lauflicht / Atmega8 / C

[Tim0 (Gast)]

Guten Tag,
ich habe vor 2 Tagen angefangen mich mit der 
Mikrocontroller-Programmierung
zu beschäftigen und habe nun versucht ein Lauflicht zu Programmieren 
welches hin und her läuft.
Das hin un her laufen habe ich hinbekommen, jedoch hat das Lauflich 
nachdem es einmal hin und her gelaufen ist eine kurze Verzögerung die 
ich nicht weg bekomme.
Könnt ihr mir evtl. sagen was ich falsch gemacht habe?

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

int main(void)
{

int n, EINAUS=0;

DDRB = 0xFF; // Alle Pins in PORTB als Ausgang definieren
DDRD = 0x00; // Alle Pins in PORTD als Eingang definieren
PORTD = 0b00001100; // PULL-UP Widerstände für PD2 und PD3 setzen



while(1)
{

  if (!(PIND & (1<<PD2))) // Lauflicht einschalten
  {
  EINAUS = 1;
  }

  if (!(PIND & (1<<PD3))) // Lauflicht ausschalten
  {
  EINAUS = 0;
  }

  if (EINAUS == 1) 
  {
    for (n=0; n<=7; n++) // Pin 0-7
    {
      PORTB |= (1<<n); // Pin n im Port B setzen
      _delay_ms(1000); // Verzögerung von 1000ms
      PORTB &= ~(1<<n); // Pin n im Port B rücksetzen
    }
    
    for (n=7; n>=1; n--) // Pin 7-0
    {
      PORTB |= (1<<n); // Pin n im Port B setzen
      _delay_ms(1000); // Verzögerung von 1000ms
      PORTB &= ~(1<<n); // Pin n im Port B rücksetzen
    }
    
  }
  else
  {
  PORTB = 0x00; // Alle Pins im Port B auf 0 setzen
  }
}

}

Viele Grüße
Tim0

[Eric B. (beric)]

Tipp: welche Werte nimmt n nacheinander an?

[Marc V. (Firma: Vescomp) (logarithmus)]

Tim0 schrieb:
> for (n=7; n>=1; n--) // Pin 7-0

 Und bit 0 ?

[Tobias (Gast)]

Endlich kann ich mal helfen!
Die Antwort ist ganz einfach:

while(1)
{

  if (!(PIND & (1<<PD2))) // Lauflicht einschalten
  {
  EINAUS = 1;
  }

  if (!(PIND & (1<<PD3))) // Lauflicht ausschalten
  {
  EINAUS = 0;
  }

  if (EINAUS == 1)

Nach jedem Durchlauf wird geprüft, ob das alles zutrifft, das braucht
Zeit. Hatte damals am Anfang exakt das gleiche "Problem".
Eine interessante Lösung bieten die Interrupts, schon mal damit 
gespielt?
Ist wirklich einfach, man muss sich nur einmal durcharbeiten!

Grüße aus Wien!
Tobias

[jaja (Gast)]

Tim0 schrieb:
>     for (n=0; n<=7; n++) // Pin 0-7
>     {
>       PORTB |= (1<<n); // Pin n im Port B setzen

eher die Frage wo PB8 ist.

[Peter II (Gast)]

Tobias schrieb:
> Nach jedem Durchlauf wird geprüft, ob das alles zutrifft, das braucht
> Zeit.

und du glaubst irgend jemand sieht das? Die Abfrage geht in viel weniger 
als 1ms.

> Endlich kann ich mal helfen!
nicht wirklich.

[Torsten C. (torsten_c)]

Tim0 schrieb:
> jedoch hat das Lauflich
> nachdem es einmal hin und her gelaufen ist eine kurze Verzögerung die
> ich nicht weg bekomme.

Einmal von 0..7 und dann von 7..1.
Die 7 ist also 'doppelt' und leuchtet 2000ms.

Du willst vermutlich

for (n=0; n<=6; n++) // Pin 0-6 und
for (n=7; n>=1; n--) // Pin 7-1

[Walter S. (avatar)]

jaja schrieb:
> eher die Frage wo PB8 ist.

das ist nicht die Frage denn es werden PB0 bis PB7 angesprochen

[Tim0 (Gast)]

Danke für schnellen Antworten :)
Ich hatte gleich mehrere Fehler.
Eric B. schrieb:
> Tipp: welche Werte nimmt n nacheinander an?
Ich dachte die Verzögerung wäre nach einem Durchlauf, aber sie war 
mittendrin beim Wechsel zwischen den Schleifen.

Außerdem habe ich nur 6 LED's angeschlossen ^^'

Tobias schrieb:
> ...
> Nach jedem Durchlauf wird geprüft, ob das alles zutrifft, das braucht
> Zeit. Hatte damals am Anfang exakt das gleiche "Problem".
> Eine interessante Lösung bieten die Interrupts, schon mal damit
> gespielt?
> Ist wirklich einfach, man muss sich nur einmal durcharbeiten!
>
> Grüße aus Wien!
> Tobias

In diesem Fall war es wohl nicht der Fehler, aber das ist gut zu Wissen 
:)
Ich arbeite gerade die Videos von Stefan Wintgen durch, Interuppts sind 
tatsächlich das nächste Thema :)
https://www.youtube.com/watch?v=UpxemFUVnFI&index=5&list=PLg8rGAQEOsuh9ZCwP6aW5ycCM5PFFqoNe
Falls ihr noch weitere gute Tutorials zum Thema µC-Programmierung kennt, 
würde ich mich freuen wenn ihr diese hier Posten würdet :)

Marc V. schrieb:
> Tim0 schrieb:
>> for (n=7; n>=1; n--) // Pin 7-0
>
>  Und bit 0 ?

Bit 0 wird bei "Neustart" der If-Schleife wieder aufgerufen :)

Hier mein korrigierter Code

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

int main(void)
{

int n, EINAUS=0;

DDRB = 0xFF; // Alle Pins in PORTB als Ausgang definieren
DDRD = 0x00; // Alle Pins in PORTD als Eingang definieren
PORTD = 0b00001100; // PULL-UP Widerstände für PD2 und PD3 setzen



while(1)
{

  if (!(PIND & (1<<PD2))) // Lauflicht einschalten
  {
  EINAUS = 1;
  }

  if (!(PIND & (1<<PD3))) // Lauflicht ausschalten
  {
  EINAUS = 0;
  }

  if (EINAUS == 1) 
  {
    for (n=0; n<=4; n++) // Pin 0-4 // bzw. LED 1-5
    {
      PORTB |= (1<<n); // Pin n im Port B setzen
      _delay_ms(1000); // Verzögerung von 1000ms
      PORTB &= ~(1<<n); // Pin n im Port B rücksetzen
    }
    
    for (n=5; n>=1; n--) // Pin 5-1 // bzw. LED 6-2
    {
      PORTB |= (1<<n); // Pin n im Port B setzen
      _delay_ms(1000); // Verzögerung von 1000ms
      PORTB &= ~(1<<n); // Pin n im Port B rücksetzen
    }
    
  }
  else
  {
  PORTB = 0x00; // Alle Pins im Port B auf 0 setzen
  }
}

}

Viele Grüße
Tim0

[Peter II (Gast)]

Tim0 schrieb:
> PORTB |= (1<<n);

wenn man ganz genau sein will, dann darf man das auch so nicht 
schreiben. Bei einem Atmel wird (1<<n) in einer schleife berechnet. 
Damit hast du abhängig von n unterschiedliche Ausführungszeiten.

[Tobias (Gast)]

Ach du meinst mit den eigenen Augen! Dachte du meinst am Oszi.
Da kann es kurz irritieren, aber eine LED zu vergessen ist auch gut.
Viel Erfolg weiterhin!

[Tim0 (Gast)]

Peter II schrieb:
> Tim0 schrieb:
>> PORTB |= (1<<n);
>
> wenn man ganz genau sein will, dann darf man das auch so nicht
> schreiben. Bei einem Atmel wird (1<<n) in einer schleife berechnet.
> Damit hast du abhängig von n unterschiedliche Ausführungszeiten.

Wie könnte ich das denn alternativ schreiben?

Gruß
Tim0

[Peter II (Gast)]

Tim0 schrieb:
> Wie könnte ich das denn alternativ schreiben?

uint8_t value = 1;
while( value != 128 )
{
      PORTB = value; // Pin n im Port B setzen
      _delay_ms(1000); // Verzögerung von 1000ms
      PORTB = 0;
      value = value << 1;
}
    
while( value != 1 )
{
      PORTB = value; // Pin n im Port B setzen
      _delay_ms(1000); // Verzögerung von 1000ms
      PORTB = 0;
      value = value >> 1;
}

aber den unterschied wirst du nicht sehen. Das geht dabei wirklich nur 
im ein paar Takte.

[Tim0 (Gast)]

Peter II schrieb:
> Tim0 schrieb:
>> Wie könnte ich das denn alternativ schreiben?
> uint8_t value = 1;
> while( value != 128 )
> {
>       PORTB = value; // Pin n im Port B setzen
>       _delay_ms(1000); // Verzögerung von 1000ms
>       PORTB = 0;
>       value = value << 1;
> }
>
> while( value != 1 )
> {
>       PORTB = value; // Pin n im Port B setzen
>       _delay_ms(1000); // Verzögerung von 1000ms
>       PORTB = 0;
>       value = value >> 1;
> }
>
> aber den unterschied wirst du nicht sehen. Das geht dabei wirklich nur
> im ein paar Takte.

Danke :)
Auch wenn der Unterschied in dem Beispiel wohl kaum merkbar ist, wollte 
ich mir ganz gerne von vornherein angewöhnen sauberen Code zu schreiben. 
:)

Gruß
Tim0