Flexibler Encoder Handler

avogra · September 13, 2009

hallo,

nachdem mich gestern die encoder doch ziemlich genervt haben, hab ich mal n bisschen ursachenforschung betrieben. Leider hab ich in Augsburg weder Werkzeug, noch MessgerÃ¤t oder Ã¤hnliches. Ging aber auch ohne, und ich hab sogar ne Flick-LÃ¶sung in C gefunden :)

Also nochmal das Problem im Detail: Bei den Pollin-Encodern gibt es einige (2-3) (Rast-)Stellungen, in denen der Rastpunkt ziemlich genau mit der Flanke des 2ten Encoder-Pins zusammenfÃ¤llt. Liegt vermutlich am ungewÃ¶hnlichen (?) Aufbau des Encoders. Das fÃ¼hrt nun dazu, dass der Pin bei der leichtesten ErschÃ¼tterung umschaltet.

Ich hab ursprÃ¼nglich MIOS_ENC_MODE_DETENTED mit "Speed-Mode Fast (2)" verwendet. Laut der Ãœbersicht http://www.ucapps.de/mios/mios_encoder_modes.gif ist das genau der erste INC-Ãœbergang. Genau das beobachte ich auch in der Software: in DIN_ENC_NotifyToggle kommen zufÃ¤llig "incrementer >0" an und mein zÃ¤hler springt munter nach oben. da der DEC-Ãœbergang auf einer anderen Flanke liegt, gehts auch nicht wieder nach unten.

probeweise hab ich mal MIOS_ENC_MODE_UNDETENTED mit "Speed-Mode Normal" verwendet. Da hier jede Flanke ausgewertet wird, habe ich erwartet, dass nun abwechselnd +/-1 als incrementer ankommt. mein zÃ¤hler wÃ¼rde also nicht mehr nach oben zÃ¤hlen sondern zufÃ¤llig zwischen zwei benachbarten Werten wechseln. Interessanterweise kamen aber immernoch aufeinanderfolgende incrementer == +1.

Also hab ich mal n bisschen nach Theorie zur Encoder-Auswertung gesucht und bin auch schnell fÃ¼ndig geworden http://www.hilpers.com/604594-prelldauer-bei-encoder-drehimulsgeber (kein schÃ¶ner thread, aber die info, die ich gesucht hab, war drin) . Kurz zusammengefasst gibts im Wesentlichen folgende gÃ¤ngige Implementierungen:

Abfrage: regelmÃ¤ÃŸiges Polling <-> Interrupt

-da die Encoder am SR hÃ¤ngen, geh ich bei MIOS mal von ersterem aus

Auswertung: Finite-State-Maschine <-> Ein Pin wird als Takt, zweiter als Richtung ausgewertet

-die zweite MÃ¶glichkeit hab ich nicht hundertprozentig verstanden, aber laut dem Thread wÃ¼rde sie genau das beobachtete Verhalten erklÃ¤ren. DafÃ¼r braucht sie weniger Rechenleistung.

Eine "saubere" Encoder-Implementierung verwendet also grundsÃ¤tzlich eine Finite-State-Maschine zur auswertung, die Art der Abfrage spielt dann keine Rolle. Also hab ich mal meinen einen Encoder aus ENC_TABLE gelÃ¶scht und in DIN_NotifyToggle eine eigene Encoder-Auswertung geschrieben. Der Encoder funktioniert jetzt einwandfrei, allerdings komm ich bei allzu schnellem Drehen an die Performance-Grenzen. Liegt mit Sicherheit an C bzw. meinem noch optimierbaren Code.

Die ZustÃ¤nde der Finite-State-Machine entsprechen der PinA/B-Kombination. Theoretisch mÃ¶glich sind 16 ÃœbergÃ¤nge zwischen diesen ZustÃ¤nden. Da nur benachbarte ZustÃ¤nde interessieren, sind nur 8 ÃœbergÃ¤nge relevant. Ich hab das ganze mal in ne Grafik gepackt:

