Probleme mit Counter auf Atmega8

[Philipp C. (noxkiller)]

Ich habe jetzt schon alles was mir einfiel versucht, das Datenblatt hin 
und her gewälzt aber ich komme nicht drauf. Assembler auf dem Atmega 
mache ich jetzt allerdings auch nicht jeden Tag sondern nur bei Bedarf, 
bin also für Anmerkungen zum Code offen. Der Ausschnitt ist Teil eines 
größeren Programms, aber hier liegt irgendwo ein Fehler von mir. Es 
sollen eigentlich nur alle Pins auf einem Port high gesetzt werden und 
dann nach einer definierten Zeit der Reihe nach ausgehen. Im Debugger 
alles kein Problem, es funktioniert wie ich will aber in real sieht es 
anders aus. Bleibt der Atmega an der ISP Schnittstelle nach dem 
Programmieren hängen geht alles, nehm ich die Schnittstelle ab sehe ich 
die LEDs erst gar nicht angehen. Als ob der Prescaler nicht richtig 
gesetzt ist und ich mit maximalen internen Takt (für mich nicht 
sichtbar) alle LEDs ausgeschaltet werden. Der Atmega soll übrigens mit 
1MHz internem Takt laufen.

.include "m8def.inc"

.def temp = r18
.def counter0 = r19
.def wait0 = r20
.def wait1 = r21

.org 0x0000
     rjmp     init_ports                   

.org OVF0addr
    rjmp     timer0_overflow

init_ports:
      ldi temp,0xFF  ;Port D0...7 Ausgang
          out ddrd,temp
      ldi temp,0xFF  ;Pullups aktiv
          out portd,temp


      ldi     temp, 0b00000101       ; Prescaler 1024
          out     TCCR0, temp

      ldi    temp, 0b00000001    ; Timer Interrupt erlauben
      out   TIMSK, temp

      ldi    counter0, 0x0B      
      out   TCNT0, counter0

      sei

start:    cpi    wait1, 1
      brlo  start
    
  wait_8:  cbi    PORTD, 0
      cpi    wait1, 2
      brlo  wait_8

  wait_7:  cbi    PORTD, 1
      cpi    wait1, 3
      brlo  wait_7

  wait_6:  cbi    PORTD, 2
      cpi    wait1, 4
      brlo  wait_6

  wait_5:  cbi    PORTD, 3
      cpi    wait1, 5
      brlo  wait_5

  wait_4:  cbi    PORTD, 4
      cpi    wait1, 6
      brlo  wait_4

  wait_3:  cbi    PORTD, 5
      cpi    wait1, 7
      brlo  wait_3

  wait_2:  cbi    PORTD, 6
      cpi    wait1, 8
      brlo  wait_2

  wait_1:  cbi    PORTD, 7
      cpi    wait1, 9
      brlo  wait_1

rjmp start

timer0_overflow:

         push    temp
        in      temp,sreg
        push    temp

      inc   wait0
      cpi   wait0, 10
      brlo  timer0_overflow_out
      ldi    wait0, 0x00
      inc   wait1
    
      timer0_overflow_out:
        pop     temp
        out     sreg,temp
        pop     temp

reti

[Hubert G. (hubertg)]

Wenn es mit angestecktem Programmer funktioniert sieht es eher nach 
einem HW-Fehler aus.
Spannung vorhanden?
Reset auf high?

[Karl H. (kbuchegg)]

Die Initialisierung des Stack Pointers fehlt. Der Pop bzw. Return aus 
dem Interrupt geht damit in die Hose.

[Karl H. (kbuchegg)]

> Bleibt der Atmega an der ISP Schnittstelle nach dem Programmieren
> hängen geht alles, nehm ich die Schnittstelle ab sehe ich die LEDs
> erst gar nicht angehen. Als ob der Prescaler nicht richtig gesetzt ist

.... so als ob es ein Problem mit dem Reset Pin gibt
.... so als ob es ein Problem mit der Spannungsversorgung gibt

[oldmax (Gast)]

Hi
Nun, ein Setzen des Stackpointers ist beim Atmega8 nicht unbedingt 
erforderlich, da der SP beim Start auf 0 steht und durch PUSH erst dec. 
und dann beschrieben wird. Dennoch kommen die paar Befehle auch bei mir 
grundsätzlich vor dem ersten CALL in das Programm:

LDI  Reg_A ,high(RAMEND)  ; Stack Pointer setzen 
OUT  SPH, Reg_A    ; "RAMEND" ist in m8def.inc festgelegt
LDI  Reg_A, low(RAMEND)  
OUT  SPL,Reg_A

Soviel zum Stack. Der Fehler liegt vermutlich in der Hardware, obwohl 
genug am Programm zu meckern gibt.
Wenn du einen Timer mit Interrupt benutzt, dann arbeite anders:
Im Timer- Interrupt zählst du deine Variable hoch. Dann vergleichst du 
den Wert mit einer Konstanten und bei Übereinstimmung setzt du ein Bit 
(Job-Flag). Dieser Programmteil hat mit deiner Programmschleife 
überhaupt nix zu tun und läuft dann autark.
Nun zum Job-Flag. Soll bei einem bestimmten Zählerstand eine Bearbeitung 
stattfinden, so kannst du natürlich auch außerhalb der ISR den Wert 
vergleichen. Das erfolgt unter Umständen aber mehrmals mit positivem 
Ergebnisin der Programmschleife (Zykluszeit < Bearbeitung im Timer). 
Nicht immer ist das erwünscht und wenn dieses Ereignis nur einmal 
bearbeitet werden soll, nimmt man ein Job-Flag. Ist dieses gesetzt, dann 
führst du die beabsichtigten Schritte durch und setzt anschließend das 
Job-Flag zurück. Nun wird dieses Programm erst wieder aufgerufen, wenn 
im Timer das Bit gesetzt wird. Dein Programm läuft also immer durch alle 
Abfragen und bleibt nicht wie beim vorgestellten Code in einer 
Unterschleife hängen. So ist es niemals über normale Eingänge bedienbar 
und du wärest gezwungen, bei Tastereingaben z.B. mit einer ISR zu 
arbeiten. Schau mal über die Suchmaschine, ob du irgend welche Beiträge 
zu Jobflags findest. Dann wird dir vermutlich auch deutlich, welche 
Struktur du beim Programmieren, grad in Assembler, dir angewöhnen 
solltest.
Ansonsten schau mal zur Konkurenz AVR-Praxis.de. Dort ist in der Rubrik 
FAQ ein Beitrag "keine Angst vor Assembler" mit hoffentlich 
interessanten Hinweisen.
Sie mal nach, ob vom Reset-Pin ein Widerstand von ca. 10 KOhm nach VCC 
geht. Auch evtl. die GND-Leitung überall richtig verschaltet ist.
Gruß oldmax

[Philipp C. (noxkiller)]

So, an RESET, VCC und AVCC liegen saubere 5V DC. An 8 und 22 sauberer 
Ground. Der Rest des Programms nach dem oberen Codeauszug funktioniert 
ja auch, der Fehler sollte demnach im Programm liegen. Versorgung beim 
Programmieren kommen auch aus der Schaltung. Es wird also nichts 
fremdgespeist, weshalb es nur mit angeschlossenem ISP funktioniert. 
Initialisieren des Stack Pointers werde ich dann auch mit aufnehmen. 
Arbeiten mit Job Flags kenne ich, werde dann das Programm gleich mal 
umstricken.

[Philipp C. (noxkiller)]

Zur Info, es lag am Programm, wo genau kann ich nicht sagen, mit 
Job-Flags gehts.