Drehgeber Routine Peter Dannegger

[Marcel (Gast)]

Hallo,

ich versuche gerade zu verstehen wie die Routine für den Drehgeber von 
Peter Dannegger funktioniert.
Könnt ihr mir sagen warum meine Version der ISR nicht funktioniert?

ISR (TIMER0_COMPA_vect)                   // 1ms for manual movement
{
  //int8_t neu;
  int8_t uc_enc_code;
  int8_t diff;
  
  //neu = 0;
  uc_enc_code = 0x00;            // Variable fuer den Code mit 0 initialisieren
  
  //if (ENC_CHA)            
    //neu = 3;
  if (ENC_CHA)              // High an Channel A?
    uc_enc_code = 1;          // Bit 0 setzen -> 00000001   
  
  //if (ENC_CHB)
    //neu ^= 1;                    // convert gray to binary
  if (ENC_CHB)              // High an Channel B?
    uc_enc_code != 2;          // Bit 1 setzen -> 0000001X
    
  if (uc_enc_code == 0x02)        // Wenn 00000010 (2)...
    uc_enc_code = 0x03;          // dann 00000011 (3)
  
  if (uc_enc_code == 0x03)        // Wenn 00000011 (3)...
    uc_enc_code = 0x02;          // dann 00000010 (2)
  
  //diff = last - neu;                    // difference last - neu
  diff = uc_enc_code_old - uc_enc_code;   
  
  if (diff & 1)               // bit 0 = value (1)
  {                           
    //last = neu;                        // store neu as next last
    uc_enc_code_old = uc_enc_code;    // Neuen Code merken
    enc_delta += (diff & 2) - 1;       // bit 1 = direction (+/-)
  }
}

[Peter D. (peda)]

Marcel schrieb:
> Könnt ihr mir sagen warum meine Version der ISR nicht funktioniert?

Weil Du den Code geändert hast, ohne zu wissen, warum.
Auch sollte der Compiler ne Warnung werfen (nutzloser Code), weil Du 
einen Vergleich machst und das Ergebnis wegwirfst.

[Karl M. (Gast)]

Hallo,

ich verstehe nicht, warum man funktionierenden Code abändern muss...

Zu deiner Frage, frage Dich bitte was hier passiert.
Spiele also im Kopf oder auf dem Papier Computer:

 if (uc_enc_code == 0x02)        // Wenn 00000010 (2)...
    uc_enc_code = 0x03;          // dann 00000011 (3)
  
  if (uc_enc_code == 0x03)        // Wenn 00000011 (3)...
    uc_enc_code = 0x02;          // dann 00000010 (2)

Sind nach deiner Analyse diese Zeilen dann sinnvoll? Oder bewirken 
irgendetwas ?

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Das schwierigste bei PeDas Code ist es, die Copy&Paste Funktionen auf 
seinem Computer zu finden und sich dann für eine der drei 'enc_readX' 
Funktionen zu entscheiden.
Das Ganze ist übrigens auch deutlich kompakter als deine Variante.

[Marcel (Gast)]

Ou..., ich sehe es.

if (uc_enc_code == 0x02)      // Wenn 00000010 (2)...
  uc_enc_code = 0x03;        // dann 00000011 (3)
else if (uc_enc_code == 0x03)      // Wenn 00000011 (3)...
 uc_enc_code = 0x02;        // dann 00000010 (2)

Ich werde die Routine natürlich so lassen wie wie sie von Herrn 
Dannegger entwickelt wurde. Ich wollte nur verstehen wie das 
funktioniert. Ich glaube ich wäre nich auf diese Befehle gekommen:

if (ENC_CHA)            
  neu = 3;
if (ENC_CHB)
  neu ^= 1;

Ich arbeite mit der Arduino Umgebung. Diese hat leider keine Warnung 
ausgesprochen.

Vielen Dank.

[Arduino (Gast)]

IDE > Datei > Voreinstellungen > Compiler Warnungen ... "Alle"

Codefaltung und Zeilennummer kannste auch noch aktivieren.
Von allen anderen erstmal Finger weg.

[Peter D. (peda)]

Das else if sollte auch gehen, wenn davor das != durch |= ersetzt wird.

[Marcel (Gast)]

Klar, |= anstelle von !=.

Danke euch.