Frage zum Debounce Makro für Anfänger

[al3ko (Gast)]

Hi,

Ich verwende das Debounce Makro für Anfänger von Peter D.

http://www.mikrocontroller.net/articles/Entprellung#Debounce-Makro_von_Peter_Dannegger


Gleichzeitig habe ich einen Taster (low-aktiv) an PD5 meines Atmega8 
angeschlossen und möchte einen Counter programmieren, der den jeweiligen 
Zählerstand auf dem LCD anzeigt. Ich verwende die LCD Routinen von 
dieser Seite zur Ansteuerung des LCDs.

Der ganze Code befindet sich hier:

#include <avr/io.h>
#include <util/delay.h>
#include "lcd-routines.h"
#include <avr/interrupt.h>
#include <stdlib.h>

/*
Makro for debouncing
*/
#define debounce( port, pin )                                         \
({                                                                    \
  static uint8_t flag = 0;     /* new variable on every macro usage */  \
  uint8_t i = 0;                                                      \
                                                                      \
  if( flag ){                  /* check for key release: */           \
    for(;;){                   /* loop ... */                         \
      if( !(port & 1<<pin) ){  /* ... until key pressed or ... */     \
        i = 0;                 /* 0 = bounce */                       \
        break;                                                        \
      }                                                               \
      _delay_us( 98 );         /* * 256 = 25ms */                     \
      if( --i == 0 ){          /* ... until key >25ms released */     \
        flag = 0;              /* clear press flag */                 \
        i = 0;                 /* 0 = key release debounced */        \
        break;                                                        \
      }                                                               \
    }                                                                 \
  }else{                       /* else check for key press: */        \
    for(;;){                   /* loop ... */                         \
      if( (port & 1<<pin) ){   /* ... until key released or ... */    \
        i = 0;                 /* 0 = bounce */                       \
        break;                                                        \
      }                                                               \
      _delay_us( 98 );         /* * 256 = 25ms */                     \
      if( --i == 0 ){          /* ... until key >25ms pressed */      \
        flag = 1;              /* set press flag */                   \
        i = 1;                 /* 1 = key press debounced */          \
        break;                                                        \
      }                                                               \
    }                                                                 \
  }                                                                   \
  i;                           /* return value of Macro */            \
})


int main(void)
{
  DDRD &= ~(1<<PD5);      // Input
  PORTD |= (1<<PD5);      // Internal Pull up activated
  DDRB |= (1<<PB1);
  TCCR1B |= (1<<CS10) | (1<<CS11) | (1<<CS12);
  lcd_init();
  char buffer[20];

  while(1)
  {
    if(debounce(PIND, PD5))
    {
      itoa(TCNT1,buffer,10);
      lcd_clear();
      lcd_string(buffer);
    }
    if(debounce(PIND, PD7))
    {
      // Unused in this example!
    }
  }
}

Grundsätzlich scheint es auch zu funktionieren. Allerdings wird ab und 
an eine Zahl übersprungen. Soll heißen, das Display gibt z.B. aus:
0,1,3,4,5,7

Weiß jemand, woran es liegt? Ich vermute ggf. ein Problem bei der 
Debounce Routine, bin mir aber nicht sicher.

Ich verwende AVR Studio 4 mit AVRisp mkII und Atmega8 uC.

Dankeschön.

Gruß

[Peter D. (peda)]

al3ko schrieb:
> Ich vermute ggf. ein Problem bei der
> Debounce Routine,

Nö.
Du benutzt Debounce nur zum Anzeigen, aber zum Zählen T1. Ergo kann sie 
das Zählen auch nicht entprellen.

Nimm ne Variable zum Zählen.


Peter

[al3ko (Gast)]

Peter Dannegger schrieb:
> al3ko schrieb:
>> Ich vermute ggf. ein Problem bei der
>> Debounce Routine,
>
> Nö.
> Du benutzt Debounce nur zum Anzeigen, aber zum Zählen T1. Ergo kann sie
> das Zählen auch nicht entprellen.
>
> Nimm ne Variable zum Zählen.
>
>
> Peter
Hi Peter,
ich habe dir und deiner Routine unrecht getan, entschuldige bitte. Mit 
der Variable läuft es wie geschmiert.

Okay, hier nun wie ich es verstehe:
Zum zählen (hardwareseitig) verwende ich T1 und da das Debouncing 
softwareseitig realisiert ist, werden an T1 auch die Prellungen 
berücksichtigt. Ergo wird das Prellen mitgezählt, allerdings nur die 
Entprellte Zahl auf dem LCD ausgegeben.

Was mich dabei aber wundert, ist, warum immer nur eine Zahl ausgelassen 
wird und nicht ab und an mal zwei oder mehrere Zahlen?

Danke

[Karl H. (kbuchegg)]

al3ko schrieb:

> Was mich dabei aber wundert, ist, warum immer nur eine Zahl ausgelassen
> wird und nicht ab und an mal zwei oder mehrere Zahlen?

Wenn du die nächsten paar Tage nichts vor hast, dann setz dich ein paar 
Stunden hin und hämmere auf den Taster ein. Ich würde mal schätzen, 
spätestens nächste Woche wirst du an deinem dann abgenutzten Taster von 
0 bis 5 Preller so ziemlich alles sehen.
Und wenn nicht, dann geht die Tortur für den Taster eben weiter :-) Es 
ist nur eine Frage der Zeit und der Anzahl Betätigungen.

[al3ko (Gast)]

Karl Heinz Buchegger schrieb:
> Wenn du die nächsten paar Tage nichts vor hast, dann setz dich ein paar
> Stunden hin und hämmere auf den Taster ein. Ich würde mal schätzen,
> spätestens nächste Woche wirst du an deinem dann abgenutzten Taster von
> 0 bis 5 Preller so ziemlich alles sehen.
> Und wenn nicht, dann geht die Tortur für den Taster eben weiter :-) Es
> ist nur eine Frage der Zeit und der Anzahl Betätigungen.

Super, vielen Dank. Auch wenn das Experiment sehr attraktiv klingt, so 
belasse ich es dabei und komme zu meiner Konklusion:
T1 ist auch nicht in Kombination mit Tastern zu bringen - genau so wenig 
wie Interruptroutinen (Int0 oder Int1 am Atmega8) :D

Schönen Gruß,
al3ko

[Karl H. (kbuchegg)]

al3ko schrieb:

> und komme zu meiner Konklusion:
> T1 ist auch nicht in Kombination mit Tastern zu bringen - genau so wenig
> wie Interruptroutinen (Int0 oder Int1 am Atmega8) :D

Dreh deine Konklusio um:
nackte Taster für etwas anderes zu verwenden, als sie per Polling 
abzufragen, ist nicht wirklich sinnvoll. Taster erfordern immer 
Mehraufwand, egal wie. Und wenn schon Mehraufwand, dann kann man sie 
auch gleich Pollen ... und sei es in der ISR eines Timers.