Mein Code is eigentlich ganz simpel. Er merkt sich bei nem neuen DIN_NotifyToggle-Aufruf den letzten Zustand und packt den mit dem neuen Zustand in ein variable, so dass darin letztendlich ein Ãœbergang entsprechend der Tabelle oben enthalten ist. Hat diese den Wert 2 oder D, wird ENC_NotifyChange mit positivem incrementer aufgerufen, bei Wert 8 oder 7 mit negativem. Die wackligen ÃœbergÃ¤nge 1 und 4 spielen also gar keine Rolle mehr.

SchÃ¶ner wÃ¤rs natÃ¼rlich, wenn MIOS schon einen entsprechenden ENC_MODE hÃ¤tte *zwinker*

u.U. kÃ¶nnten die rastungen natÃ¼rlich auch in die andere Richtung verschoben sein. Deshalb wÃ¤ren zwei neue Modi ne feine Sache:

MIOS_ENC_MODE_DETENTED4

positiver incrementer fÃ¼r Ãœbergang 2 und D, negativer fÃ¼r Ãœbergang 8 und 7

MIOS_ENC_MODE_DETENTED5

positiver incrementer fÃ¼r Ãœbergang B und 4, negativer fÃ¼r Ãœbergang E und 1

BTW, mit is grad MIOS_ENC_MODE_DETENTED3 in mios_enc_table.inc aufgefallen. was macht denn der?

GrÃ¼ÃŸe,

Alex

TK. · September 13, 2009

Hallo Alex,

danke fuer die Ursachenforschung! Ich habe bereits erwartet, dass sich das Problem so einfach loesen laesst, bin jedoch nicht davon ausgegangen, dass da noch jemand anderes "durchsteigt". ;)

MIOS scannt die Pins periodisch, und wertet sie dann mit einer Finite State Machine aus.

Deine Tabelle findest Du unter mios_enc.inc wieder:

; State Machine (own Design, the Machine from Microchip on AN519
; is too simple and doesn't work correctly!)
; changed 2000-1-5: special "analyse" state which corrects the ENC direction
; if encoder is rotated to fast - I should patent it ;-)

;; ENC_STAT[3..0] = LB, LA, B, A

;; type 0x00 - INC: 2, B, D, 4
;; DEC: 1, 7, E, 8
;; type 0x80 - INC: B, 4
;; DEC: 7, 8
;; type 0x81 - INC: B
;; DEC: 7
;; type 0x82 - INC: 4
;; DEC: 8

movf IRQ_TMP4, W
andlw 0x0f
JUMPTABLE_2BYTES_UNSECURE
rgoto MIOS_ENC_Do_Nothing ; 0
rgoto MIOS_ENC_Do_Dec_ND ; 1 - only if NON_DETENTED
rgoto MIOS_ENC_Do_Inc_ND ; 2 - only if NON_DETENTED
rgoto MIOS_ENC_Do_Nothing ; 3
rgoto MIOS_ENC_Do_Inc_D13 ; 4 - only if NON_DETENTED, DETENTED1 or DETENTED3
rgoto MIOS_ENC_Do_Nothing ; 5
rgoto MIOS_ENC_Do_Nothing ; 6
rgoto MIOS_ENC_Do_Dec_D12 ; 7 - only if NON_DETENTED, DETENTED1 or DETENTED2
rgoto MIOS_ENC_Do_Dec_D13 ; 8 - only if NON_DETENTED, DETENTED1 or DETENTED3
rgoto MIOS_ENC_Do_Nothing ; 9
rgoto MIOS_ENC_Do_Nothing ; A
rgoto MIOS_ENC_Do_Inc_D12 ; B - only if NON_DETENTED, DETENTED1 or DETENTED2
rgoto MIOS_ENC_Do_Nothing ; C
rgoto MIOS_ENC_Do_Inc_ND ; D - only if NON_DETENTED
rgoto MIOS_ENC_Do_Dec_ND ; E - only if NON_DETENTED
rgoto MIOS_ENC_Do_Nothing ; F
[/code]

Bisher ist sie jedoch nur fuer 4 Encoder Typen ausgelegt. Um Speicher zu sparen (was ja beim PIC leider gang und gaebe ist...) sind diese Typen trickreich zusammengefasst, was eine Anpassung an einen weiteren Typ etwas schwierig machen wird. Doch Du koenntest die Tabelle ja mal versuchen so anzupassen, dass Deine Encoder richtig ausgewertet werden - und ich koennte dann bei Bedarf bei der Speicher-Optimierung helfen.

