Problem bei einfachen Lauflicht :-(

[R. S. (schmidtbot)]

Hallo,

ich habe bis gestern gedacht, ich kann mal schnell nebenbei ein 
Lauflicht programmieren. Doch leider wurde mir nach längerer Fehlersuche 
eines besseren belehrt.

Das Licht sollte einfach nur nach rechts laufen. Das tut es auch mehr 
oder weniger, aber jeden LED blinkt nur kurz auf statt für 1sec zu 
leuchten. Das lustige an der Sach ist aber wenn ich über ledra oder 
ledla alle LEDs blinken lasse bzw. aktivieren Funktioniert es.
Kann mir bitte jemand helfen? Ich finden den Fehler leider nicht.

AVR Atmega8
Internen Quarz
AVR Studio 6
LEDS über BC639 an:
PB6
PB7
PD5
PD6
PD7
PB0

das wäre der Sourcecode:
------------------------------------------------------------------------ 
---
#include <avr/io.h>
#include <util/delay.h>
#define F_CPU 1000000

unsigned char ledl1 = 0;
unsigned char ledl2 = 0;
unsigned char ledl3 = 0;
unsigned char ledr1 = 0;
unsigned char ledr2 = 0;
unsigned char ledr3 = 0;
unsigned char ledla = 0;
unsigned char ledra = 0;



void ausgabe(void)
{
  if (ledl1 == 1)
  {
    PORTB = PORTB | (1<<PD0);
  }
  else
  {
    PORTB = PORTB &~ (1<<PB0);
  }

  if (ledl2 == 1)
  {
    PORTD = PORTD | (1<<PD7);
  }
  else
  {
    PORTD = PORTD &~ (1<<PD7);
  }


  if (ledl3 == 1)
  {
    PORTD = PORTD | (1<<PD6);
  }
  else
  {
    PORTD = PORTD &~ (1<<PD6);
  }

  if (ledr1 == 1)
  {
    PORTD = PORTD | (1<<PD5);
  }
  else
  {
    PORTD= PORTD &~ (1<<PD5);
  }
  if (ledr2 == 1)
  {
    PORTB = PORTB | (1<<PB7);
  }
  else
  {
    PORTB = PORTB &~ (1<<PB7);
  }

  if (ledr3 == 1)
  {
    PORTB = PORTB | (1<<PB6);
  }
  else
  {
    PORTB = PORTB &~ (1<<PB6);
  }

  if (ledla == 1)
  {
    PORTB = PORTB | (1<<PB0);
    PORTD = PORTD | (1<<PD7);
    PORTD = PORTD | (1<<PD6);
  }
  else
  {
    PORTB = PORTB &~ (1<<PB0);
    PORTD = PORTD &~ (1<<PD7);
    PORTD = PORTD &~ (1<<PD6);
  }

  if (ledra == 1)
  {
    PORTD = PORTD | (1<<PD5);
    PORTB = PORTB | (1<<PB7);
    PORTB = PORTB | (1<<PB6);
  }
  else
  {
    PORTD = PORTD &~ (1<<PD5);
    PORTB = PORTB &~ (1<<PB7);
    PORTB = PORTB &~ (1<<PB6);
  }

}

int main(void)
{


DDRB = DDRB | (1<<PB0)| (1<<PB6)| (1<<PB7); // Ausgänge
DDRD = DDRD | (1<<PD5)| (1<<PD6)| (1<<PD7); // Ausgänge
PORTD = 0x1F; //Pullups für Eingänge aktiviren



    while(1 == 1)
 {
  ledl1=0;
  ledl2=0;
  ledl3=0;
  ledr1=0;
  ledr2=0;
  ledr3=0;
  ausgabe();
  _delay_ms(1000);

  ledl1=0;
  ledl2=1;
  ledl3=0;
  ledr1=0;
  ledr2=0;
  ledr3=0;
  ausgabe();
  _delay_ms(1000);

  ledl1=0;
  ledl2=0;
  ledl3=1;
  ledr1=0;
  ledr2=0;
  ledr3=0;
  ausgabe();
  _delay_ms(1000);

  ledl1=0;
  ledl2=0;
  ledl3=0;
  ledr1=1;
  ledr2=0;
  ledr3=0;
  ausgabe();
  _delay_ms(1000);


  ledl1=0;
  ledl2=0;
  ledl3=0;
  ledr1=0;
  ledr2=1;
  ledr3=0;
  ausgabe();
  _delay_ms(1000);

  ledl1=0;
  ledl2=0;
  ledl3=0;
  ledr1=0;
  ledr2=0;
  ledr3=1;
  ausgabe();
  _delay_ms(1000);
    }
}
------------------------------------------------------------------------ 
--
Besten Dank!

PS: Ich würde fast darauf wetten, dass es ein peinlicher Fehler ist :-)

