Ich werde irre ;-) (Timer compare match interrupt)

[Michael K. (stoertie)]

Hallo zusammen,

ich brauche mal einen kleinen Schubbs in die richtige Richtung. Ich 
weiß, das klingt lächerlich, aber ich sitze schon seit einer gefühlten 
Ewigkeit daran und versuche auf einem ATmega8 (derzeit erstmal nur 
testhalber) einen Compare Match Interrupt zum laufen zu bringen.(Zu 
meiner Entschuldigung muss ich sagen, ich habe eine ganze Zeit nichts 
mit Controllern gemacht und jetzt hänge ich irgendwie fest. heul

Der ATmega8 soll an PORTD,7 (Port ist auf Ausgang konfiguriert) beim 
ersten Interrupt (nach ca. einer Sekunde) eine LED einschalten und beim 
nächsten Interrupt (wieder nach ca. einer Sekunde) die LED wieder 
ausschalten und so weiter und so fort.

Ich liste Euch mal die wichtigen Teile des Codes auf, vielleicht kann 
einer von mal drüber gucken und mich in die richtige Richtung schubbsen. 
;-)

Interrupt-Vektor ist gesetzt.

.include  "m8def.inc"
;------------------------------------------------------------------------
;Reset and Interrupt vector             ;VNr.  Beschreibung
.
.
rjmp  HELLIGKEIT    ;8   TC1 Compare Match B
.
.

Den Timer habe ich folgendermaßen konfoguriert:

  ldi  temp,0b00000000
  out  TCCR1A,temp  

  ldi  temp,0b00001101
  out  TCCR1B,temp  ; CTC und Prescaler auf 1024

; Timer soll bis 1000(dez) zählen, das macht bei 1MHz gut eine Sekunde (1,024 Sekunden um genau zu sein).

  ldi  r16,0b00000011
  ldi  r17,0b11101000
  out  OCR1BH,r16  ;lade Comparewert 3(dez) ins H-Byte
  out  OCR1BL,r17  ;lade Comparewert 232 ins L-Byte 
  
  ldi  temp,0b0001000
  out  TIMSK,temp  ; aktiviere Output Compare B Match Interrupt

