151824ba04e10322f31a165308f2e450383152fc
[hwmod.git] / doc / review / designspec.txt
1 1. Korrektheit: Enthält die Spezifikation inhaltiche Fehler, widersprüchliche Aussagen oder werden falsche Annahmen getroffen? Vergeben Sie eine Note (1-5) und begründen Sie!
2
3 2. Vollständigkeit: Enthält das Spezifikations-Dokument alle wesentlichen Requirements/Module des Designs? Vergeben Sie eine Note (1-5) und begründen Sie!
4
5 Mir persoenlich fehlen die Zustandsmaschinen. Ich wuerde vorschlagen hier
6 entweder Grafen oder Tabellen zu nehmen. Die interfaces sind grundsaetzlich
7 vorhanden und tabellarisch fest gehalten, jedoch wird nicht spezifiziert wie
8 diese werte zu verwenden sind. Zum Beispiel STack hat ein signal enable, aber
9 ist der nun High- oder Low-Aktiv?
10
11 Ist das poppen des Stacks destruktiv? wenn ja wie wird die ausgabe dann
12 produziert? Beim ASCII Stack bin ich mir nicht sicher ob das so gemeint ist und
13 ob das auch gut ist. beim Operanden stack ist diesees verhalten jedoch
14 gewuenscht.
15
16 Die Grafik koennte man entweder weglassen oder ueberarbeiten, kann meines
17 erachtens auch eine gute handzeichnung sein, solange klar ist was passiert.
18
19 divisor dividend sind output signale?
20 und wenns muldiv heisst wieso isses dann immer nur ein operand und operator?
21 (vermutlich irgendwie stack basiert aber wie greift der auf den stack zu.)
22
23 wichtig ist finde ich nicht nur wer alles zugriff auf den stack hat sondern
24 auch, ob der zugriff auf den stack gleichzeitig geschehen darf. und wie der
25 zugriff auf den stack zum beispiel passiert, es wird nirgends erwaehnt.
26
27 das interface vom stack fehlt einfach total.
28
29 3. Verständlichkeit: Ist das Dokument klar und verständlich geschrieben? Besitzt das Dokument eine ordentliche Struktur? Vergeben Sie eine Note (1-5) und begründen Sie!
30
31 Grundssaetzlich ja das zusammenspiel einiger komponenten ist mir kurz gesagt
32 unklar. vorallem im bezug auf die stacks. ich nehme an der lese meint es wird
33 zwei stacks geben und immer wenn ein mal eingelesen wird kommt es auf einen
34 priorisierten stack und immer wenn ein plus eingelesen wird kommt es auf einen
35 anderen bzw. wird der stack von a nach b verschoben. klar kommt das aus dem
36 dokument nicht heraus. auch unklar ist wie der decoder mit dem ascii stack und
37 dem parser zusammenarbeiten. vermutlich ist die zeichnung unpassend vorallem
38 weil womoeglich leitungen mit zugriff verwechselt wurden.
39
40 4. Gesamtbeurteilung: Vergeben Sie eine Gesamtnote (1-5) und geben Sie Ihren Kollegen ein kurzes Feedback über die Qualität ihres Spezifikations-Dokumentes!
41
42 man erkennt grundsaetzliche komponenten aufteilung jedoch sind einige dinge
43 glaube ich so wie sie drinnen stehtn nicht so realisierbar wie vorgestellt.
44 insbesondere wuerde ich gerne den stack zugriff sehen, also wie ist das
45 interface des stacks, um feststellen zu koennen ob das funktioniert.