Pin-Abfrage Probleme bei Atmega32-Verbindung

[Jakob (Gast)]

Hallo, ich habe ein Problem.

Vor weg, was will ich machen ?

Habe ein Atmega32 der mit ein Display als Codeschloss funktioniert oder 
andere Funktionen. Angeschlossen ist eine 3x4 Tastatur. Und einen 
LED-Cube mit einem Atmega32.

Nun möchte ich dass, wenn ich auf meinen "Codeschloss" eine Auswahl 
treffe das der Led-Cube das Muster abspielt....Was habe ich gemacht ? 
Freien Pin vom "Codeschloss" Atmega32 mit dem Led_Cube verbunden (PD5). 
Auch GND Verbunden.

Wenn ich nun wenn ich auf dem CS auswähle das der Cube Starten soll 
passiert nix, die 5V High Signal liegt aber an...

Ich denke mal das ich einfach iwo ein Denk-Fehler drinnen habe.




Code :

in der Main:

char test=PIND;

for(int e=0;e<10000;e++)
    {
      if(test==0b00100000)
      {
        e=10000;
      }
      _delay_ms(4);
    }

so habe auch noch eine Tastatur Routine gestet aber auch mit wenig 
Erfolg :

void scan_key () {
  
  
unsigned  char key1,key2; 
  key1 = PORTD;
  key1 ^= 0b11011111;
  key1 &= 0b11011111;
  if (key1 != 0){
  _delay_ms(4);
    key2 = PORTD;
    key2 ^=0b11011111;
    key2 &=0b11011111;
    if ((key1 == key2) & (key1 != lastkey)) {
      lastkey = key1;
      if (keybuf == 0){
        keybuf = key1; 
    }
  }  
  }
  else lastkey = 0;  
  }
/*---------------------------------------------------------------------------
 readkey(): Liefert den gelesenen Tastaturwert zurück                             
---------------------------------------------------------------------------*/
char readkey()
{
  char a;
  a = keybuf;
  if(a != 0) 
  {
  keybuf = 0;
  return(a);
  } 
};

in der Main:

for(int e=0;e<10000;e++)
    {
      if(readkey()==1)
      {
        e=10000;
      }
      _delay_ms(4);
    }

''Wenn der die Taste gedrückt wird soll er aus der Schleife 
rausspringen..(Nur zum Test wecken eine for Schleife)

[Karl H. (kbuchegg)]

char test=PIND;

for(int e=0;e<10000;e++)
    {
      if(test==0b00100000)
      {
        e=10000;
      }
      _delay_ms(4);
    }

kompliziert geschrieben.

Das hier

      if(test==0b00100000)

willst du mit Sicherheit nicht. Denn das bedeutet, dass an diesem Port 
ALLE Bits genau wie vorgesehen stehen müssen. Nicht nur das eine Bit, an 
dem dein Taster hängt (der offenbar an PD5 hängt).

Zum anderen: wie soll denn test innerhalb der Schleife jemals seinen 
Wert ändern können? Es kriegt vor der Schleife seinen Wert von PIND und 
auch wenn du 10 Millionen mal abfragst, dieser Wert ändert sich nicht.

Wenn schon dann

    for(int e=0;e<10000;e++)
    {
      if( PIND & ( 1 <<PD5) )
      {
        e=10000;
      }
      _delay_ms(4);
    }

so wie man eben Taster abfragt und wie man das in den Vorübungen zur 
Programmierung mit LED und Taster zur Genüge gelernt hat, ehe man dann 
seinen ersten Cube baut.

[Karl H. (kbuchegg)]

void scan_key () {
  
  
unsigned  char key1,key2; 
  key1 = PORTD;

PIND


PORTD ist für die Ausgabe. PIND für die Eingabe.

[Karl H. (kbuchegg)]

> Ich denke mal das ich einfach iwo ein Denk-Fehler drinnen habe.

Ich denke mal du solltest deinen Cube beiseite legen und erst mal mit 
den Grundlagen, wie sie in jedem Tutorial beschrieben sind, anfangen.
Portpins sicher und zuverlässig programmieren zu können, sowohl als 
Output Pin als auch als Input Pin, ist das kleine Einmal Eins in der 
µC-Programmierung. Solange das nicht 100% sitzt, hat es keinen Sinn, da 
irgendwas komplexeres zu bearbeiten.

AVR-GCC-Tutorial

[Jakob (Gast)]

SEHR VIELEN DANK =)

Karl Heinz ! Funktioniert jetzt ^^ Mein Projekt kann weiter gehen

[Jakob (Gast)]

Den Cube habe ich schon lange fertig und auch alles verstanden !

Mit dem Codeschloss ( Tastatur + Atmega32 + Display ) bin ich dran, um 
mir das etwas unterhaltsamer zumachen verbinde ich diese !

Ich habe jetzt nach über ein halben Jahr wieder angefangen und muss mich 
erstmal wieder rein arbeiten ;)

[Karl H. (kbuchegg)]

Jakob schrieb:
> Den Cube habe ich schon lange fertig und auch alles verstanden !

Mit Verlaub: das glaub ich nicht.
(also das du alles verstanden hast, das er funktioniert glaub ich dir, 
wenn du das Programm kopiert hast)

[Jakob (Gast)]

Nein..habe ich nicht..ist ja auch egal...bei mir harkt es noch mit dem 
Port/Pin abfragen. Der Cube ist ne andere Geschichte...

[ConvertsQuestionsToAnswers (Gast)]

Erlaube mir den Rat, Deine Kenntnisse, soweit Du sie früher erworben 
hast, noch einmal aufzufrischen.

Einer das "ALLES" verstanden hat schreibt auch nicht:

    for(int e=0;e<10000;e++)
    {
      if( PIND & ( 1 <<PD5) )
      {
        e=10000;
      }
      _delay_ms(4);
    }

sondern:

    for(int e = 0;(( e < 10000) && (PIND & (1 << PD5))); e++)
      _delay_ms(4);

wenn er überhaupt Busy-Waiting verwendet.

Glaube ich jedenfalls.