[Blueberry (Gast)]

Rolf Schmidt schrieb:
> PS: Ich würde fast darauf wetten, dass es ein peinlicher Fehler ist :-)

Das glaube ich auch, so macht doch kein Mensch ein Lauflicht!
Schau Dir mal die logischen Operationen wie Bitshift an.
Alternativ auch die Multiplikation/Division mit/durch 2.

[Ingo (Gast)]

Wie ist deine Optimierung eingestellt?
Ich sehe keinen Fehler. Beschreib mal dein Blitzen!
Welche IDE?



Ingo

[hans (Gast)]

Reihenfolge

#include <util/delay.h>
#define F_CPU 1000000

falsch ;(

hans

[Karl H. (kbuchegg)]

Wow.
Das ist das komplizierteste Lauflicht, an das mich erinnern kann.
Und das will in diesem Forum was heißen.

[trollalala (Gast)]

Schmeiß den Code weg und fang von vorne an. Vorher aber bitte in einem 
Buch über den shift-operator lesen...

Alle LEDs an einen PORT, dann ist das ein 3-Zeiler.

[trollalala (Gast)]

OK, zu langsam...

[Ingo (Gast)]

hans schrieb:
> Reihenfolge#include <util/delay.h>
> #define F_CPU 1000000
> falsch ;(
>
> hans

Richtig, hatte ich überlesen (Bei dem riesen Code).
Aber wenn der interne Oszillator läuft (8Mhz oder 1MHz), dann sollte die 
1s halt etwas länger sein.



Ingo

[Karl H. (kbuchegg)]

Karl Heinz Buchegger schrieb:
> Wow.
> Das ist das komplizierteste Lauflicht, an das mich erinnern kann.
> Und das will in diesem Forum was heißen.

Aber ....
Ingo hat Recht.
Einen offensichtlichen Fehler gibt es nicht.

[John B. (johnbauer)]

Wie sieht deine Schaltung aus (LEDs, Transistoren etc.)?

[debugger (Gast)]

Dämlichster (Troll-)Fehler wo gibt!!!

Bei ausgabe() schaltest Du deine Wunsch-Led ein und unten bei der 
letzten if-clause setzt Du beide Ports wieder auf aus. Ausnahme: Es 
werden alle LEDs ein- oder ausgeschaltet.

Ab in die Ecke und schämen!!!

[Karl H. (kbuchegg)]

debugger schrieb:
> Dämlichster (Troll-)Fehler wo gibt!!!
>
> Bei ausgabe() schaltest Du deine Wunsch-Led ein und unten bei der
> letzten if-clause setzt Du beide Ports wieder auf aus. Ausnahme: Es
> werden alle LEDs ein- oder ausgeschaltet.

<Hand an die Strin klatsch>

>
> Ab in die Ecke und schämen!!!

Richtig.
Nämlich: wir Helfer.

[R. S. (schmidtbot)]

Klar kenn ich die shift-operatoren. Wenn man sich den Code an sieht, 
dann sieht man auch, dass ich ihn zum setzten der Ausgänge nehme :-)
Er ist aber wegen Zusatzfunktionen (nicht online) leider ungeeignet.
Und wenn die Hardware schon fertig ist, kann man nicht einfach 
irgendwelche Pins tauschen :-)

Aber ich hatte recht, es war ein sehr peinlicher und einfacher Fehler. 
X+2 Augen sind halt immer besser als nur zwei :-)

Danke für den Hinweis

[noonien (Gast)]

Wirst du nach Codezeilen bezahlt? :)

[Stefan F. (sfrings)]

Du könntest doch wenigstens die ganzen (unsigned char) durch eine 
einzige ersetzen, bei der jedes Bit für eine LED steht. Dann brauchst Du 
im Hauptprogramm auch nicht (gefühlte) 500 Schreibzugriffe, sondern nur:

unsigned char value;
while (1) {
  value=0b000001;
  do {
    ausgabe();
    _delay_ms(1000);
    value=value<<1;
  }
  while (value<=0b100000);
}

Nur so als Denkanstoß. Wie die ausgabe() Funktion umgeschrieben werden 
muss, kannst Du Dir sicher selbst denken.

