On Saturday 19 November 2011 12:36:00 Stephen Warren wrote:
> On 01/-10/-28163 12:59 PM, Macpaul Lin wrote:
> > This reverts commit 02f3029f1810b99869254d0cf0a71946a008a728.
> >
> > This patch add 3 times retry to CMD8 because the Marvell mmc controller
> > doesn't obey the power ramp up process in the SD specification 6.4.1.
> > (Please refer to figure 6.1 and 6.2 in the specification.)
> >
> > The CMD0 should be send after power ramp up has been finished.
> > However, the Marvell mmc contorller must do power ramp up after the
> > first CMD0 command has been send.
> >
> > This patch also affect existing platforms like Nokia N900 and other
> > platforms.
> >
> > Signed-off-by: Macpaul Lin <macp...@andestech.com>
> > Acked-by: Lei Wen <lei...@marvell.com>
>
> Acked-by: Stephen Warren <swar...@nvidia.com>
> Tested-by: Stephen Warren <swar...@nvidia.com>
>
> As background, I'm working on patches to support the Toshiba AC100 /
> Dynabook AZ in mainline U-Boot. The internal MMC didn't work at all
> until this revert was applied. The SD slot worked perfectly though. I
> guess this is related to the MMC itself not supporting this
> (SD-specific?) command, and the patch caused MMC mode never to be attempted.

Same for Nokia N900 (SD slot working fine). After reverting commit
02f3029f1810b99869254d0cf0a71946a008a728 internal eMMC working too.

--
Pali Rohár
pali.ro...@gmail.com

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to