doc/review/hwmodspec.txt

   1 1. Korrektheit: Enthält die Spezifikation inhaltiche Fehler, widersprüchliche Aussagen oder werden
   2    falsche Annahmen getroffen? Vergeben Sie eine Note (1-5) und begründen Sie!
   3
   4 Wofuer sind die verschiedenen Clocks im Design da? Ein Multiclockdesign ist
   5 nicht wuenschenswert, ausserdem zu komplex? Wir glauben man sollte das durch
   6 Kontrollsignale ersetzen, also z.B. ein Requestsignal geht hoch wenn alle Daten
   7 ins Register geladen wurden und bis zur Fertigstellung der Arbeit bleibt das
   8 ACK-Signal niedrig. Es koennte sein, dass es so gemeint ist, dann ist aber die
   9 Bezeichnung '*_clk' sehr irrefuehrend gewaehlt.
  10
  11 Note: 2
  12
  13
  14 2. Vollständigkeit: Enthält das Spezifikations-Dokument alle wesentlichen Requirements/Module des
  15    Designs? Vergeben Sie eine Note (1-5) und begründen Sie!
  16
  17 > alex: ich denke das hier nicht nur die vollstaendigkeit der requirements gemeint ist sondern auch
  18 > die vollstaendigkeit der module bzw. interfaces. also ist zu jedem interface mindestens ein modul
  19 > definiert und umgekehrt. Natuerlich gehort auch dazu ob die Module einen vollstaendigen
  20 > Taschenrechner erzeugen.
  21
  22 > bernhard: aso, ich dachte das bezieht sich auf die Requirements. dann passts.
  23
  24 Requirements: passen.
  25
  26
  27 Module:
  28 Ueberblicksmaessig wirkt es recht gut (abgesehen von den
  29 '*_clk'-Signalen), aber in der "Detailed Design Description" fehlen eben die
  30 Details.
  31 Ablaeufe werden zwar textuell kurz erklaert, aber sie werden nur sehr
  32 oberflaechig und abstrakt "angekratzt". Ausserdem fehlen grafische
  33 Darstellungen (z.B. State-Maschinen). Weiters waere eine Beschreibung
  34 wuenschenswert wie ihr die eigentliche Berechnung umsetzen wollt (z.B. mit
  35 Hilfe von Pseudocode). Da muss einfach noch viel gemacht werden!
  36
  37
  38 Testfaelle: Sind sehr gut!
  39
  40
  41 Note: 3
  42
  43
  44 3. Verständlichkeit: Ist das Dokument klar und verständlich geschrieben? Besitzt das Dokument eine
  45    ordentliche Struktur? Vergeben Sie eine Note (1-5) und begründen Sie!
  46
  47 Ja, grundsaetzlich schon, bis auf die Schnittstellenbeschreibung.
  48 Verbesserungsvorschlaege:
  49 [list]
  50 [*]
  51 eine Spalte zu welchen Module (oder externes Geraet, z.B. PS/2) die Signale
  52 fuehren.
  53 [/*]
  54 [*]
  55 einheitlichere Beschriftung der Signale (z.B. in "Calculator State Maschine",
  56 wo fuehren die Signale "result_data" bzw. "result_clk" hin? Das ist am ersten
  57 Blick nicht klar ersichtlich)
  58 [/*]
  59 [*]
  60 und wie schon angesprochen: die divisieren '*_clk'-Signale verwirren.
  61 [/*]
  62 [/list]
  63
  64 Note: 2
  65
  66
  67 4. Gesamtbeurteilung: Vergeben Sie eine Gesamtnote (1-5) und geben Sie Ihren Kollegen ein kurzes
  68    Feedback über die Qualität ihres Spezifikations-Dokumentes!
  69
  70 Wenn die nicht naeher beschriebenen Module funktionieren, schaut das Gesamtdesign recht
  71 zuversichtlich aus. (Sofern das mit den mehreren Clocksignalen beseitigt wird).
  72
  73 Die "Detailed Design Description" muss unbedingt noch ueberarbeitet werden.
  74
  75 Note: 2
  76