Drehimpulsgeber mit Rasterstellung bei 00/11 auswerten

Kann mir jemand den übersetzten C-Code von oben geben (AVR-ASM)?
Habe mit meinen Unkenntnissen in C versucht, das in ASM zu wandeln aber
ich komme auf keinen Nenner.
Kann leider nur ASM.

Danke im Voraus!

MfG
Andi

Hier, nur kopiert, nicht näher angeschaut....

a) Optimization for size

70:         new_state=PINE & (_BV(PINE4) | _BV(PINE3));
+000003A4:   91800000    LDS     R24,0x0000       Load direct from data
space
+000003A6:   7188        ANDI    R24,0x18         Logical AND with
immediate
+000003A7:   8389        STD     Y+1,R24          Store indirect with
displacement
71:         if ((new_state^last_cnt)==(_BV(PINE4) | _BV(PINE3)) )  //
we are either in main state 00 or 11
+000003A8:   8199        LDD     R25,Y+1          Load indirect with
displacement
+000003A9:   9180014C    LDS     R24,0x014C       Load direct from data
space
+000003AB:   2789        EOR     R24,R25          Exclusive OR
+000003AC:   3188        CPI     R24,0x18         Compare with
immediate
+000003AD:   F4A1        BRNE    PC+0x15          Branch if not equal
73:           if ((new_state ^ last_state)==_BV(PINE4))
+000003AE:   8189        LDD     R24,Y+1          Load indirect with
displacement
+000003AF:   9190014B    LDS     R25,0x014B       Load direct from data
space
+000003B1:   2789        EOR     R24,R25          Exclusive OR
+000003B2:   3180        CPI     R24,0x10         Compare with
immediate
+000003B3:   F431        BRNE    PC+0x07          Branch if not equal
74:             enc_delta+=1;
+000003B4:   9180014A    LDS     R24,0x014A       Load direct from data
space
+000003B6:   5F8F        SUBI    R24,0xFF         Subtract immediate
+000003B7:   9380014A    STS     0x014A,R24       Store direct to data
space
+000003B9:   C005        RJMP    PC+0x0006        Relative jump
76:             enc_delta-=1;
+000003BA:   9180014A    LDS     R24,0x014A       Load direct from data
space
+000003BC:   5081        SUBI    R24,0x01         Subtract immediate
+000003BD:   9380014A    STS     0x014A,R24       Store direct to data
space
77:           last_cnt=new_state;
+000003BF:   8189        LDD     R24,Y+1          Load indirect with
displacement
+000003C0:   9380014C    STS     0x014C,R24       Store direct to data
space
79:         last_state=new_state;
+000003C2:   8189        LDD     R24,Y+1          Load indirect with
displacement
+000003C3:   9380014B    STS     0x014B,R24       Store direct to data
space
+000003C5:   9621        ADIW    R28,0x01         Add immediate to
word

b) Optimization for speed
70:         new_state=PINE & (_BV(PINE4) | _BV(PINE3));
+000003A4:   91800000    LDS     R24,0x0000       Load direct from data
space
+000003A6:   7188        ANDI    R24,0x18         Logical AND with
immediate
+000003A7:   8389        STD     Y+1,R24          Store indirect with
displacement
71:         if ((new_state^last_cnt)==(_BV(PINE4) | _BV(PINE3)) )  //
we are either in main state 00 or 11
+000003A8:   8199        LDD     R25,Y+1          Load indirect with
displacement
+000003A9:   9180014C    LDS     R24,0x014C       Load direct from data
space
+000003AB:   2789        EOR     R24,R25          Exclusive OR
+000003AC:   3188        CPI     R24,0x18         Compare with
immediate
+000003AD:   F4A1        BRNE    PC+0x15          Branch if not equal
73:           if ((new_state ^ last_state)==_BV(PINE4))
+000003AE:   8189        LDD     R24,Y+1          Load indirect with
displacement
+000003AF:   9190014B    LDS     R25,0x014B       Load direct from data
space
+000003B1:   2789        EOR     R24,R25          Exclusive OR
+000003B2:   3180        CPI     R24,0x10         Compare with
immediate
+000003B3:   F431        BRNE    PC+0x07          Branch if not equal
74:             enc_delta+=1;
+000003B4:   9180014A    LDS     R24,0x014A       Load direct from data
space
+000003B6:   5F8F        SUBI    R24,0xFF         Subtract immediate
+000003B7:   9380014A    STS     0x014A,R24       Store direct to data
space
+000003B9:   C005        RJMP    PC+0x0006        Relative jump
76:             enc_delta-=1;
+000003BA:   9180014A    LDS     R24,0x014A       Load direct from data
space
+000003BC:   5081        SUBI    R24,0x01         Subtract immediate
+000003BD:   9380014A    STS     0x014A,R24       Store direct to data
space
77:           last_cnt=new_state;
+000003BF:   8189        LDD     R24,Y+1          Load indirect with
displacement
+000003C0:   9380014C    STS     0x014C,R24       Store direct to data
space
79:         last_state=new_state;
+000003C2:   8189        LDD     R24,Y+1          Load indirect with
displacement
+000003C3:   9380014B    STS     0x014B,R24       Store direct to data
space