Fuer MIOS_ENC_MODE_DETENTED3 gibt es noch keine Grafik, ich habe das mal irgendwann fuer jemanden auf die Schnelle eingebaut, doch nie sauber dokumentiert... :-/

Zum besseren Verstaendnis des Algorithmus koenntest Du auch mal in die C Variante des Encoder Treibers reinschauen: mios32_enc.c

Gruss, Thorsten.

avogra · September 13, 2009

danke fuer die Ursachenforschung! Ich habe bereits erwartet, dass sich das Problem so einfach loesen laesst, bin jedoch nicht davon ausgegangen, dass da noch jemand anderes "durchsteigt". ;)

Das fass ich einfach mal als Kompliment auf, danke :) (Auch wenn das Ganze wohl vor allem eine Frage der MuÃŸe ist :P )

Ich hab mir den MIOS32-Code mal angeschaut. bin weitgehend klar gekommen, lustigerweise hast du sogar die gleiche Reihenfolge fÃ¼r die Pins in ENC_STAT hergenommen, es sind also auch bei dir case 1 und 4, wo mein encoder zickt. Seh jetzt auch, warum das mit dem Erweitern bissl ungÃ¼nstig ist.

Nachdem sich ja die States fÃ¼r DETENTED2 und DETENTED3 gegenseitig ausschlieÃŸen, hab ich die beiden einfach fÃ¼r meine States missbraucht. DETENTED1 ist damit zwar identisch mit UNDETENTED aber damit werd ich wohl leben kÃ¶nnen. Mein Table sieht jetzt so aus:

	JUMPTABLE_2BYTES_UNSECURE
	rgoto	MIOS_ENC_Do_Nothing	; 0
	rgoto	MIOS_ENC_Do_Dec_D13	; 1  -  only if NON_DETENTED, DETENTED1 or DETENTED3
	rgoto	MIOS_ENC_Do_Inc_D12	; 2  -  only if NON_DETENTED, DETENTED1 or DETENTED2
	rgoto	MIOS_ENC_Do_Nothing	; 3
	rgoto	MIOS_ENC_Do_Inc_D13	; 4  -  only if NON_DETENTED, DETENTED1 or DETENTED3
	rgoto	MIOS_ENC_Do_Nothing	; 5
	rgoto	MIOS_ENC_Do_Nothing	; 6
	rgoto	MIOS_ENC_Do_Dec_D12	; 7  -  only if NON_DETENTED, DETENTED1 or DETENTED2
	rgoto	MIOS_ENC_Do_Dec_D12	; 8  -  only if NON_DETENTED, DETENTED1 or DETENTED2
	rgoto	MIOS_ENC_Do_Nothing	; 9
	rgoto	MIOS_ENC_Do_Nothing	; A
	rgoto	MIOS_ENC_Do_Inc_D13	; B  -  only if NON_DETENTED, DETENTED1 or DETENTED3
	rgoto	MIOS_ENC_Do_Nothing	; C
	rgoto	MIOS_ENC_Do_Inc_D12	; D  -  only if NON_DETENTED, DETENTED1 or DETENTED2
	rgoto	MIOS_ENC_Do_Dec_D13	; E  -  only if NON_DETENTED, DETENTED1 or DETENTED3
	rgoto	MIOS_ENC_Do_Nothing	; F

Funktioniert einwandfrei :D Vieeelen Dank fÃ¼r den Hinweis!

GrÃ¼ÃŸe,

Alex

rosch · September 14, 2009

cool, dass du dir die arbeit machst, die panasonic encoder einzubinden!

avogra · September 14, 2009

na wenn ich kein geld fÃ¼r neue ausgeben will, bleibt mir wohl kaum was anderes Ã¼brig :P

ich nehm mal an, thorsten sieht ungern ne zweite version von mios hier rumgeistern. wenn noch jemand probleme mit den panasonic encodern hat und mios nicht selber anpassen will, dem kann ich ja meine version auf anfrage schicken.

