Commit 699b958b authored by Adrian Hunter's avatar Adrian Hunter Committed by Chris Ball

mmc: omap_hsmmc: correct debug report error status mnemonics

CERR and BADA were in the wrong place and there are only
32 not 35.
Signed-off-by: default avatarAdrian Hunter <adrian.hunter@nokia.com>
Reviewed-by: default avatarVenkatraman S <svenkatr@ti.com>
Signed-off-by: default avatarChris Ball <cjb@laptop.org>
parent 393f9a08
...@@ -975,14 +975,14 @@ static void omap_hsmmc_dma_cleanup(struct omap_hsmmc_host *host, int errno) ...@@ -975,14 +975,14 @@ static void omap_hsmmc_dma_cleanup(struct omap_hsmmc_host *host, int errno)
* Readable error output * Readable error output
*/ */
#ifdef CONFIG_MMC_DEBUG #ifdef CONFIG_MMC_DEBUG
static void omap_hsmmc_report_irq(struct omap_hsmmc_host *host, u32 status) static void omap_hsmmc_dbg_report_irq(struct omap_hsmmc_host *host, u32 status)
{ {
/* --- means reserved bit without definition at documentation */ /* --- means reserved bit without definition at documentation */
static const char *omap_hsmmc_status_bits[] = { static const char *omap_hsmmc_status_bits[] = {
"CC", "TC", "BGE", "---", "BWR", "BRR", "---", "---", "CIRQ", "CC" , "TC" , "BGE", "---", "BWR" , "BRR" , "---" , "---" ,
"OBI", "---", "---", "---", "---", "---", "ERRI", "CTO", "CCRC", "CIRQ", "OBI" , "---", "---", "---" , "---" , "---" , "ERRI",
"CEB", "CIE", "DTO", "DCRC", "DEB", "---", "ACE", "---", "CTO" , "CCRC", "CEB", "CIE", "DTO" , "DCRC", "DEB" , "---" ,
"---", "---", "---", "CERR", "CERR", "BADA", "---", "---", "---" "ACE" , "---" , "---", "---", "CERR", "BADA", "---" , "---"
}; };
char res[256]; char res[256];
char *buf = res; char *buf = res;
...@@ -999,6 +999,11 @@ static void omap_hsmmc_report_irq(struct omap_hsmmc_host *host, u32 status) ...@@ -999,6 +999,11 @@ static void omap_hsmmc_report_irq(struct omap_hsmmc_host *host, u32 status)
dev_dbg(mmc_dev(host->mmc), "%s\n", res); dev_dbg(mmc_dev(host->mmc), "%s\n", res);
} }
#else
static inline void omap_hsmmc_dbg_report_irq(struct omap_hsmmc_host *host,
u32 status)
{
}
#endif /* CONFIG_MMC_DEBUG */ #endif /* CONFIG_MMC_DEBUG */
/* /*
...@@ -1057,9 +1062,7 @@ static void omap_hsmmc_do_irq(struct omap_hsmmc_host *host, int status) ...@@ -1057,9 +1062,7 @@ static void omap_hsmmc_do_irq(struct omap_hsmmc_host *host, int status)
dev_dbg(mmc_dev(host->mmc), "IRQ Status is %x\n", status); dev_dbg(mmc_dev(host->mmc), "IRQ Status is %x\n", status);
if (status & ERR) { if (status & ERR) {
#ifdef CONFIG_MMC_DEBUG omap_hsmmc_dbg_report_irq(host, status);
omap_hsmmc_report_irq(host, status);
#endif
if ((status & CMD_TIMEOUT) || if ((status & CMD_TIMEOUT) ||
(status & CMD_CRC)) { (status & CMD_CRC)) {
if (host->cmd) { if (host->cmd) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment