Drehimpulsgeber und Taster

[Marc H. (bauerpilot)]

Ich stehe vor folgendem Problem:
Drehimpulsgeber mit Taster an einem ATMega8. Die Impulse des 
Drehimpulsgebers sollen nur gezählt werden, wenn der Taster gedrückt 
ist.
Ich habe verschiedene Codes von PD zusammengesetzt und alles klappt 
bestens nur das mit Taster als Bedingung für Drehimpulsgeber nicht.
Hier die wesentlichen Codeanteile:

 
...
#define ENC_2_A      (PIND & 1<<PD2)
#define ENC_2_B      (PIND & 1<<PD3)

volatile int8_t  enc_delta_2;
static int8_t last_2;
void encode_init( void )
   {
    int8_t neu_2;
    neu_2 = 0;
    if( ENC_2_A ) neu_2 = 3;
    if( ENC_2_B ) neu_2 ^= 1;                           
    last_2 = neu_2;                        
    enc_delta_2 = 0;
  TCCR1B = (1<<WGM12) | (1<<CS11) | (1<<CS10);     // CTC, XTAL / 64
  OCR1A = (uint16_t)(F_CPU / 64.0 * 1e-3 - 0.5);   // 1ms
  TIMSK |= 1<<OCIE1A;
    }
volatile uint8_t key_state;
volatile uint8_t key_press;

//ISR Dregeber auslesen
ISR( TIMER1_COMPA_vect )             // 1ms for manual movement
{
  int8_t neu_2;
  int8_t diff_2;
  neu_2 = 0;
  if( ENC_2_A ) neu_2 = 3;
  if( ENC_2_B ) neu_2 ^= 1;                           
  diff_2 = last_2 - neu_2;                 
  if( diff_2 & 1 )
    {                         
    last_2 = neu_2;
    enc_delta_2 += (diff_2 & 2) - 1;
    }
}
//ISR Taster auslesen
ISR( TIMER0_OVF_vect )                    // every 10ms
{
...
}

int8_t encode_read_2( void )         // read two step encoders
{
  int8_t val; 
  cli();
  val = enc_delta_2;
  enc_delta_2 = val & 1;
  sei();
  return val >> 1;
}

int main (void){
while(1) {    
    if( get_key_press( 1<<KEY2 )) flag = 0x01;
    val_2 += encode_read_2();
}}

Im Teil "main" ist zum einen die Abfrage ob "KEY2" gedrückt worden ist 
und in "val_2" werden die Zählschritte des Encoders gespeichert.
Ein einfaches:

    if( get_key_press( 1<<KEY2 ))  val_2 += encode_read_2();

funktioniert leider nicht. Das führt nur dazu, dass wenn nach dem drehen 
der Taster betätigt wird die letzte Drehung übernommen wird. "val_2" 
soll sich aber nur ändern, wenn der Taster gedrückt wird. Irgendwo muss 
die if-Bedingung hin, aber wo?
Das ganze ist eine Art Sicherheitsschaltung nach dem Motto: Die Werte 
können nur bei gedrücktem Taster geändert werden.
Bedanke mich für jede Antwort.

[Falk B. (falk)]

while(1) {    
    if( get_key_press( 1<<KEY2 )) {
        val_2 += encode_read_2();
    } else {
        encode_read_2();
    } 
}

MFG
Falk

[Karl H. (kbuchegg)]

> Die Werte
> können nur bei gedrücktem Taster geändert werden.

Also: Solange der Taster gedrückt IST, bewirkt Drehen am Encoder etwas?

"Solange gedrückt IST" übersetzt sich zu einem ganz banalen


   if ( !( KEY_PORT & ( 1<<KEY2 ) ) )
     val_2 += encode_read_2();
   else
     encode_read_2();


Dazu brauchst du die Tastenauswertung und Entprellung mit der Timer 
Lösung gar nicht. Die kommt nur dann ins Spiel, wenn du einen einzelnen 
Tastendruck auch nur einmal erkennen willst.

[Marc H. (bauerpilot)]

Vielen Dank für die schnellen Antworten.
Leider haben sie nicht den gewünschten Erfolg, obwohl mir die Logik mit 
der "else" Anweisung schon klar ist. Einfach die Funktion aufrufen, die 
Rückgabe aber nicht weiter verarbeiten.
Die Entprellung habe ich nur drin gehabt, weil noch drei weitere Taster 
ausgewertet werden sollen und da wirklich nur ein einzelner Tastendruck 
dedektiert werden soll, weil zwischen zwei Werten umgeschaltet werden 
soll.
Insgesamt sollen 8 Drehencoder und vier Taster angeschlossen werden.
Die Daten werden dann über I2C weitergemeldet.
Auch wenn der Fehler noch nicht gefunden wurde, vielen Dank für die 
schnellen Antworten.
Ich habe mal den gesamten Code angehängt, vielleicht ist der Fehler ja 
auch an einer ganz anderen Stelle, ich habe aber keine Ahnung wo.