GrÃ¼ÃŸe, Alex

TK. · September 14, 2009

Ich werde Deine Tabelle in die aktuelle MIOS Version einbauen.

Das kostet dann zwar 40 zusaetzliche Bytes, doch die sind es wert! ;)

Wenn man DETENTED[45] wie folgt codiert:

MIOS_ENC_MODE_DETENTED2 EQU 0xc3
MIOS_ENC_MODE_DETENTED3 EQU 0xc4
[/code]

dann koennte man einfach nach einer positiven Ueberpruefung des 7ten Bits zur alternativen Tabelle verzweigen.

Gruss, Thorsten.

avogra · September 14, 2009

achso, ich dachte, jedes byte extra ist ein absolutes No-Go.

wenn dem nicht so ist, schreib ich auch gern die tabelle und die aufgerufenen funktionen um, so dass meine vorgeschlagenen modi auch nen platz drin haben (leider nur in C). Das einfach mit ner Verzweigung zu machen, und die Tabelle doppelt reinzuschreiben ist ja nicht die sauberste LÃ¶sung.

Ich bin mal durchgegangen, was man Ã¤ndern mÃ¼sste. Bisher sinds ja 7 Funktionen, die Aufgerufen wurden. Und es geht sich wunderschÃ¶n aus, dass du pro Funktion einen Vergleich brauchst, um den ENC_MODE zu filtern.

Wenn ich meine beiden Modi mit rein nehm, brauch ich eine zusÃ¤tzliche Funktion, und jede Funktion braucht dann 2 Vergleiche gegen ENC_MODE. Ich weiÃŸ nicht, wieviel Platz das beansprucht, aber ich vermute mal, weniger als 40 Byte. Performance-mÃ¤ÃŸig sollte es nicht mehr kosten als ne zusÃ¤tzliche verzweigung in meine tabelle.

Ich schreib die Tabelle und die Funktionen mal entsprechend um. Kannst ja dann schaun, obs dir gefÃ¤llt und in Assembler umschreiben :-P

//edit: hab mir den Assembler nochmal genauer angeschaut. Ich werd's da auch mal probiern. Mehr als nicht funktioniern kanns ja nicht :)

GrÃ¼ÃŸe,

Alex

avogra · September 14, 2009

Hey Thorsten,

ich glaub ich hab grad ne MÃ¶glichkeit gefunden, wie man den jump-table los wird und gleichzeitig jeden beliebigen detent-mode ohne zusÃ¤tzlichen code implementieren kann. -> Speicher gespart, bessere Performance, mehr FlexibilitÃ¤t -> was will man mehr 8)

Also, folgende Ãœberlegung:

Es gibt ja 8 Zustands-ÃœbergÃ¤nge, die uns interessieren. Ein Char hat bekanntlich 8 Bit. Man kÃ¶nnte die ENC_MODEs also so nutzen, dass jedes Bit angibt, ob bei dem entsprechenden Ãœbergang ein Do_Inc bzw. Do_Dec ausgelÃ¶st werden soll. Die ENC_MODEs kÃ¶nnten dann z.B. so aussehen:

Bit            7   6   5   4   3   2   1   0
Ãœbergang      0x8 0xE 0x7 0x1 0x4 0xD 0xB 0x2
---------------------------------------------
NonDetented    1   1   1   1   1   1   1   1    = 0xFF
Detented1      1   0   1   0   1   0   1   0    = 0xAA
Detented2      0   0   1   0   0   0   1   0    = 0x22

Detented4      1   0   1   0   0   1   0   1    = 0xA5 (Pollin-Mode1 :) )
...

Die ÃœbergÃ¤nge hab ich aus meiner Grafik von oben entnommen und sortiert nach Bit 0-3 = Do_Inc, Bit 4-7 = Do_Dec

Wenn sich einer der Encoder-Pins Ã¤ndert, muss die Routine also wie bisher nen Enc_State/Ãœbergang zusammenstellen.

dann in nem Lookup-Table dem Ãœbergang nen Wert Enc_State_Nr = 0 bis 7 zuordnen, entsprechend dem Tabellen-Kopf, oder 0xFF, falls es einer der ZustÃ¤nde ist, die nicht interessieren (ok, gibt immer noch nen Table, aber der enthÃ¤lt nur Zahlen).

