8051: Problem mit mov bzw. rlc / rrc Befehlen

[Michael L. (eagle87)]

Hallo zusammen,
ich hab das Problem, dass beim hin und her Kopieren eines Bytes nach 
einem rotier Befehl auf einmal alle Bits gelöscht sind. Konkret geht es 
um die Stelle an der auskommentiert "weiterer Code" steht. Wenn ich die 
mov Befehle davor und danach (mov nb0,a & mov a,nb0) entferne 
funktioniert es auf einmal. Ich komm einfach nicht drauf woran das 
liegen könnte. Eigentlich wird doch nur der Akku auf nb0 geschrieben und 
danach wieder nb0 auf den Akku.
Komischerweise werden die Bits nicht gelöscht wenn ich den Rotierbefehl 
weglasse. Aber ohne macht das Programm halt keinen Sinn.

#include <REG8253.H>

;Stack-Bereich im indirekt adressierbaren internen RAM
stackbereich  segment idata
  rseg  stackbereich
  ds  10h    ;16 Byte

;Bytes im int. direkt adressierbaren RAM
dir_daten  segment data
  rseg  dir_daten
nb0: ds 1

;LJMP von Adresse 0 im Code-Speicher zum Programmanfang
  cseg  at  0
  ljmp  start

;Programmcode (Code-Beginn über Linker festgelegt)
programmcode  segment code
  rseg  programmcode
  using  0    ;Registerbank 0 für Programmcode (rseg -> verschiebbar)

;Initialisierung  Stack
start:
  mov sp,#stackbereich-1  ;STACK-Definition

;--------------------------------------------------------------------------


  mov nb0,#0a6h
main:

  clr c
  mov a,nb0    ;Nennerbyte 0 durch Carry nach links
  rlc a
  mov nb0,a

  ;!!!weiterer Code!!!

  mov a,nb0    ;Nennerbyte 0 auf Port 0 ausgeben
  cpl a
  mov p0,a

  ljmp main


  end

Gruß
Michael

[Peter D. (peda)]

Alle Segmente beginnnen bei 0.
Dein nb0 zeigt also auf R0 der ersten Bank und der Stack auch.

Du mußt daher sinnvolle Adressen vergeben.
Daten legt man gerne hinter den bitadressierbaren Bereich, also ab 30h.
Und den Stack einfach hinter das letzte Datenbyte.

  rseg at 30h
nb0:   ds 1
; hier alle weiteren Daten
stack: ds 10h

Peter

[Jobst M. (jobstens-de)]

Wenn ich das richtig verstehe, funktioniert dieser Fetzen schon nicht. 
Korrekt?

Michael L. schrieb:
> mov nb0,#0a6h
> main:
>
>   clr c
>   mov a,nb0    ;Nennerbyte 0 durch Carry nach links
>   rlc a
>   mov nb0,a

Der Stack ist dem hier nicht im Weg. Auch R0 nicht.

Was passiert, wenn Du vor und hinter das 'rlc a' ein nop setzt?
Sollte das funktionieren, dann hast Du einen handfesten Hardware-Bug 
gefunden.


Gruß

Jobst

[Michael L. (eagle87)]

@ Peter:
Zur Speicherplatzeinteilung fehlen mir leider noch einige Grundlagen. 
Das hab ich bisher nur aus einer Vorlage genommen und halt versucht zu 
verstehen. Was die einzelnen Direktiven machen ist inzwischen glaub 
relativ klar. Aber bei dem Sinn der dahinter steckt haperts noch. Hast 
du dazu vielleicht einen guten Link oder ähnliches. Hab im Netz noch 
nichts wirklich hilfreiches gefunden.
rseg bedeutet doch reloctable segment, also verschiebbares Segment. 
Sucht der Linker dann nicht selbst eine passende Adresse aus? Die 
Direktive at hab ich bisher nur in Kombination mit cseg (code segment) 
gesehen. Daher dachte ich dass rseg für variable Adressen ist und cseg 
at für feste Adressen.
Den Stack hinter den Programmcode zu hängen werd ich gleich mal 
ausprobieren.

@Jobst:
Ein nop vor und hinter dem rlc bringt nichts. Aber ich konnte das 
Problem noch ein bischen eingrenzen. Wenn ich nur den Befehl mov nb0,a 
also die Zeile nach rlc und vor "weiterer Code"auskommentiere werden die 
Bits nicht gelöscht. Das wär also in dem von dir genannten Codesegment.

Gruß
Michael

[Jobst M. (jobstens-de)]

Okay, von welchen Bits, die gelöscht werden redest Du eigentlich?

Mit 'mov nb0,a' überschreibst Du den Inhalt von nb0 mit dem vom Akku.

Natürlich ist der vorherige Inhalt von nb0 dann nicht mehr da ...


Gruß

Jobst

[Michael L. (eagle87)]

Also, da sollt ich wohl doch etwas genauer werden. Der Codeblock 
"Nennerbyte 0 auf Port 0 ausgeben" ist dazu da um den Wert von nb0 
anzuzeigen. Zwischen diesem Open-Drain-Port und VCC hängen LEDs mit 
Vorwiderstand. Deswegen auch cpl. Darauf dass alle Bits 0 sind bin ich 
gekommen weil keine LED leuchtet.

Der Befehl "mov nb0,a" muss sein, da es eim eigentlichen Programm (3 
Byte Division) außer nb0 noch nb1 und nb2 gibt und die Werte im Programm 
immer wieder gebraucht werden. Eigentlich sollen die Bits in nb0 rotiert 
werden, aber das geht ja nicht direkt.

Wenn ich das gesamte Programm / den Programmablaufplan posten soll 
einfach melden.

Gruß
Michael

[Peter D. (peda)]

Las es dochmal im Simulator laufen.
Google: sim51


Peter

[Jobst M. (jobstens-de)]

Naja, Dein '#0A6h' ist nach 8 Durchläufen aus nb0 herausgeschoben. Was 
meinst Du wie lange der Prozessor dafür benötigt? Meinst Du, daß Dein 
Auge schnell genug dafür ist?


Gruß

Jobst

[Michael L. (eagle87)]

So ein blöder Fehler...
Klar dass es dann nicht funktionieren kann. Ich hätt einfach die 
Wertzuweisung hinter oder die Rotation vor die main Marke setzen müssen.

Das eigentliche Unterprogramm für das der Test war funktioniert zwar 
immer noch nicht richtig. Doch zumindest krieg ich nichtmehr lauter 
Nullen. Aber ich versuchs besser erstmal selbst hinzukriegen.

Danke für die Hilfe

Gruß
Michael