Kleines Testprogramm tut nicht was es soll (C)

[Fritz B. (kleinfritzchen)]

Ich hab ein kleines Programm geschrieben was eigenzlich auf einen 
Tstendruck einen Zähler hochzählen soll und das Ergebnis auf 4 
Leuchtdioden ausgeben soll.
Irgendwie hab ich das Gefühl das das Programm nur ein mal durchlaufen 
wird.
Ich hab zum lernen ein AVR-C Tutorial von 2007. Ich hoffe das passt noch 
halbwegs da ich Anfänger bin und noch nicht die Ahnung hab.
Hier mal das Programm:

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

int main (void) {
  DDRD= 0x0f; //Port D die unteren 4 Bit als Ausgang
  DDRB= 0b00000000;
  PORTB = 0x02;// pull Up Wiederstand für ALLE PinS  aktiviert
  
  
  
    uint8_t tg=0;
    uint8_t taste=0;
    uint8_t dioden = 0;
  PORTD=0x0f;
  _delay_ms(500);
  PORTD=0x00;
  _delay_ms(100);
    
  while(1){
    
    if (!(PINB & (1<<PB1))){
      dioden++;
      PORTD=dioden;
      _delay_ms(100);
      loop_until_bit_is_set(PINC,PC3);
    }
    PORTD ^= (1<<PD3);
    _delay_ms(200);
    PORTD ^= (1<<PD3);
    
    //dioden &= 0x0f;
    //PORTD = ++dioden;
    //PORTD = taste;
    return 0;
  }
    
}

Ich hoffe  es kann mir mal einer erklähren warum die Diode an PD3 nicht 
laufend blinkt.

MfG Fritz

[Captain Obvious (Gast)]

return 0 steht in der schleife

[Christian Karle (Gast)]

Um die LED anzuschalten musst Du schreiben:

PORTD|= (1<<PD3);

Und um sie auszuschalten:

PORTD&= ~(1<<PD3);

Deine Befehle Löschen weder ein Bit, noch setzten Sie eines.

[Captain Obvious (Gast)]

^= ist ein XOR, d.h. toggeln (ist schon alles wies sein soll)
Nach dem zweiten ^= solltes man ein delay einfügen, weil du sonst das 
Blinken nicht siehst, weil es zu schnell ist.

[Christian Karle (Gast)]

Die Delays vor der While-Schleife sind unnötig. Um ein Blinken 
wahrnehmen zu können fehlt ein Delay nach

PORTD &= ~(1<<PD3);

_delay_ms(200);

[Christian (Gast)]

Was tut denn diese ominöse Funktion?
loop_until_bit_is_set()
Hängt er da vielleicht in dem loop?
Ansonsten halte ich es ebenfalls für Unsinn, ein Return 0 in die 
While(1)-Schleife einzubauen ;-)

[Rolf M. (rmagnus)]

Christian schrieb:
> Was tut denn diese ominöse Funktion?
> loop_until_bit_is_set()

Der Name könnte doch eindeutiger nicht sein. Ist aber ein Makro:
http://www.atmel.com/webdoc/AVRLibcReferenceManual/group__avr__sfr_1gaaf6857fa882da35f8685e2001e5c3bbe.html

> Ansonsten halte ich es ebenfalls für Unsinn, ein Return 0 in die
> While(1)-Schleife einzubauen ;-)

Was auch relativ gut dieses Problem erklärt:

Fritz Bie schrieb:
> Irgendwie hab ich das Gefühl das das Programm nur ein mal durchlaufen
> wird.

[Fritz B. (kleinfritzchen)]

Danke das war auch der Fehler mit dem Return in der Schleife!
Die Funktion loop_until_bit_is_set() dient hier eigentlich nur um ein 
sehr einfaches entprellen zu ermöglichen. Ist zwar unschön aber tuts für 
den Anfang.
Das Blinken vor der While Schleife dient nur dazu die Kontakte auf dem 
Steckbrett zu überprüfen. Und die Zeitschleifen in der While Schleife 
sind auch nur zum verlangsamen da.
Ich hab schon mal einen Einführungskurs in Assembler auf der 
Folkshochschule gemacht von da her kenn ich wenigstens ein bischen aus.
Danke für die schnelle Hilfe!!!


Mfg Fritz

[Christian (Gast)]

Rolf Magnus schrieb:
> Christian schrieb:
>> Was tut denn diese ominöse Funktion?
>> loop_until_bit_is_set()
>
> Der Name könnte doch eindeutiger nicht sein. Ist aber ein Makro:
> 
http://www.atmel.com/webdoc/AVRLibcReferenceManual/group__avr__sfr_1gaaf6857fa882da35f8685e2001e5c3bbe.html

Und ich dachte es wäre eine selbst-gefrickelte Funktion, die uns 
vorenthalten wird ;-)

[NSA (Gast)]

F_CPU im makefile definieren (oder wenigstens vorm einbinden der 
delay.h). Sollte eigentlich ne Warnung verursachen.

Ist aber wahrscheinlich nicht Ursache des Problems.

[Karl H. (kbuchegg)]

weiters

    if (!(PINB & (1<<PB1))){      <-------------- PIN B
      dioden++;
      PORTD=dioden;
      _delay_ms(100);
      loop_until_bit_is_set(PINC,PC3); <--------- PIN C
    }

du solltest dich schon mit dir selbst darüber einig sein, an welchem Pin 
nun der Taster hängt