Falls Enc_State_Nr nicht 0xFF ist:

Das Bit des ENC_MODEs an der Stelle Enc_State_Nr auf 1 prÃ¼fen.

falls das der Fall ist:

wenn Enc_State_Nr <= 3, Do_Inc aufrufen, ansonsten Do_Dec

Die Definition der ENC_MODEs am Anfang kann man sich dann auch schenken, da die jetzt (zumindest mit der Grafik) selbstredend sind. FÃ¼r Ã„nderungen an den Modes brauchts dann nur noch nen Eintrag in cmios.h.

Ich werd das Ganze gleich mal in C implementieren, musste den Gedanken nur ganz schnell irgendwo hin schreiben :P Ich fÃ¼rchte, fÃ¼r Assembler reichts bei mir noch nicht, aber vielleicht schaff ichs ja teilweise und ne gute Seele hilft mir bestimmt ;)

GrÃ¼ÃŸe, Alex

P.S.: irgendwie passt das ganze nur noch bedingt ins Topic, eigener Thread?

TK. · September 14, 2009

Das nenne ich mal eine Innovation! :)

Die Verwendung einer Tabelle wuerde zu mehr Code und einer laengeren Laufzeit fuehren, da TBLPTR[LHU] ja bereits zum Auslesen der Encoder-Tabelle verwendet wird.

Es gibt jedoch eine einfache Methode, um selbst die Jumptable zu umgehen (via Checkbit- und Branch).

Ich werde das gleich mal blind einhacken, und die Aenderung in das Repository stellen - heute noch Lust, die Aenderung auszuprobieren? Dank der speziellen Codierung muss MIOS wohl nur einmal neu aufgeladen werden :)

Gruss, Thorsten.

P.S.: den Thread splitte ich gleich

TK. · September 14, 2009

Ok, habe die Aenderung eingebaut, nun muesstest Du mal Dein Repository updaten, ein neues MIOS bauen und aufladen.

Geaenderte Files:

mios_enc.inc

macros.h

cmios.h

Unschoen: die Applikationen sind nun nicht mehr binaerkompatibel zur alten MIOS Version - muessen also ebenfalls neu kompiliert werden. :-/

(zum Glueck kein Problem fuer MIOS32, da es direkt in die Applikation gelinkt wird)

Viel Spass beim Testen!

Gruss, Thorsten.

P.S.: hier der geaenderte Code - er sieht zwar etwas laenglich aus, verbraucht jedoch exakt genausoviele Bytes wie die Jumptable zuvor - und wird zudem schneller abgearbeitet und ist dank Deiner Idee mit der speziellen Kodierung des Encoder Typs flexibler!

BRA_IFSET IRQ_TMP4, 3, ACCESS, MIOS_ENC_Do_89ABCDEF
MIOS_ENC_Do_01234567
BRA_IFSET IRQ_TMP4, 2, ACCESS, MIOS_ENC_Do_4567
MIOS_ENC_Do_0123
BRA_IFSET IRQ_TMP4, 1, ACCESS, MIOS_ENC_Do_23
MIOS_ENC_Do_01
;; 0: Do_Nothing
;; 1: Do_Dec
BRA_IFCLR IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 4, ACCESS, MIOS_ENC_Do_Dec
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_23
;; 2: Do_Inc
;; 3: Do_Nothing
BRA_IFSET IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 0, ACCESS, MIOS_ENC_Do_Inc
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_4567
BRA_IFSET IRQ_TMP4, 1, ACCESS, MIOS_ENC_Do_67
MIOS_ENC_Do_45
;; 4: Do_Inc
;; 5: Do_Nothing
BRA_IFSET IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 3, ACCESS, MIOS_ENC_Do_Inc
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_67
;; 6: Do_Nothing
;; 7: Do_Dec
BRA_IFCLR IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 5, ACCESS, MIOS_ENC_Do_Dec
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_89ABCDEF
BRA_IFSET IRQ_TMP4, 2, ACCESS, MIOS_ENC_Do_CDEF
MIOS_ENC_Do_89AB
BRA_IFSET IRQ_TMP4, 1, ACCESS, MIOS_ENC_Do_AB
MIOS_ENC_Do_89
;; 8: Do_Dec
;; 9: Do_Nothing
BRA_IFSET IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 7, ACCESS, MIOS_ENC_Do_Dec
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_AB
;; A: Do_Nothing
;; B: Do_Inc
BRA_IFCLR IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 1, ACCESS, MIOS_ENC_Do_Inc
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_CDEF
BRA_IFSET IRQ_TMP4, 1, ACCESS, MIOS_ENC_Do_EF
MIOS_ENC_Do_CD
;; C: Do_Nothing
;; D: Do_Inc
BRA_IFCLR IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 2, ACCESS, MIOS_ENC_Do_Inc
rgoto MIOS_ENC_Do_Nothing

