Beispielcode (hier aus dem Artikel) für Drehgeber auf STK500 testen

[Max B. (theeye)]

Hallo!

Ich versuche gerade einen Drehencoder zum laufen zu bringen. Basis ist 
der Beispielcode hier aus dem Artikel Drehgeber von Peter Dannegger. 
Klappt leider noch nicht.

Ich nutze ein STK500 und der Drehgeber steckt auf dem Breadboard. Es 
handelt sich um den STEC12E08 (Datenblatt: 
http://www.reichelt.de/index.html?&ACTION=7&LA=3&OPEN=0&INDEX=0&FILENAME=F100%252F402097STEC12E08.PDF).

Hier mein aktueller Code:

/************************************************************************/
/*                                                                      */
/*                      Reading rotary encoder                      */
/*                      one, two and four step encoders supported   */
/*                                                                      */
/*              Author: Peter Dannegger                                 */
/*                                                                      */
/************************************************************************/
#include <avr/io.h>
#include <avr/interrupt.h>
#include <inttypes.h>

                // target: ATmega8
//------------------------------------------------------------------------

//#define XTAL        8e6         // 8MHz

#define PHASE_A     (PIND & 1<<PD0)
#define PHASE_B     (PIND & 1<<PD1)

#define LEDS_DDR    DDRB
#define LEDS        PORTB           // LEDs against VCC



volatile int8_t enc_delta;          // -128 ... 127
static int8_t last;


void encode_init( void )
{
  int8_t new;

  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<<CS02)|(1<<CS00);         // divide by 1024
  TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 10e-3 + 0.5);  // preload for 10ms
  TIMSK |= 1<<TOIE0;                   // enable timer interrupt
}


ISR( TIMER0_COMP_vect )             // 1ms for manual movement
{
  int new, diff;

  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 (+/-)
  }
}


int8_t encode_read1( void )         // read single step encoders
{
  int8_t val;

  cli();
  val = enc_delta;
  enc_delta = 0;
  sei();
  return val;                   // counts since last call
}


int main( void )
{
  int32_t val = 0;

  LEDS_DDR = 0xFF;
  encode_init();
  sei();

  for(;;){
    val += encode_read1();          // read a single step encoder
    LEDS = val;
  }
}

Über tatkräftige Unterstützung würde ich mich sehr freuen! :)

Gruß Max

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

Max B. schrieb:
> TIMSK |= 1<<TOIE0;                   // enable timer interrupt
> }
>
> ISR( TIMER0_COMP_vect )             // 1ms for manual movement
> {

Das passt nicht zusammen. Du gibst den Overflow Interrupt frei, 
behandelst aber den COMP Interrupt.

Entweder nimmst du den CTC Mode (so macht PeDa das m.W.) oder du lässt 
den Timer einfach überlaufen und taktest ihn so, das du so ca. 1-10ms 
Zyklus hast.
In jedem Fall muss aber die Interrupt Freigabe zur ISR passen.

[Max B. (theeye)]

Ah okay, danke! Mache mich noch einmal dran :)

Gruß Max