AVR-n00b will Passworteingabe machen

[Walternativ (Gast)]

Hallo,
ich habe einen atmega8 auf nem alten STK500 und wollte eine Art 
Passworteingabe machen. Die Knöpfe 0-2 können gedrückt werden und es 
wird jeweils ein Wert in einen Puffer geschrieben. Dieser wird mit einem 
anderen verglichen und wenn sie übereinstimmen geht eine LED an.

Ich bin nicht sehr erfahren in der AVR-Programmierung und habe keine 
Ahnung wie ich eine Debug-Ausgabe realisieren soll. Ich glaube die Werte 
werden nicht richtig in den Puffer geschrieben. Die compare funktion ist 
auf jeden Fall richtig, denn wenn ich puffer und passwort gleichsetze 
leuchtet die LED. Auch wenn ich die Länge des Passworts auf 1 setze geht 
es. Ihr Profis erkennt sicher auf dem ersten Blick meinen Fehler:

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

#define INPUT_MASK 0b00000111 //3 knöpfe
#define LENGTH 2

uint8_t getInput(uint8_t pin, uint8_t mask) {
  return ~(pin & mask);
}

uint8_t it = 0;

void add(uint8_t* buf, uint8_t num) {
  buf[it] = 1;
  if (it < LENGTH - 1)
    it++;
  _delay_ms(500);
}

int compareArrays(uint8_t* buf1, uint8_t* buf2) {
  for (uint8_t i; i < LENGTH; i++) {
    if (buf1[i] != buf2[i])
      return 0;
  }
  return 1;
}

int main(void) {

  DDRB = 0b11111111; //out
  DDRD = 0; //in
  PORTB = 0xff;

  uint8_t buffer[LENGTH] = { 0, 0 };
  uint8_t pass[LENGTH] = { 1, 1 };

  while (1) {
          uint8_t input = getInput(PIND, INPUT_MASK);

    if (input & 0b00000001) //wenn knopf 1 gedrückt
      add(buffer, 1);

    if (compareArrays(buffer, pass))
      PORTB = ~0b00001000;
    else
      PORTB = 0xff;
  }
  return 0;
}

[Paul (Gast)]

Stichwort: Entprellung

[Walternativ (Gast)]

Paul schrieb:
> Stichwort: Entprellung

Das heißt doch nur, dass das einmalige Drücken als mehrfaches 
wahrgenommen wird, was aber nicht die Lösung zu meinem Problem sein 
kann, oder?

[Max B. (theeye)]

Walternativ schrieb:
> dass das einmalige Drücken als mehrfaches
> wahrgenommen wird

Wenn du eine PIN Eingabe realisieren möchtest, ist das doch ganz 
entscheidend!

Beispiel
PIN lautet 012 (Taster)

Also d®ückst du die Taster 0, 1 und 2. Ergebnisse:

Ohne entprellen z.B.: 0000011111222

Mit entprellen: 012

Gruß Max

[Paul (Gast)]

Hm... da ist ja ein recht großes Delay drin.

Walternativ schrieb:
> return ~(pin & mask);

Warum eigentlich hier das nicht?

[Max H. (hartl192)]

Wenn ein Knopf gedrückt wird wie add(); aufgerufen. add enthält ein 
_delay_ms(500); die Taste wird also entprellt.

[Walternativ (Gast)]

Paul schrieb:
> Hm... da ist ja ein recht großes Delay drin.
>
> Walternativ schrieb:
>> return ~(pin & mask);
>
> Warum eigentlich hier das nicht?

Weil das STK500 ganz komisch ist.

Max B. schrieb:
> Wenn du eine PIN Eingabe realisieren möchtest, ist das doch ganz
> entscheidend!
>
> Beispiel
> PIN lautet 012 (Taster)
>
> Also d®ückst du die Taster 0, 1 und 2. Ergebnisse:
>
> Ohne entprellen z.B.: 0000011111222
>
> Mit entprellen: 012
>
> Gruß Max

Das ist schon richtig, aber es erklärt nicht, warum die LED überhaupt 
nicht angeht. Theoretisch müsste ja dann schon beim ersten Tastendruck 
der Puffer auf {1,1} stehen und damit dem Passwort entsprechen.

[Drrt (Gast)]

Hä

[jetzt (Gast)]

Dann gib doch den jeweils gelesenen pin ueber einen anderen aus und 
beobachte mit einem Oszilloskop

[Walternativ (Gast)]

Ein n00b hat kein Oszilloskop

[spontan (Gast)]

Manche Tasten prellen nicht nur beim Drücken, sondern auch beim 
Loslassen,
kann das Dein delay und der Rest der Software ausgleichen?

[Walternativ (Gast)]

Ich habe in der compare Funktion i uninitialisiert gelassen. Es geht 
jetzt.

Der Thread ist hiermit offiziell geschlossen

[jibi (Gast)]

Hat dich der Compiler nicht mit einem Warnhinweis darauf aufmerksam 
gemacht?
Die zu lesen und zu verstehen ist nämlich Gold wert.

Gruß Jonas