[R. S. (schmidtbot)]

das muss ich natürlich zugeben, diese Variante ist bedeutent schöner 
programmiert als meine "quick and dirty" Version.

[Bart (Gast)]

Bei der langen Tipparbeit und Fehlersuche, ist das wohl kaum als "quick" 
zu bezeichnen...

[Karl H. (kbuchegg)]

R. Schmidt schrieb:

> Und wenn die Hardware schon fertig ist, kann man nicht einfach
> irgendwelche Pins tauschen :-)

Das ist richtig.
Aber man kann sich zb in einem, nein 2, Arrays die jeweils an den ports 
auszugebenden Werte ablegen und dann ganz einfach in ausgabe() in die 
Arrays indizieren um den Wert zu erfahren, der ausgegeben werden soll.
Dann spart man sich einen Großteil dieser if - else kette

[Holger (Gast)]

R. Schmidt schrieb:
> mal schnell nebenbei ein
> Lauflicht programmieren.

Gratuliere. Dein Code hat es in meine All-Time "so-macht-man-es-nicht" 
Sammlung geschafft...

[Karl H. (kbuchegg)]

Bart schrieb:
> Bei der langen Tipparbeit und Fehlersuche, ist das wohl kaum als "quick"
> zu bezeichnen...

Dafür dirty!

[Checker Bunny (Gast)]

Also ich finde den Code toll.
Den hebe ich mir auf und erschrecke damit kleine Kinder.

[RTFM (Gast)]

Checker Bunny schrieb:
> Den hebe ich mir auf

Das wäre doch was zur Abschreckung neugieriger Kollegen? Das muss auch 
in die Schreibunterlage mit rein!

[Fred (Gast)]

Ich bin zwar auch Anfänger, aber da musste auch ich schlucken. 
Kompliziert.
Wie es der Zufall so will, habe ich vor 2 Tagen auch ein Lauflicht 
gemacht.
Mein Code ist extrem simpel da er erstmal zum testen war, man kann noch 
etwas optimieren.
Das ganze läuft auf einem Attiny 44 mit internem Takt. Das einzigste was 
Du tun musste ist die Pinausgänge festlegen und Pinnamen entsprechend 
Deines Avr ändern.

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

int main(void)
{
  DDRA = (1 << PA0) | (1 << PA1) | (1 << PA2) | (1 << PA3) | (1 << PA5) 
| (1 << PA7);
  DDRB = (1 << PB0) | (1 << PB1) | (1 << PB2);

    while(1)
    {
    PORTA |= (1 << PA0); _delay_ms(70);
  PORTA &= ~((1 << PA0));
  PORTA |= (1 << PA1); _delay_ms(70);
  PORTA &= ~((1 << PA1));
  PORTA |= (1 << PA2); _delay_ms(70);
  PORTA &= ~((1 << PA2));
  PORTA |= (1 << PA3); _delay_ms(70);
  PORTA &= ~((1 << PA3));
  PORTB |= (1 << PB0); _delay_ms(70);
  PORTB &= ~((1 << PB0));
  PORTB |= (1 << PB1); _delay_ms(70);
  PORTB &= ~((1 << PB1));
  PORTB |= (1 << PB2); _delay_ms(70);
  PORTB &= ~((1 << PB2));
  PORTA |= (1 << PA7); _delay_ms(70);
  PORTA &= ~((1 << PA7));
  PORTA |= (1 << PA5); _delay_ms(70);
  PORTA &= ~((1 << PA5));

  PORTA |= (1 << PA7); _delay_ms(70);
  PORTA &= ~((1 << PA7));
  PORTB |= (1 << PB2); _delay_ms(70);
  PORTB &= ~((1 << PB2));
  PORTB |= (1 << PB1); _delay_ms(70);
  PORTB &= ~((1 << PB1));
  PORTB |= (1 << PB0); _delay_ms(70);
  PORTB &= ~((1 << PB0));
  PORTA |= (1 << PA3); _delay_ms(70);
  PORTA &= ~((1 << PA3));
  PORTA |= (1 << PA2); _delay_ms(70);
  PORTA &= ~((1 << PA2));
  PORTA |= (1 << PA1); _delay_ms(70);
  PORTA &= ~((1 << PA1));

}
}

Also wie man sieht geht es auch extrem einfach.
Gruß
Fred

[Fred (Gast)]

Achja. Du kannst z.B. einfach einen Vorwiderstand vor die LED bauen. 
Plus kommt vom Chip und Minus an gemeinsame Masse.

