Frage zu Artikel Drehgeberauswertung

[Ralf (Gast)]

Hallo,
 ich habe eine Frage zu dem folgenden Artikel über die Auswertung von 
Drehgebern: http://www.mikrocontroller.net/articles/Drehgeber

In der Erklärung des soliden Codes steht der Satz “…allerdings handelt 
man sich die Unschönheit ein, dass man im entsprechenden Makro auf eine 
interne Variable der Auswertefunktion zugreift. (Als „saubere“ 
Alternative müsste man den Code so umschreiben, dass das Erfassen der 
Werte und die Auswertung der gewünschten Spur in separate 
inline-Funktionen getrennt werden.)“

Wurde der Beispielcode nachträglich noch angepasst, oder sehe ich das 
Problem einfach nur nicht? Es wird doch ausschließlich in der 
Auswertefunktion auf die Variable zugegriffen.

ISR( TIMER0_COMP_vect )             // 1ms for manual movement
{
  int8_t new, diff;
  uint8_t encoderpin = PINA;
        
  if(encoderpin & 1<<PA1)
    new = 3;
  if(encoderpin & 1<<PA3)
    new ^= 1;    

  // …
}

[guest (Gast)]

Ralf schrieb:
> Wurde der Beispielcode nachträglich noch angepasst, oder sehe ich das
> Problem einfach nur nicht? Es wird doch ausschließlich in der
> Auswertefunktion auf die Variable zugegriffen.
>
> ISR( TIMER0_COMP_vect )             // 1ms for manual movement
> {
>   int8_t new, diff;
>   uint8_t encoderpin = PINA;
>
>   if(encoderpin & 1<<PA1)
>     new = 3;
>   if(encoderpin & 1<<PA3)
>     new ^= 1;
>
>   // …
> }

Tja, wenn Du aus dem Originalcode die fragllichen Makros 
wegrationalisierst, kannst Du das Problem natürlich nicht mehr sehen.
Gemeint sind hier die Makros PHASE_A und PHASE_B. Die funktionieren halt 
nur, wenn es in den Funktionen, in denen sie benutzt werden auch eine 
lokale Variable encoderpin gibt.

#define PHASE_A     (encoderpin & 1<<PA1)
#define PHASE_B     (encoderpin & 1<<PA3)
#define PIN_ENCODER PINA

void encode_init( void )
{
  int8_t new;
  uint8_t encoderpin = PIN_ENCODER;

  new = 0;
  if( PHASE_A )
    new = 3;
  if( PHASE_B )
    new ^= 1;                   // convert gray to binary
  last = new;                   // power on state
  enc_delta = 0;
  TCCR0 = 1<<WGM01^1<<CS01^1<<CS00;     // CTC, XTAL / 64
  OCR0 = (uint8_t)(XTAL / 64.0 * 1e-3 - 0.5);   // 1ms
  TIMSK |= 1<<OCIE0;
}
 
 
ISR( TIMER0_COMP_vect )             // 1ms for manual movement
{
  int8_t new, diff;
  uint8_t encoderpin = PIN_ENCODER;

  new = 0;
  if( PHASE_A )
    new = 3;
  if( PHASE_B )
    new ^= 1;                   // convert gray to binary
  diff = last - new;                // difference last - new
  if( diff & 1 ){               // bit 0 = value (1)
    last = new;                 // store new as next last
    enc_delta += (diff & 2) - 1;        // bit 1 = direction (+/-)
  }
}