Danke Christian!

Funktioniert jetzt.
Allerdings scheint es sich genau so wie mein bisheriger Algorythmus zu
verhalten (siehe Datenblatt des DDM427 von ALTRON unten rechts).
Erst ab 2KHz Abtastrate geht beim schnellen Drehen nichts verloren.

MfG
Andi

Hallo Andi,

ich verwende 500Hz Abtastrate. Läuft allerdings nur als manuelles
Eingabegerät. Dass heisst, es stört eigentlich nicht, wenn beim
schnellen drehen per Hand Pulse verloren gehen. Der User kann beim
Kurbeln sowieso nicht mitzählen... Bei der langsamen Feineinstellung -
solange man mitzählen kann - reichen die 500Hz.

Grüße
  Christian

Wieso?

last_cnt und last_state sind verschieden, kann man leicht übersehen
:-)

last_cnt ist die Bitkombination bei der zuletzt gezählt wurde (00) oder
(11)

last_state ist der letzte Zustand (10)/(01) d.h. woher ich komme

Gruesse
  Christian

Ich weis net, wies euch da geht, aber ich krieg immer PINEx nicht
deklariert und damit bricht der gcc ab!

Sorry, mein oberes Posting is absolouter Mist, habs jetzt schon kapiert
was PINE sein soll. Brett vorm Kopf.

Hallo,

habe den Code so erweitert, dass man auch mehrere Drehgeber einlesen
kann.

1

void evalRots(unsigned int rotData)

2

//Erkennt die Umdrehungen der Encoder

3

{

4

  static unsigned int last_state = 0;

5

  static unsigned int last_cnt = 0;

6

7

  for(unsigned char i=0; i< 12; i+=2)    //Encoder Anzahl: 6

8

  {

9

    if (((rotData^last_cnt) & (1 << i))  && ((rotData^last_cnt) & (1 <<

10

(i+1))))

11

    {

12

      if ((rotData ^ last_state) & (1 << i))

13

        enc_delta[(i>>1)]--;

14

      else

15

        enc_delta[(i>>1)]++;

16

17

      if((last_cnt^rotData) & (1 << i))      //bits verschieden?

18

        last_cnt^=(1 << i);            //togglen

19

      if((last_cnt^rotData) & (1 << (i+1)))    //bits verschieden?

20

        last_cnt^=(1 << (i+1));          //togglen

21

22

    }

23

    if((last_state^rotData) & (1 << i))        //bits verschieden?

24

      last_state^=(1 << i);            //togglen

25

    if((last_state^rotData) & (1 << (i+1)))      //bits verschieden?

26

      last_state^=(1 << (i+1));          //togglen

27

  }

28

}

Bei mir befinden sich die Graycode werte in einem Integer, da ich die
Encoder über zwei schieberegister einlese. rotData kann aber auch
einfach in Port des uc sein. Der Code macht im prinzip genau das
gleiche wie der Orginalcode des Threads, wahrscheinlich lässt sich da
noch einiges vereinfachen, laufen und funktionieren tut er aber.

Tubbu

Hallo Christian,

kannst Du mal das hier ausprobieren ?

http://www.mikrocontroller.net/forum/read-4-37992.html#360188


