Gray Code und Zweierkomplement Addition

Hallo Mo,

wenn deine Eingangszahlen immer positiv sind, gibt es keinen Grund für 
signed.
Weiterhin gibt es keinen signed/unsigned Adder. Der typ signed oder 
unsigned wirkt sich nur darauf aus, wie ein Compiler/Syntheser (bei 
Vergleichen) oder ein Mensch eine Menge von Bits interpretiert.

Tom

Mo schrieb:
> Das hier ist die Aufgabe. Das ist das einzige Problem mit dem Ich nicht
> weiterkomme.
Du hast den Eingang A also schon vom Graycode in eine positive Binärzahl 
umgewandelt? Dann mach einfach ein resize() beider Eingangszahlen auf 
die Zielbreite (ein Bit mehr als der breiteste der beiden 
Eingangsvektoren) und addiere dann einfach beide Zahlen zusammen. Du 
erhältst eine Zahl, die du wieder als vorzeichenbehaftete 
Zweierkomplement-Zahl betrachtest. Fertig.

Im Fall des unsigned Vektors doch wohl eher so:
X  <= '0'&A;

Bei A+2B hast du zwei Additionen: A+B+B und brauchst deshalb noch ein 
weiteres Bit vornedran, damit der Wertebereich nicht überschritten wird. 
Und dieses führende Bit ist hier '0', womit das Ergebnis wieder tadellos 
passt.

Mo schrieb:
> Könnt Ihr mir einen Tipp geben wie Ich es am besten umsetzte ?
Ich würde die Zahlen einfach in ausreichend "breite" Integer umwandeln 
und  damit die gewünschten Berechungen machen und dementsprechend die 
LEDs ansteuern.

Am Rande:
Den S11, der als "Schiebetakt" dienen soll, darfst du nicht als 
"richtigen" Takt verwenden (mit rising_edge() oder 'event), sondern 
musst ihn einsynchronisieren und eine Flankenerkenung drauf machen.

Mo schrieb:
> Mit einer Vier Bit Zahl würde Ich wissen wie es geht, aber nicht mit
> einer Zahl die länger als 4 Bit ist.
Nimm die Konvertierungen und die Casts aus der numeric_std:
http://www.lothar-miller.de/s9y/categories/16-Numeric_Std

Damit koonvertierst du den Integer mit to_unsigned() zum 4 bit breiten 
Unsigned und castest diesen Unsigned zum Std_logic_vector und gibst den 
deinem Modul. Passt alles locker in 1 Zeile:

ergebnisvektor <= std_logic_vector(to_unsigned(integerwert,4));

Mo schrieb:
> weiß nicht wo der Fehler liegt.
Mach einfach mal das "begin" der Architecture vor die Zuweisungen...

Und lass das "else null;" weg und gib den LEDs Defaultwerte, oder gib in 
diesem letzten "else" den LEDs auch noch Werte. Sonst bekommst du 
Latches.

Und die Sache mit dem Absolutwert und dem extrahierten Vorzeichen 
musst du auch noch einbauen.

Und diese kuriose Verwendung des CLK mitten im Nirgendwo wird dir auch 
noch Probleme bereiten. Sieh dir einfach mal an, wie alle anderen 
einen synchronen oder einen kombinatorischen Prozess beschreiben.

Mo schrieb:
> Das Programm
VHDL ist, wie das 'D' darin sagt, keine Programmiersprache, sondern eine 
Beschreibungssprache. Wenn du damit "programmierst", wirst du laufend 
Schiffbruch erleiden.

> für die simple Rechnung funktioniert jetzt so wie es soll
Wundert mich, denn das ist ja wohl nur für positive Zahlen richtig:
"0000" & B
Du musst hier den Vektor vorzeichenkorrekt erweitern:
B(3)&B(3)&B(3)&B(3) & B
Oder das eben mit dem bereits erwähnten resize(B,8) automatisieren.


Und: übergib doch zur Klarheit den Wert von A als unsigned Vektor. Dann 
sparst du dir da die Konvertierung und weißt du auch in einem halben 
Jahr noch, was da tatsächlich übergeben wird.

Mo schrieb:
> Wenn A = "1111" und B ="1110" ist sollte ja y="0100 0000" sein.
> Es ist bei mir aber y= "0001 0000".
Bei mir weder, noch. Denn 2*15 - (-2) ergibt 32 und damit 0010_0000...

Mo schrieb:
> Woran liegt das
Deine Sensitivliste ist falsch. Da gehört nicht O rein, sondern y. Und 
das ganze nennt sich "Delta-Cycle" Problem, denn auch die Zuweisung 
"y<=O" findet nur dann statt, wenn "O" sich ändert. Gleichzeitig wird 
dann auch der Prozess berechnet, aber eben noch mit dem alten "y".

BTW, als Stichwort für die Suche mit Google: was du machen willst, ist 
eine Binär-nach-BCD Umwandlung. Sieh dazu mal den 
Beitrag "VHDL Grundlagen Binaer to BCD" und die Links darin an...