get_key_long() von Peter Dannegger

[Alex K. (derheld2k1)]

Hallo zusammen,

ich habe schon lange zu meinem Problem hier gesucht, aber dieses Thema 
zu den Entprell Routinen von P.Dannegger wurde noch nicht angesprochen.

Ich schaffe es nicht die Funktion get_key_long() zum Funktionieren zu 
überreden.

Meine Defines:

#define KEY_DDR         DDRA
#define KEY_PORT        PORTA
#define KEY_PIN         PINA
#define DIR_INC_KEY     0
#define DIR_DEC_KEY     1
#define SETAXIS_KEY     2
#define GOTO_REF_KEY    3
#define ENCODER_KEY     4
#define ALL_KEYS        (1<<DIR_INC_KEY | 1<<DIR_DEC_KEY | 1<<SETAXIS_KEY | 1<<GOTO_REF_KEY | 1<<ENCODER_KEY)
 
#define REPEAT_MASK     (1<<ENCODER_KEY | 1<<DIR_INC_KEY | 1<<DIR_DEC_KEY)       // repeat: key1, key2
#define REPEAT_START    50                        // after 500ms
#define REPEAT_NEXT     20                        // every 200ms

Es soll get_key_long() mit dem ENCODER_KEY klappen.
Dazu muss ich diesen ja erstmal in die REPEAT_MASK eintragen, soweit 
richtig?

Nun meine erste Frage: Wo stelle ich die Zeit ein nach der ein 
Tastendruck als "long" gewertet wird.

In main() sieht das ganze bei mir so aus:

int main() 
{
  // initzeug
  while(1) {
    if( get_key_short( 1<<ENCODER_KEY )) {
      foo();
    }
    
    if( get_key_long( 1<<ENCODER_KEY )) {
      bar();    
    }
  }
}

Habe ich hier irgendwo einen Denkfehler?
Ich kann den Taster solange drücken wie ich will, bar() wird nie 
aufgerufen.

Danke schonmal,
Alex

[Peter D. (peda)]

Keine Schnipsel und erst recht keinen Fantasiecode.
Sondern den getesteten compilierbaren Code als Anhang!


Peter

[Alex K. (derheld2k1)]

gekürzter Code mit reproduzierbarem Fehler im Anhang.

[Bernhard M. (boregard)]

Muss man nicht immer get_key_long() vor get_key_short() aufrufen?
Andersherum gehts glaube ich nicht.

[Uwe (de0508)]

Bernhard M. schrieb:
> Muss man nicht immer get_key_long() vor get_key_short() aufrufen?
> Andersherum gehts glaube ich nicht.

Glauben ist nicht wissen, was soll also der Beitrag.

Ich verwende die Abfragen nacheinande und ohne Probleme.

      // Auslesen der Encoder Zustände
      // alle weiteren Funktionen greifen auf diese Daten in |encoder| zu.

      ENCODER_DATA(direction)       = ENCODER_READ();
      ENCODER_DATA(key_short_press) = get_key_short( _BV(KEY_ENCODER_BIT) );  // beide Key-Ereignisse können nicht zusammen auftreten
      ENCODER_DATA(key_long_press)  = get_key_long ( _BV(KEY_ENCODER_BIT) );  // beide Key-Ereignisse können nicht zusammen auftreten

[Peter D. (peda)]

#ifdef ACTIVE_HIGH
  key_rpt = ~(key_rpt | (key_state & REPEAT_MASK));
#else
    key_rpt |= key_state & REPEAT_MASK;
#endif

Wer hat sich denn sowas ausgedacht?

Ob Taste gegen GND oder VCC, wird nur beim Einlesen des Input-Ports 
unterschieden, nirgendwo anders!


Peter

[Alex K. (derheld2k1)]

> Ob Taste gegen GND oder VCC, wird nur beim Einlesen des Input-Ports
> unterschieden, nirgendwo anders!

Also ist diese Stelle in der ISR korrekt?

#ifdef ACTIVE_HIGH
  i = key_state ^ KEY_PIN;                        // key changed ?
#else
  i = key_state ^ ~KEY_PIN;                       // key changed ?
#endif

Dann werde ich

#ifdef ACTIVE_HIGH
  key_rpt = ~(key_rpt | (key_state & REPEAT_MASK));
#else
    key_rpt |= key_state & REPEAT_MASK;
#endif

wieder durch

    key_rpt |= key_state & REPEAT_MASK;

ersetzen.

Obs dann klappt weiß ich erst wenn ich wieder im Keller sitze.

[Alex K. (derheld2k1)]

Hat geklappt, Danke !