mmc: remove the hard setting for tran_speed

mmc_set_clock is set to the hard-coding.
But i think good that use the tran_speed value.

Signed-off-by Jaehoon Chung <[email protected]>

Signed-off-by Kyungmin Park <[email protected]>

---
?drivers/mmc/mmc.c | ? 13 +++++++------
?1 files changed, 7 insertions(+), 6 deletions(-)

drivers/mmc/mmc.c +1 -1

index e035012..cba96cf 100644
@@ -1124,9 +1124,9 @@ int mmc_startup(struct mmc *mmc)
 ? ? ? ? ? ? ? ?}
 
 ? ? ? ? ? ? ? ?if (mmc->card_caps & MMC_MODE_HS)
- ? ? ? ? ? ? ? ? ? ? ? mmc_set_clock(mmc, 50000000);
+ ? ? ? ? ? ? ? ? ? ? ? mmc->tran_speed = 50000000;

? ? ? ? ? ? ? ?else
- ? ? ? ? ? ? ? ? ? ? ? mmc_set_clock(mmc, 25000000);
+ ? ? ? ? ? ? ? ? ? ? ? mmc->tran_speed = 25000000;
? ? ? ?} else {
? ? ? ? ? ? ? ?for (width = EXT_CSD_BUS_WIDTH_8; width >= 0; width--) { ? ? ? ? ? ? ? ? ? ? ? ?/* Set the card to use 4 bit*/

drivers/mmc/mmc.c +1 -1

@@ -1161,13 +1161,14 @@ 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 */
? ? ? ?mmc->block_dev.lun = 0;
? ? ? ?mmc->block_dev.type = 0;
--
1.7.4.1
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Quoted 10 lines

? ? ? ? ? ? ? ?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;
+ ? ? ? ? ? ? ? }

Quoted 10 lines · Jae hoon Chung

drivers/mmc/mmc.c int mmc_startup(struct mmc *mmc) · line 1167

Quoted 10 lines

               if (mmc->card_caps & MMC_MODE_HS) {
                       if (mmc->card_caps & MMC_MODE_HS_52MHz)

drivers/mmc/mmc.c +3 -4

-                               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;
+               }

Why did you remove the outer else clause, here (the one that set the speed to 20000000)?

Hi Andy.

Quoted 14 lines · Andy Fleming

Quoted 10 lines · Jae hoon Chung

Quoted 10 lines

? ? ? ? ? ? ? ?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;
+ ? ? ? ? ? ? ? }

Quoted 3 lines

Why did you remove the outer else clause, here (the one that set the speed to 20000000)?

If card->caps didn't set MMC_MODE_HS, then it's set to mmc->tran_speed. That value is assigned from freq * mult.

I think that is reasonable..if set to 20000000 here, then why need freq & mult?