From 8f170b0e40cd3c7edc7a4bad0671caa8ff214dbd Mon Sep 17 00:00:00 2001
From: Mark Pizzolato <mark@infocomm.com>
Date: Thu, 31 Jan 2013 16:18:38 -0800
Subject: [PATCH] Fixed modifier table to properly declare things which are
 showable vs settable, added modifier descriptions as appropriate

---
 PDP11/pdp11_xq.c | 28 ++++++++++++++--------------
 PDP11/pdp11_xu.c | 16 ++++++++--------
 2 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/PDP11/pdp11_xq.c b/PDP11/pdp11_xq.c
index 1581cd66..bf18487c 100644
--- a/PDP11/pdp11_xq.c
+++ b/PDP11/pdp11_xq.c
@@ -438,17 +438,17 @@ REG xqb_reg[] = {
 
 MTAB xq_mod[] = {
   { MTAB_XTD|MTAB_VDV, 004, "ADDRESS", NULL,
-    NULL, &show_addr, NULL },
+    NULL, &show_addr, NULL, "Qbus address" },
   { MTAB_XTD|MTAB_VDV, 0, "VECTOR", NULL,
-    NULL, &show_vec, NULL },
+    NULL, &show_vec, NULL,  "Interrupt vector" },
   { MTAB_XTD | MTAB_VDV, 0, "MAC", "MAC=xx:xx:xx:xx:xx:xx",
-    &xq_setmac, &xq_showmac, NULL },
-  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "ETH", "ETH",
-    NULL, &eth_show, NULL },
-  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "FILTERS", "FILTERS",
-    NULL, &xq_show_filters, NULL },
+    &xq_setmac, &xq_showmac, NULL, "MAC address" },
+  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "ETH", NULL,
+    NULL, &eth_show, NULL, "Display attachable devices" },
+  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "FILTERS", NULL,
+    NULL, &xq_show_filters, NULL, "Display address filters" },
   { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "STATS", "STATS",
-    &xq_set_stats, &xq_show_stats, NULL },
+    &xq_set_stats, &xq_show_stats, NULL, "Display or reset statistics" },
   { MTAB_XTD | MTAB_VDV, 0, "TYPE", "TYPE={DEQNA|DELQA|DELQA-T}",
     &xq_set_type, &xq_show_type, NULL },
 #ifdef USE_READER_THREAD
@@ -459,9 +459,9 @@ MTAB xq_mod[] = {
     &xq_set_poll, &xq_show_poll, NULL },
 #endif
   { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "SANITY", "SANITY={ON|OFF}",
-    &xq_set_sanity, &xq_show_sanity, NULL },
-  { MTAB_XTD | MTAB_VDV , 0, "LEDS", "LEDS",
-    NULL, &xq_show_leds, NULL },
+    &xq_set_sanity, &xq_show_sanity, NULL, "Sanity timer" },
+  { MTAB_XTD | MTAB_VDV | MTAB_NMO , 0, "LEDS", NULL,
+    NULL, &xq_show_leds, NULL, "Display status LEDs" },
   { 0 },
 };
 
@@ -835,9 +835,9 @@ t_stat xq_show_leds (FILE* st, UNIT* uptr, int32 val, void* desc)
 {
   CTLR* xq = xq_unit2ctlr(uptr);
 
-  fprintf(st, "leds=(%s,%s,%s)", xq->var->setup.l1 ? "ON" : "OFF", 
-                                 xq->var->setup.l2 ? "ON" : "OFF", 
-                                 xq->var->setup.l3 ? "ON" : "OFF");
+  fprintf(st, "leds=(%s,%s,%s)\n", xq->var->setup.l1 ? "ON" : "OFF", 
+                                   xq->var->setup.l2 ? "ON" : "OFF", 
+                                   xq->var->setup.l3 ? "ON" : "OFF");
   return SCPE_OK;
 }
 
diff --git a/PDP11/pdp11_xu.c b/PDP11/pdp11_xu.c
index 8954df37..8634b756 100644
--- a/PDP11/pdp11_xu.c
+++ b/PDP11/pdp11_xu.c
@@ -157,18 +157,18 @@ MTAB xu_mod[] = {
     &set_vec, &show_vec, NULL },
 #else
   { MTAB_XTD|MTAB_VDV, 004, "ADDRESS", NULL,
-    NULL, &show_addr, NULL },
+    NULL, &show_addr, NULL, "Unibus address" },
   { MTAB_XTD|MTAB_VDV, 0, "VECTOR", NULL,
-    NULL, &show_vec, NULL },
+    NULL, &show_vec, NULL, "Interrupt vector" },
 #endif
   { MTAB_XTD | MTAB_VDV, 0, "MAC", "MAC=xx:xx:xx:xx:xx:xx",
-    &xu_setmac, &xu_showmac, NULL },
-  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "ETH", "ETH",
-    NULL, &eth_show, NULL },
+    &xu_setmac, &xu_showmac, NULL, "MAC address" },
+  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "ETH", NULL,
+    NULL, &eth_show, NULL, "Display attachable devices" },
   { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "STATS", "STATS",
-    &xu_set_stats, &xu_show_stats, NULL },
-  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "FILTERS", "FILTERS",
-    NULL, &xu_show_filters, NULL },
+    &xu_set_stats, &xu_show_stats, NULL, "Display or reset statistics" },
+  { MTAB_XTD | MTAB_VDV | MTAB_NMO, 0, "FILTERS", NULL,
+    NULL, &xu_show_filters, NULL, "Display address filters" },
   { MTAB_XTD | MTAB_VDV, 0, "TYPE", "TYPE={DEUNA|DELUA}",
     &xu_set_type, &xu_show_type, NULL },
   { 0 },