ahci: drop debug and reformat ahciidentify dump
authorFergus Simpson <afergs@google.com>
Fri, 27 Jan 2017 23:34:28 +0000 (15:34 -0800)
committerBarret Rhoden <brho@cs.berkeley.edu>
Mon, 30 Jan 2017 17:03:28 +0000 (12:03 -0500)
Disables debug prints and shortens the ahciidentfy0 hex dump. Reduces
clutter in the terminal. Also required a bugfix in the call to the
printd macro where a semicolon in a call had to be dropped.

Change-Id: I08a2823c5c16ff70e796678b809e36763aa7f390
Signed-off-by: Fergus Simpson <afergs@google.com>
Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/drivers/dev/sdiahci.c

index 746d210..24d5473 100644 (file)
@@ -1,4 +1,3 @@
-#define DEBUG
 /*
  * This file is part of the UCB release of Plan 9. It is subject to the license
  * terms in the LICENSE file found in the top-level directory of this
@@ -652,13 +651,16 @@ static int64_t ahciidentify(struct aportc *pc, uint16_t *id)
        }
        if (ahciidentify0(pc, id, i) == -1)
                return -1;
+
+#ifdef DEBUG
        printd("ahci: %s: ahciidentify0 return dump=\n\t", __func__);
-       for (cnt = 0; cnt < 101; cnt++) {
+       for (cnt = 0; cnt < 64; cnt++) {
                printd("0x%08x  ", id[cnt]);
-               if (cnt % 5 == 4 && cnt != 100)
+               if (cnt % 4 == 3 && cnt != 63)
                        printd("\n\t");
        }
        printd("\n");
+#endif
 
        i = gbit16(id + 83) | gbit16(id + 86);
        if (i & (1 << 10)) {
@@ -1284,7 +1286,7 @@ static int doportreset(struct drive *d)
        i = -1;
        qlock(&d->portm.ql);
        if (ahciportreset(&d->portc) == -1)
-               printd("ahci: doportreset: fails\n");
+               printd("ahci: doportreset: fails\n")
        else
                i = 0;
        qunlock(&d->portm.ql);