MIOS_ENC_Do_EF
;; E: Do_Dec
;; F: Do_Nothing
BRA_IFSET IRQ_TMP4, 0, ACCESS, MIOS_ENC_Do_Nothing
BRA_IFSET TABLAT, 6, ACCESS, MIOS_ENC_Do_Dec
;; rgoto MIOS_ENC_Do_Nothing
[/code]

TK. · September 14, 2009

P.P.S.: Du musst nochmal updaten, da war noch ein Schreibfehler...

avogra · September 14, 2009

hui, das ging ja schneller als ich fÃ¼r den post gebraucht hab ^^ in c funktionierts bei mir. dann werd ich gleich mal testen.

TK. · September 14, 2009

Haette den Code vorher vielleicht doch nochmal ausprobieren sollen - ich habe ihn nach einer zweiten visuellen Kontrolle abermals geaendert, ihn jedoch nach wie vor nicht getestet - dafuer bist Du ja da ;)

Der Typo: das TABLAT, <bit> war nach IRQ_TMP4, <bit> gerutscht.

Das kommt davon, wenn man am kleinen Notebook Bildschirm tippt...

Gruss, Thorsten.

TK. · September 14, 2009

Habe die Aenderung nun an meiner MBSID (DETENTED2) gestestet, und einen weiteren Tippfehler gefunden.

Nun funktioniert es!

Es zeigt sich mal wieder, wie fehleranfaellig Assemblercode beim RabidRapid Prototyping ist...

Gruss, Thorsten.

avogra · September 14, 2009

wollt, grad posten, dass er state nr. 7 / 0x08 nicht erkennt -.-

nILS · September 14, 2009

Rabid prototyping hehehe, kein Wunder dass das nicht so tut wie's soll. Vielleicht ein bisschen mehr rapid und dafÃ¼r mit weniger Tollwut :-)

SchÃ¶nes Ding ihr zwei!

avogra · September 14, 2009

jepp, funktioniert perfekt :D

jetzt fehlt nur noch

#define MIOS_ENC_MODE_DETENTED5 0x5a

in cmios.h, falls bei jemand die rastung in die andere richtung verschoben ist :P

Soll ich fÃ¼r ucapps.de mal n neues bild mit allen encoder-typen machen, oder die state-machine dokumentieren?

GruÃŸ, Alex

TK. · September 14, 2009

Waehrenddessen habe ich die Aenderung auch in MIOS32 eingebaut: mios32_enc.c


      if( (enc_state_ptr->state == 0x01 && (enc_type & (1 << 4))) ||
	  (enc_state_ptr->state == 0x07 && (enc_type & (1 << 5))) ||
	  (enc_state_ptr->state == 0x0e && (enc_type & (1 << 6))) ||
	  (enc_state_ptr->state == 0x08 && (enc_type & (1 << 7))) ) {
        // DEC
        ...
      } else if( (enc_state_ptr->state == 0x02 && (enc_type & (1 << 0))) ||
		 (enc_state_ptr->state == 0x0b && (enc_type & (1 << 1))) ||
		 (enc_state_ptr->state == 0x0d && (enc_type & (1 << 2))) ||
		 (enc_state_ptr->state == 0x04 && (enc_type & (1 << 3))) ) {
        // INC
        ...
      }
[/code]


