Hi Peng, > On Mar 19, 2015, at 10:22 , Peng Fan <peng....@freescale.com> wrote: > > If in mmc_send_op_cond, OCR_BUSY is set in CMD1's response, then > state is transfered to Ready state, and there is no need to send > CMD1 again. Otherwise following CMD1 will recieve no response, or > timeour error from driver such as fsl_esdhc.c. > > If not into Ready state in previous CMD1, then continue CMD1 command. > > In mmc_complete_op_cond, we use the value mmc->op_cond_response > from mmc_send_op_cond, since there should be no CMD1 command between > mmc_send_op_cond and mmc_complete_op_cond > > Before fixing this, uboot log shows: > " > CMD_SEND:0 > ARG 0x00000000 > MMC_RSP_NONE > CMD_SEND:8 > ARG 0x000001AA > MMC_RSP_R1,5,6,7 0x18EC1504 > CMD_SEND:55 > ARG 0x00000000 > MMC_RSP_R1,5,6,7 0x18EC1504 > CMD_SEND:0 > ARG 0x00000000 > MMC_RSP_NONE > CMD_SEND:1 > ARG 0x00000000 > MMC_RSP_R3,4 0x00FF8080 > CMD_SEND:1 > ARG 0x40300000 > MMC_RSP_R3,4 0xC0FF8080 --> Already OCR_BUSY set > CMD_SEND:1 > ARG 0x40300000 > MMC_RSP_R3,4 0x0096850A --> Failed CMD1 > MMC init failed > " > > Using this patch, this issue is fixed, emmc can be detected correctly. > > > Signed-off-by: Peng Fan <peng....@freescale.com> > --- > drivers/mmc/mmc.c | 13 +++++++++++-- > 1 file changed, 11 insertions(+), 2 deletions(-) >
Thanks for the patch, but Andrew’s patchset is more complete. I’ll pick up his fix instead. Regards — Pantelis _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
