Entprellungsfunktion arbeitet nicht zuverlässig

[Johannes (menschenskind)]

Hallo

Ich habe diese Entprellungsroutine 
https://www.mikrocontroller.net/articles/Entprellung#Flankenerkennung 
für einen "Active HIGH"-Taster umgebaut:

#include <avr/io.h>

#define BUTTONPORT PIND
#define BUTTONPIN PD2
 
char debounce(void)
{
    static unsigned char state;
    char rw = 0;
 
    if(state == 0 && (BUTTONPORT & (1<<BUTTONPIN)))   //Taster wird gedrueckt (steigende Flanke)
    {
        state = 1;
        rw = 1;
    }
    else if (state == 1 && (BUTTONPORT & (1<<BUTTONPIN)))   //Taster wird gehalten
    {
         state = 2;
         rw = 0;
    }
    else if (state == 2 && !(BUTTONPORT & (1<<BUTTONPIN)))   //Taster wird losgelassen (fallende Flanke)
    {
        state = 3;
        rw = 0;
    }
    else if (state == 3 && !(BUTTONPORT & (1<<BUTTONPIN)))   //Taster losgelassen
    {
        state = 0;
        rw = 0;
    }
 
    return rw;
}

Über

while(1){
    
    if(debounce() == 1)  LED_EVENT(LED2,2);
}

lasse ich eine LED blinken, wenn "rw" 1 ist.
Leider klappt das aber irgendwie nur bei jedem 2. Mal (selten auch erst 
beim 3. Drücken)
Woran kann das liegen?

Der µC läuft mit 16 MHz und nebenbei (in der While-Schleife) wird noch 
ein NeoPixelRing angesteuert.

Danke
Hannes

[Peter D. (peda)]

Johannes H. schrieb:
> Ich habe diese Entprellungsroutine
> https://www.mikrocontroller.net/articles/Entprellung#Flankenerkennung

Drüber steht ja "Flankenerkennung", da wird also gar nichts entprellt.

Weiter steht "Die Entprellung geschieht dabei durch die ganze Laufzeit 
des Programms.", was ganz großer Bockmist ist.

[Joe F. (easylife)]

Du könntest statt

    else if (state == 1 && (BUTTONPORT & (1<<BUTTONPIN)))   //Taster wird gehalten
    {
         state = 2;
         rw = 0;
    }

folgendes machen:

    else if (state == 1 && (BUTTONPORT & (1<<BUTTONPIN)))   //Taster wird gehalten
    {
         state = 2;
         rw = 1;
    }

Dadurch wird dir zumindest beim Halten des Tasters auch eine 1 
zurückgeliefert.
Was macht denn deine Funktion "LED_EVENT(LED2,2)" genau?
Du müsstest die LED ja auch irgendwo wieder ausschalten...?

Insgesamt ist diese Entprell-Funktion nicht so super.
Sie geht davon aus, dass der Taster beim Prellen bei jedem 
Funktionsaufruf seinen Zustand geändert hat.
Wenn er "langsamer" prellt (also 2 Funktionsaufrufe den gleichen Wert 
liefert) steigt diese Funktion schon aus.

Ich würde es eher über ein Timeout machen.
Also nach der Flankenerkennung (in beiden Richtungen) eine gewisse Zeit 
lang (ein paar ms) den Taster nicht abfragen.
Das kann man z.B. über einen Counter in debounce() realisieren.

Also so:

#define DEBOUNCE_DELAY (100)    // <--- entsprechend anpassen

char debounce(void)
{
    static unsigned char state = 0;
    static unsigned int  delay = 0;
    
    if (delay > 0)
    {
      delay--;
    }
    else
    {
      if(state == 0 && (BUTTONPORT & (1<<BUTTONPIN)))   //Taster wird gedrueckt (steigende Flanke)
      {
          state = 1;
          delay = DEBOUNCE_DELAY;
      }
      else if (state == 1 && !(BUTTONPORT & (1<<BUTTONPIN)))   //Taster losgelassen
      {
          state = 0;
          delay = DEBOUNCE_DELAY;
      }
    }

    return state;
}

[Johannes (menschenskind)]

@Peter
Ach so! Hm, ja diesen einen Satz hatte ich dann in der Tat nicht richtig 
verstanden.

@Joe
Wie Peter schon sagte, ist das also keine Entprellfunktion. Die stand 
nur in der Liste ganz oben, deswegen hab ich die zuerst ausprobiert.
Die Timeoutvariante ist auch in dem Tutorial enthalten, wenn auch in 
anderer Form.
"LED_EVENT" schaltet die LED kurz ein und dann wieder aus.

[Johannes (menschenskind)]

Hi Peter,

Ich habe Deinen Code (Komfortroutine) jetzt integriert.
Woran ich aber gerade grüble ist, dass das "get_key_long" bereits nach
sehr kurzem "Lang-Drücken" des Tasters registriert wird.

Ich habe in den Defines diesen Wert von 50 auf 100 geändert "#define
REPEAT_START    100 "
Da sich das Verhalten aber nicht ändert, nehme ich also an, dass ich an
der falschen Stelle gedreht habe.
Ebenso habe ich den Vorteilquotienten von 1024 auf 1600 gesetzt da ich 
mit einem höheren Takt als in Deinem Beispiel arbeite.

Mein Ziel ist es, ein Lang-Drücken von etwa 2 Sekunden zu detektieren.

[Route_66 (Gast)]

Johannes H. schrieb:
> Ebenso habe ich den Vorteilquotienten von 1024 auf 1600 gesetzt da ich
> mit einem höheren Takt als in Deinem Beispiel arbeite.

Du kannst doch nur Vorteiler wählen, die in Deiner Hardware auch 
vorhanden sind!?