avr assembler frage

[M. S. (sadmarvin)]

ich habe mein erstes etwas längeres Programm in assembler geschrieben.

Es ist zur Ansteuerung einer 3x3 LED Matrix mit einem ATtiny2313

Ich habe 2 Timer verwendet. Einer für das Multiplexing der Spalten
und einer für das durchschalten der einzelnen Komplettbilder.

Die Bilder sind so im Programmspeicher abgelegt

bilder:
  .db 0b00000011  ;3 Bilder
bild0:
  .db 0b00100000
  .db 0b00010010    
  .db 0b00001000

  .db 0b00100000
  .db 0b00010010    
  .db 0b00001000

  .db 0b00100010
  .db 0b00010101    
  .db 0b00001010

am Anfang des Programmes wird dann der Wert an der Stelle bilder in 
einem Register gespeichert.

  ldi ZH,LOW(bilder*2)    ;Low-Byte der Adresse von bilder in ZL  
  ldi ZH,HIGH(bilder*2)    ;High-Byte der Adresse von bilder in ZH
  lpm 
  mov bilderanzahl,r0

In einem weiteren REgister ist die nr. des aktuellen Bildes gespeichert.
Im Interrupt Handler Für den Bilder Timer wird dieser Wert dann mit der 
gesamtanzahl der Bilder verglichen und bei Bedarf wieder auf null 
gesetzt.
Und am ende um 1 erhöht.

  cp nextbild,bilderanzahl  ;nextbild mit Anzahl Bilder vergeleichen
  brne next
  mov nextbild,zero
next:

Das funktioniert leider nicht. Es sieht so aus als ob weiter als 3 
gezählt wird.

Wenn ich aber

cpi nextbild,3

nehem funktioniert es wie gewünscht

muss ich da evtl. einen anderen Vergleichsbefehl oder eine andere 
Sprungbedingung nehmen?

Das komplette Programm ist hier zu finden
http://pastebin.com/ebNeXcN2

p.s.:
ich beschäftige mich erst seit gut 1-2 Wochen mit der µC Programmierung.

[MWS (Gast)]

M. S. schrieb:
> ldi ZH,LOW(bilder*2)

Du musst das LB auch in's ZL laden...

[Spess53 (Gast)]

Hi

>Du musst das LB auch in's ZL laden...

Genau.

> lpm
>  mov bilderanzahl,r0

Das kannst du dir auch sparen

  lpm bilderanzahl,Z

Oder besser

  lpm bilderanzahl,Z+

dann zeigt Z gleich auf den naächsten Wert.

MfG Spess

[M. S. (sadmarvin)]

MWS schrieb:
>> ldi ZH,LOW(bilder*2)
>
> Du musst das LB auch in's ZL laden...
ups, das habe ich ganz übersehen, war wohl ein Tippfehler ;)
danke

[Steffen (Gast)]

Ich vermute mal, dass das Register "bilderanzahl" noch irgendwo anders 
mit genutzt wird. Um welches Register handelt es sich denn?

[M. S. (sadmarvin)]

r22

aber der Fehler war wie gesagt, dass ich

ldi ZH,LOW(bilder*2)

statt

ldi ZL,LOW(bilder*2)

da stehen hatte. Jetzt funktioniert es.