Peter

hallo

ich versuche gerade das zum laufen zu bekommen
scheitere aber kläglich

hier mal ein paar schnipsel:



// belegung des Rotary Encoders
#define ROTARY_DDR      DDRD
#define ROTARY_PIN      PIND
#define ROTARY_PORT     PORTD
#define ROTARY_A        5
#define ROTARY_B        6
#define ROTARY_P        7

int enc_delta=0;

void encoder(void)
{
  static uint8_t last_state = 0;
  static uint8_t last_cnt = 0;
     uint8_t new_state;

  new_state=ROTARY_PIN & ((1<<ROTARY_A) | (1<<ROTARY_B));

  if ((new_state^last_cnt)== (1<<ROTARY_A) | (1<<ROTARY_B))
    {

  if ((new_state ^ last_state)==(1<<ROTARY_A))
    enc_delta+=1;
  else
    enc_delta-=1;

  last_cnt=new_state;
  }
  last_state=new_state;
}


int main (void)
{
     ROTARY_DDR &=~ (1<<ROTARY_A)|(1<<ROTARY_B)|(1<<ROTARY_P);
     ROTARY_PORT |= (1<<ROTARY_A)|(1<<ROTARY_B)|(1<<ROTARY_P);



  for(;;)
  {

        encoder();

        itoa(enc_delta,out,10);
  lcd_gotoxy(13,0);
  lcd_puts(out);

        }

}



es kommt aber leider nur ein extrem schnelles zählen auf das LCD
new_state und last_state sind immer auf 0
es passiert nix bei drehung

könnte mir bitte jemand sagen warum ?
hab schin gesucht , finde aber nix

Danke

leider keine besserrung

hi

also ich habe es nun zum laufen bekommen
zwar mit anderem code

da dieser aber immer 2 schritte zählt  teile ich das ganze immer durch 2


#define ROTARY_DDR      DDRD
#define ROTARY_PIN      PIND
#define ROTARY_PORT     PORTD
#define ROTARY_A        5
#define ROTARY_B        6
#define ROTARY_P        7

volatile int enc_delta =0 ;


SIGNAL (SIG_OVERFLOW0)
{
  static char enc_last  = 0x01;
  char     i       = 0;
  if( (ROTARY_PIN & (1<<ROTARY_B) ))
    i = 1;
  if( (ROTARY_PIN & (1<<ROTARY_A) ))
    i ^= 3;
    i -= enc_last;
  if( i & 1 )
    {
    enc_last += i;
      enc_delta += (i & 2) - 1;
      }
}

@  wilfried mühlenhoff (Gast)

>Ich verwende Encoder mit 30 oder 40iger Auflösung.
>Da kriegt der AVR ganz schön was zu tun.

Nö.

>Sollte man den kleinsten Eingang über IRQ aufnehmen,
>anstatt mit hoher Samplerate zu arbeiten?

Nein, siehe Artikel Drehgeber.

>Habe ich das auch bei Drehgebern mit HALL-IC?

Ja. Siehe Artikel.

@  Harry (Gast)

>Because you cannot write in the register PINX.

Yes, you can! ;-)
In the new AVRs, writing a PINX bit will toggle the corresponding PORTX 
Bit!

Regards
Falk

Hallo,

ich habe den Code verwendet, nur ergibt sich bei mir bei der Abfrage

new_state ^ last_state = 11

Also ein unerwünschte Ergebnis, da ja nur 10 oder 01 erwartet wird.

Kann mir jemand helfen?

1

ISR( TIMER2_OVF_vect )

2

{

3

4

new_state = PINL & ((1<<5) | (1<<3))  ;  // save state

5

6

// Change in State on both important positions?

7

if (  (new_state ^ last_count) == ((1<<5 )|(1<<3))    ) 

8

  {

9

    // right or left turn?

10

    if(  (new_state ^ last_state) == (1<<3 ) )

11

      {enc_delta += 1;}

12

    else                               // Falle für unerwünschten Zustand

13

      {enc_delta -= 1;}

14

15

    enc_last_count = enc_new_state;    // save Bitcombination of last count

16

  }

17

enc_last_state = enc_new_state;        // save last state

18

19

}