[EGS_TI (Gast)]

lol

[troll (Gast)]

Fred schrieb:
> Also wie man sieht geht es auch extrem einfach.
Genau:

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

int main(void)
{
  DDRA = 0xFF;
  uint16_t i;
  for(;;)
  {
    for(i=1;i!=(1<<8);i<<=1)
    {
      PORTA=(uint8_t)i;
      _delay_ms(70);
    }
  }
  return 0; //neaver reached
}

SCNR

Wie wäre es mit einem Wettbewerb: Wer schreibt das kürzeste Lauflicht?

[Fred (Gast)]

Das ist mir klar, daß ich das ganze in einer schleife machen kann.
Ich wollte halt nur zeigen, daß seines schon extrem umständlich ist.
Auch wenn Deines schön klein ist, so finde ich es absolut 
unübersichtlich.
Zum lernen finde ich die Portnamen etc. besser.

[troll (Gast)]

troll schrieb:
> Wie wäre es mit einem Wettbewerb: Wer schreibt das kürzeste Lauflicht?
Ich leg dann mal vor:

#include <avr/io.h>
#include <util/delay.h>
main(){DDRA=~0;char i;for(;;)for(i=0,PORTA=1;i<8;i++,PORTA<<=1,_delay_ms(70));}

Kann sein dass man noch unsigned vor den char setzen muss, hab gerade 
keinen AVR-GCC zur Hand. Mal sehen, vielleicht schlage ich Karl-Heinz ja 
dieses Mal...

[troll (Gast)]

troll schrieb:
> Kann sein dass man noch unsigned vor den char setzen muss
Nein Blödsinn, das war für eine andere Version. :-(

[Fred (Gast)]

Ich sehe schon, wie in so manchen anderen Threads wird hier alles ins 
lächerliche gezogen.
Vielleicht sollte man mal über die Gastfunktion nachdenken.

[troll (Gast)]

Fred schrieb:
> Ich sehe schon, wie in so manchen anderen Threads wird hier alles ins
> lächerliche gezogen.
> Vielleicht sollte man mal über die Gastfunktion nachdenken.

Ich ziehe überhaupt nichts ins Lächerliche... Sei nicht so humorlos, 
kurze (und unübersichtliche) Programme in C haben eine lange Tradition. 
Siehe auch IOCC. :-)

[troll (Gast)]

Sorry for x-fach-Post. Ich bessere nochmal nach: Beispiel =~ s!#include 
<util/delay.h>!! Sollte funktionieren, Defaultwerte passen.

[Thorsten S. (whitejack)]

...viel interessanter ist ein Lauflicht mit 10 Duo LED  an 5 Portpins 
mit 5 Widerständen. Mit integriertem weichem Farbwechsel von rot zu grün 
während das Lauflicht läuft....

TS

[Peter D. (peda)]

Die Ausgabefunktion kann man auch drastisch vereinfachen.
Und Parameterübergabe macht man nicht in globalen Variablen, sondern 
direkt:

#include <avr/io.h>
#define F_CPU   1e6
#include <util/delay.h>

#define LEDL1B  (1<<PB0)
#define LEDL2D  (1<<PD7)
#define LEDL3D  (1<<PD0)
#define LEDR1D  (1<<PD5)
#define LEDR2B  (1<<PB7)
#define LEDR3B  (1<<PB6)

void ausgabe( uint8_t val )
{
  uint8_t vald = PORTD & ~(LEDL1B | LEDR2B | LEDR3B);
  uint8_t valb = PORTB & ~(LEDL2D | LEDL3D | LEDR1D);
  if( val & 1<<0 )
    valb |= LEDL1B;
  if( val & 1<<1 )
    vald |= LEDL2D;
  if( val & 1<<2 )
    vald |= LEDL3D;
  if( val & 1<<3 )
    vald |= LEDR1D;
  if( val & 1<<4 )
    valb |= LEDR2B;
  if( val & 1<<5 )
    valb |= LEDR3B;
  PORTD = vald;
  PORTB = valb;
}

int main()
{
  DDRB = LEDL1B | LEDR2B | LEDR3B;
  DDRD = LEDL2D | LEDL3D | LEDR1D;
  
  for(;;){
    uint8_t val = 1;
      ausgabe( 0 );
    do{
      _delay_ms(1000);
      ausgabe( val );
      val <<= 1;
    }while( val != 0b1000000 );
  }
}

Peter