Die Mainloop ist völlig simpel (ist ja nur zum Testen:

mainloop:  wdr
          rjmp  mainloop

...und hier die Interrupt-Routine:

;----------LED-Interruptroutine --------------------------------------
LED: 
  sbic  PORTD,7  ; springe wenn LED "AUS"
  rjmp  AUS      ; gehe zu "AUS"
  rjmp  EIN      ; gehe zu "EIN" 
AUS:
  cbi  PORTD,7  ; schalte LED aus
  reti           ; zurück
  
EIN:
  sbi  PORTD,7  ; schalte LED ein
  reti             ; zurück

Die LED blinkt ums verrecken nicht. Sie schaltet nach einen Reset nach 
der richtigen Zeit ein und bleibt dann eingeschaltet. (Verlängere ich 
die Zeit durch vergrößeren der OCR1B Werte schaltet sich die LED später 
ein, aber sie schaltet sich nicht mehr aus.

Ich bin sicher ich habe irgendwo, irgendeine Kleinigkeit übersehen und 
mich dann in einer "Endlosschleife" verrannt und brauche tatsächlich nur 
einen kleinen Schubbs in die richtige Richtung.

Viele Grüße
Michael

[MaWin O. (mawin_original)]

Michael K. schrieb:
> Interrupt-Vektor ist gesetzt.

Bitte kompletten Source als Anhang posten.
Ich sehe dort nicht, wie die Vektoren korrekt gesetzt sind.

[Oliver S. (oliverso)]

Vor allem der Stackpointer…

Oliver

[Robert L. (bart)]

Kenne mich jetzt mit ATmega nicht so aus, aber Interrupts müssen 
allgemein freigegeben werden - Stichwort: SEI

[Achim H. (pluto25)]

Dieser CTC hat OC1a als Top. Wenn die kleiner als b sind wird er keinen 
weiteren int auslösen.

[Michael K. (stoertie)]

Guten Morgen zusammen,

sorry, dass ich mich erst jetzt melde...

Hier zuerst mal mein aktueller, kompletter Code:

.include  "m8def.inc"
;------------------------------------------------------------------------
;Reset and Interrupt vector             ;VNr.  Beschreibung
  rjmp  main  ;1   POWER ON RESET
  reti    ;2   Int0-Interrupt
  reti    ;3   Int1-Interrupt
  reti    ;4   TC2 Compare Match
  reti    ;5   TC2 Overflow
  reti    ;6   TC1 Capture
  reti    ;7   TC1 Compare Match A (
  rjmp  LED    ;8   TC1 Compare Match B
  reti    ;9   TC1 Overflow
  reti    ;10  TC0 Overflow 
  reti    ;11  SPI, STC Serial Transfer Complete
  reti    ;12  UART Rx Complete
  reti    ;13  UART Data Register Empty
  reti    ;14  UART Tx Complete
  reti    ;15  ADC Conversion Complete
  reti    ;16  EEPROM Ready
  reti    ;17  Analog Comparator
  reti    ;18  TWI (I²C) Serial Interface
  reti    ;19  Store Program Memory Ready
;------------------------------------------------------------------------
main:
;Start, Power ON, Reset
init:
  ldi  temp,HIGH(RAMEND)  ; HIGH-Byte der obersten RAM-Adresse
  out  SPH,temp
  ldi  temp,LOW(RAMEND)  ; LOW-Byte der obersten RAM-Adresse
  out  SPL,temp

;================================================================
; Ports initialisieren
;----------------------------------------------------------------

  sbi  DDRD,7  ; Port auf Ausgang 
  cbi  PORTD,7  ; Port auf H über Pullup 


;--------------------------------------------------------------------------------------
; Zuordnung der Register u. Laden von Vorgaben
;--------------------------------------------------------------------------------------
.def  temp  = r16

;------------------------------------------------------------------------
; TC1 - Timer Compare Match Interrupt setzen
;------------------------------------------------------------------------
  cli
  ldi  temp,0b00000000
  out  TCCR1A,temp  ; setze Compare Output Mode for channel A

  ldi  temp,0b00001101
  out  TCCR1B,temp  ; CTC und Prescaler auf 1024

  ldi  temp,0b00000011
  ldi  r17,0b11101000
  out  OCR1BH,temp  ;lade Comparewert 3(dez) ins H-Byte
  out  OCR1BL,r17  ;lade Comparewert 232(dez) ins L-Byte
  
  ldi  temp,0b0001000
  out  TIMSK,temp  ; aktiviere Output Compare B Match
  
  

  
  sei

mainloop:  wdr
  rjmp  mainloop

;----------LED-Interruptroutine -----------------------------------------------
LED: 
  sbic  PORTD,7
  rjmp  AUS
  rjmp  EIN
AUS:
  cbi  PORTD,7
  reti  
  
EIN:
  sbi  PORTD,7
  reti

@ MaWin O.:
Anhang in welchem Datei-Format? ist *.s OK?

Viele Grüße
Michael

[Reinhard R. (reirawb)]

Michael K. schrieb:
> ldi  temp,0b0001000      <---- da fehlt 'ne Ziffer, sind nur 7

> out  TIMSK,temp  ; aktiviere Output Compare B Match

Reinhard

[Michael K. (stoertie)]

Moin Reinhard,

stimmt, aber auch damit (OCIE1B = 1):

  ldi  temp,0b00001000
  out  TIMSK,temp

schaltet er die LED ein, aber nicht wieder aus. :-(

Viele Grüße
Michael

[Wastl (hartundweichware)]

Michael K. schrieb:
> Guten Morgen zusammen,

Wenn's noch schlimmer werden sollte: verinnerliche die Prosa
die vor jedem Textfenster zum Posten steht:

Wichtige Regeln - erst lesen, dann posten!
  ...........
  Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

Längerer Sourcecode ist der, der über eine Seite hinausgeht und
der Betrachter dauernd Scrollen muss um alles zu sehen/verstehen.

[Michael K. (stoertie)]

@Wastl:
Sorry, kommt nicht wieder vor! ;-)

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

Michael K. schrieb:
> ldi  temp,0b00001101
>   out  TCCR1B,temp  ; CTC und Prescaler auf 1024

CTC gut und schön, aber wo setzt du denn TOP? Bei Mode 4 (nur WGM12 
gesetzt) muss TOP in OCR1A geschrieben werden, sonst steht da eine Null 
drin.

[Michael K. (stoertie)]

Moin Matthias,

ich möchte den 16 Bit Counter (B) nutzen und habe deshalb auch OCIE1B im 
TCCR1B gesetzt und den TOP-Wert in OCR1BH und OCR1BL geladen. - Falsch?

Viele Grüße
Michael

[MaWin O. (mawin_original)]

Michael K. schrieb:
> Falsch?

ja
TOP im CTC-Modus ist OCR1A

[Michael K. (stoertie)]

Hat sich erledigt! Es läuft.

Und ich habe absolut Keine Ahnung warum...
Ich hatte probehalber den Code auf den Compare Match A umgeschrieben, da 
blinkte LED plötzlich - wieder zurück auf "B" und sie blinkte auch.

Ich danke Euch für Eure "Anteilnahme"...

Viele Grüße
Michael

[MaWin O. (mawin_original)]

Michael K. schrieb:
> wieder zurück auf "B" und sie blinkte auch.

kann nicht sein.