[src/trunk]: src/sys/dev/sdmmc Be more verbose about error conditions.

[mlelstv <mlelstv%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/172ab7bee5d0
branches:  trunk
changeset: 339645:172ab7bee5d0
user:      mlelstv <mlelstv%NetBSD.org@localhost>
date:      Sun Aug 02 07:14:10 2015 +0000

description:
Be more verbose about error conditions.

diffstat:

 sys/dev/sdmmc/sdhc.c |  14 +++++++++-----
 1 files changed, 9 insertions(+), 5 deletions(-)

diffs (50 lines):

diff -r b59aef86f43e -r 172ab7bee5d0 sys/dev/sdmmc/sdhc.c
--- a/sys/dev/sdmmc/sdhc.c      Sun Aug 02 07:07:02 2015 +0000
+++ b/sys/dev/sdmmc/sdhc.c      Sun Aug 02 07:14:10 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: sdhc.c,v 1.67 2015/08/02 07:07:02 mlelstv Exp $        */
+/*     $NetBSD: sdhc.c,v 1.68 2015/08/02 07:14:10 mlelstv Exp $        */
 /*     $OpenBSD: sdhc.c,v 1.25 2009/01/13 19:44:20 grange Exp $        */
 
 /*
@@ -23,7 +23,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: sdhc.c,v 1.67 2015/08/02 07:07:02 mlelstv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: sdhc.c,v 1.68 2015/08/02 07:14:10 mlelstv Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_sdmmc.h"
@@ -991,6 +991,8 @@
         */
        if (!sdhc_clock_divisor(hp, freq, &div)) {
                /* Invalid base clock frequency or `freq' value. */
+               aprint_error_dev(hp->sc->sc_dev,
+                       "Invalid bus clock %d kHz\n", freq);
                error = EINVAL;
                goto out;
        }
@@ -1163,8 +1165,8 @@
                        return 0;
                sdmmc_delay(10);
        }
-       DPRINTF(0,("%s: timeout waiting for %x (state=%x)\n", HDEVNAME(hp),
-           value, state));
+       aprint_error_dev(hp->sc->sc_dev, "timeout waiting for %x (state=%x)\n",
+           value, state);
        return ETIMEDOUT;
 }
 
@@ -1344,8 +1346,10 @@
 
        /* Wait until command and data inhibit bits are clear. (1.5) */
        error = sdhc_wait_state(hp, SDHC_CMD_INHIBIT_MASK, 0);
-       if (error)
+       if (error) {
+               aprint_error_dev(sc->sc_dev, "command or data phase inhibited\n");
                return error;
+       }
 
        DPRINTF(1,("%s: writing cmd: blksize=%d blkcnt=%d mode=%04x cmd=%04x\n",
            HDEVNAME(hp), blksize, blkcount, mode, command));