Sieht schon wesentlich kompakter aus, und funktionierte deshalb auch auf Anhieb mit meiner MBSEQ V4! :)


So sieht uebrigens der vom Compiler generierte ARM Assembler Code aus:
[code]
      if( (enc_state_ptr->state == 0x01 && (enc_type & (1 << 4))) ||
 801d56c:	f818 2007 	ldrb.w	r2, [r8, r7]
 801d570:	f002 030f 	and.w	r3, r2, #15	; 0xf
 801d574:	2b01      	cmp	r3, #1
 801d576:	d102      	bne.n	801d57e <MIOS32_ENC_UpdateStates+0x9e>
 801d578:	f014 0f10 	tst.w	r4, #16	; 0x10
 801d57c:	e00d      	b.n	801d59a <MIOS32_ENC_UpdateStates+0xba>
 801d57e:	2b07      	cmp	r3, #7
 801d580:	d102      	bne.n	801d588 <MIOS32_ENC_UpdateStates+0xa8>
 801d582:	f014 0f20 	tst.w	r4, #32	; 0x20
 801d586:	e008      	b.n	801d59a <MIOS32_ENC_UpdateStates+0xba>
 801d588:	2b0e      	cmp	r3, #14
 801d58a:	d102      	bne.n	801d592 <MIOS32_ENC_UpdateStates+0xb2>
 801d58c:	f014 0f40 	tst.w	r4, #64	; 0x40
 801d590:	e003      	b.n	801d59a <MIOS32_ENC_UpdateStates+0xba>
 801d592:	2b08      	cmp	r3, #8
 801d594:	d13b      	bne.n	801d60e <MIOS32_ENC_UpdateStates+0x12e>
 801d596:	f014 0f80 	tst.w	r4, #128	; 0x80
 801d59a:	f000 808a 	beq.w	801d6b2 <MIOS32_ENC_UpdateStates+0x1d2>
	  (enc_state_ptr->state == 0x07 && (enc_type & (1 << 5))) ||
	  (enc_state_ptr->state == 0x0e && (enc_type & (1 << 6))) ||
	  (enc_state_ptr->state == 0x08 && (enc_type & (1 << 7))) ) {
	// DEC

...
      } else if( (enc_state_ptr->state == 0x02 && (enc_type & (1 << 0))) ||
 801d60e:	2b02      	cmp	r3, #2
 801d610:	d102      	bne.n	801d618 <MIOS32_ENC_UpdateStates+0x138>
 801d612:	f014 0f01 	tst.w	r4, #1	; 0x1
 801d616:	e00d      	b.n	801d634 <MIOS32_ENC_UpdateStates+0x154>
 801d618:	2b0b      	cmp	r3, #11
 801d61a:	d102      	bne.n	801d622 <MIOS32_ENC_UpdateStates+0x142>
 801d61c:	f014 0f02 	tst.w	r4, #2	; 0x2
 801d620:	e008      	b.n	801d634 <MIOS32_ENC_UpdateStates+0x154>
 801d622:	2b0d      	cmp	r3, #13
 801d624:	d102      	bne.n	801d62c <MIOS32_ENC_UpdateStates+0x14c>
 801d626:	f014 0f04 	tst.w	r4, #4	; 0x4
 801d62a:	e003      	b.n	801d634 <MIOS32_ENC_UpdateStates+0x154>
 801d62c:	2b04      	cmp	r3, #4
 801d62e:	d140      	bne.n	801d6b2 <MIOS32_ENC_UpdateStates+0x1d2>
 801d630:	f014 0f08 	tst.w	r4, #8	; 0x8
 801d634:	d03d      	beq.n	801d6b2 <MIOS32_ENC_UpdateStates+0x1d2>
		 (enc_state_ptr->state == 0x0b && (enc_type & (1 << 1))) ||
		 (enc_state_ptr->state == 0x0d && (enc_type & (1 << 2))) ||
		 (enc_state_ptr->state == 0x04 && (enc_type & (1 << 3))) ) {

Im Endeffekt benoetigt der Cortex M3 ca. 90 Bytes zur Auswertung, und der PIC 106 Bytes. Hier laesst sich vielleicht noch etwas rausholen, doch dann wird der Code wirklich unlesbar...

Gruss, Thorsten.

TK. · September 14, 2009

Rabid prototyping hehehe, kein Wunder dass das nicht so tut wie's soll.

Auch diesen Typo habe ich nun gefixed ;)

