Dear Wolfgang Denk, On 25 October 2010 13:19, Jaehoon Chung <[email protected]> wrote: > This patch use card's tran_speed instead of hard setting value. > I think mmc_set_clock(mmc, 52000000) is not good idea. > because this is hard setting. we need use card's tran_speed. > > So If card_caps did't support High speed, we need set card's speed value > > > Signed-off-by: Jaehoon Chung <[email protected]> > Signed-off-by: Minkyu Kang <[email protected]> > Signed-off-by: Kyungmin Park <[email protected]> > > > --- > drivers/mmc/mmc.c | 9 +++++---- > 1 files changed, 5 insertions(+), 4 deletions(-) > > diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c > index c543d83..f1b5552 100644 > --- a/drivers/mmc/mmc.c > +++ b/drivers/mmc/mmc.c > @@ -815,11 +815,12 @@ int mmc_startup(struct mmc *mmc) > > if (mmc->card_caps & MMC_MODE_HS) { > if (mmc->card_caps & MMC_MODE_HS_52MHz) > - mmc_set_clock(mmc, 52000000); > + mmc->tran_speed = 52000000; > else > - mmc_set_clock(mmc, 26000000); > - } else > - mmc_set_clock(mmc, 20000000); > + mmc->tran_speed = 26000000; > + } > + > + mmc_set_clock(mmc, mmc->tran_speed); > } > > /* fill in device description */ > _______________________________________________ > U-Boot mailing list > [email protected] > http://lists.denx.de/mailman/listinfo/u-boot >
This patch has already been reviewed by Andy. http://lists.denx.de/pipermail/u-boot/2010-May/071213.html Thanks Minkyu Kang -- from. prom. www.promsoft.net _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
