mmc: Align MMC_TRACE with tiny printf

The tiny printf implementation only supports %x format specifier,
it does not support %X . Since it makes little difference whether
the debug output prints hex numbers in capitals or not, change it
to %x and make the MMC_TRACE output work with tiny printf too.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Simon Glass <sjg@chromium.org>
This commit is contained in:
Marek Vasut 2019-03-23 18:54:45 +01:00 committed by Tom Rini
parent 00717b00a5
commit 7d5ccb1ae7

View file

@ -67,7 +67,7 @@ __weak int board_mmc_getcd(struct mmc *mmc)
void mmmc_trace_before_send(struct mmc *mmc, struct mmc_cmd *cmd) void mmmc_trace_before_send(struct mmc *mmc, struct mmc_cmd *cmd)
{ {
printf("CMD_SEND:%d\n", cmd->cmdidx); printf("CMD_SEND:%d\n", cmd->cmdidx);
printf("\t\tARG\t\t\t 0x%08X\n", cmd->cmdarg); printf("\t\tARG\t\t\t 0x%08x\n", cmd->cmdarg);
} }
void mmmc_trace_after_send(struct mmc *mmc, struct mmc_cmd *cmd, int ret) void mmmc_trace_after_send(struct mmc *mmc, struct mmc_cmd *cmd, int ret)
@ -83,21 +83,21 @@ void mmmc_trace_after_send(struct mmc *mmc, struct mmc_cmd *cmd, int ret)
printf("\t\tMMC_RSP_NONE\n"); printf("\t\tMMC_RSP_NONE\n");
break; break;
case MMC_RSP_R1: case MMC_RSP_R1:
printf("\t\tMMC_RSP_R1,5,6,7 \t 0x%08X \n", printf("\t\tMMC_RSP_R1,5,6,7 \t 0x%08x \n",
cmd->response[0]); cmd->response[0]);
break; break;
case MMC_RSP_R1b: case MMC_RSP_R1b:
printf("\t\tMMC_RSP_R1b\t\t 0x%08X \n", printf("\t\tMMC_RSP_R1b\t\t 0x%08x \n",
cmd->response[0]); cmd->response[0]);
break; break;
case MMC_RSP_R2: case MMC_RSP_R2:
printf("\t\tMMC_RSP_R2\t\t 0x%08X \n", printf("\t\tMMC_RSP_R2\t\t 0x%08x \n",
cmd->response[0]); cmd->response[0]);
printf("\t\t \t\t 0x%08X \n", printf("\t\t \t\t 0x%08x \n",
cmd->response[1]); cmd->response[1]);
printf("\t\t \t\t 0x%08X \n", printf("\t\t \t\t 0x%08x \n",
cmd->response[2]); cmd->response[2]);
printf("\t\t \t\t 0x%08X \n", printf("\t\t \t\t 0x%08x \n",
cmd->response[3]); cmd->response[3]);
printf("\n"); printf("\n");
printf("\t\t\t\t\tDUMPING DATA\n"); printf("\t\t\t\t\tDUMPING DATA\n");
@ -107,12 +107,12 @@ void mmmc_trace_after_send(struct mmc *mmc, struct mmc_cmd *cmd, int ret)
ptr = (u8 *)&cmd->response[i]; ptr = (u8 *)&cmd->response[i];
ptr += 3; ptr += 3;
for (j = 0; j < 4; j++) for (j = 0; j < 4; j++)
printf("%02X ", *ptr--); printf("%02x ", *ptr--);
printf("\n"); printf("\n");
} }
break; break;
case MMC_RSP_R3: case MMC_RSP_R3:
printf("\t\tMMC_RSP_R3,4\t\t 0x%08X \n", printf("\t\tMMC_RSP_R3,4\t\t 0x%08x \n",
cmd->response[0]); cmd->response[0]);
break; break;
default: default:
@ -226,7 +226,7 @@ int mmc_send_status(struct mmc *mmc, int timeout)
if (cmd.response[0] & MMC_STATUS_MASK) { if (cmd.response[0] & MMC_STATUS_MASK) {
#if !defined(CONFIG_SPL_BUILD) || defined(CONFIG_SPL_LIBCOMMON_SUPPORT) #if !defined(CONFIG_SPL_BUILD) || defined(CONFIG_SPL_LIBCOMMON_SUPPORT)
pr_err("Status Error: 0x%08X\n", pr_err("Status Error: 0x%08x\n",
cmd.response[0]); cmd.response[0]);
#endif #endif
return -ECOMM; return -ECOMM;