jepp, funktioniert perfekt :D
jetzt fehlt nur noch
#define MIOS_ENC_MODE_DETENTED5 0x5a
in cmios.h, falls bei jemand die rastung in die andere richtung verschoben ist :P

Ok, habe ich in MIOS und MIOS32 nun ebenfalls eingebaut.

Soll ich fÃ¼r ucapps.de mal n neues bild mit allen encoder-typen machen, oder die state-machine dokumentieren?

Das waere nett - das Tolle an Deiner Loesung ist ja u.A, dass sie sich so einfach dokumentieren laesst! :)

Ich werde noch ein wenig warten, bis ich eine neue MIOS Version release - vorher moechte ich mir 100% sicher sein, dass die Aenderung nicht zu neuen Problemen fuehrt. Es ist uebrigens schon abzusehen, dass ein paar Newbies ueber diese Aenderung stolpern werden - bspw. wenn sie eine neu kompilierte Applikation auf einen PIC aufspielen, der von SmashTV mit MIOS V1.9f vorprogrammiert wurde - umso wichtiger, dass alle betroffenen Projekte einen nicht uebersehbaren Hinweis ueber den erforderlichen MIOS Update erhalten.

In MIOS32 ist die Aenderung mit sofortiger Wirkung gueltig ;)

Gruss, Thorsten.

lemonhorse · September 15, 2009

(Begeisterung!) :D

sneakthief · October 11, 2009

Ich habe ein paar Pollin-Encoder und habe mit Mios 1.9g MIOS_ENC_MODE_DETENTED4 gerade probiert - es gibt noch Stellungen, wo es wackelt ...ich benutze Clockbox Simple v1c.

Vlt. kann es sein, dass ich meine Encoders beim LÃ¶ten beschÃ¤digt habe?

lemonhorse · October 11, 2009

Bei mir wurden durch das Update auf MIOS 1.9g und MBSEQ V2.4f die frÃ¼heren "ÃœberspÃ¼nge" der Polin Encoder beseitigt (soweit ich das bisher prÃ¼fen konnte).

avogra · October 11, 2009

Probier alternativ mal DETENTED5 aus. das ist fast der gleiche modus wie DETENTED4, aber es werden genau die anderen Flanken ausgewertet. wenn das auch nichts hilft, kÃ¶nntest du auÃŸerdem noch statt MIOS_ENC_MODE_DETENTEDx direkt 0x66 oder 0x99 ausprobieren.

Sollte also dann so aussehen:

...
MIOS_ENC_PIN_TABLE
        ;; encoders 1-16
        ;;  SR  Pin  Mode
  ENC_ENTRY  4,  0,  0x66
...

GrÃ¼ÃŸe,

Alex

sneakthief · October 12, 2009

DETENTED5 funktioniert ganz perfekt :) Super!!!

vielen vielen Dank. ich befÃ¼rchtete, dass ich neue Encoders in meinem 'Sneaquencer' aufzubauen musste ( http://sneak-thief.com/sneakyseq.html )

km2301 · October 19, 2009

Hallo,

habe auch das Problem mit den Encoder und war begeistert von der LÃ¶sung.

Habe mich aber seit langem nicht mehr mit der Programmierung beschÃ¤ftigt(bis midibox1,6a).Also erst einmal wieder gelesen ,Hilfsprogramme runtergeladen und..... nichts geht.Die Version midibox 1,6b bekomme ich noch hin.Vielleicht kann ein User mal in Stichpunken das erzeugen einer hex Datei bei der Version midibox 1,6d erklÃ¤ren.Ich sehe schon das KopfschÃ¼tteln der meisten User aber ohne Hilfe komme ich nicht weiter.Also habt Gnade mit einem absoluten"Programiertrottel".Danke

GruÃŸ Bernhard

hat sich erledigt. Bei mir fehlte die" mingw32-make" da kann es ja nicht funktionieren.

Sign In

Flexibler Encoder Handler

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation