diff options
| author | Diego Santa Cruz <Diego.SantaCruz@spinetix.com> | 2014-12-23 10:50:28 +0100 |
|---|---|---|
| committer | Pantelis Antoniou <pantelis.antoniou@konsulko.com> | 2015-01-19 17:04:48 +0200 |
| commit | 9cf199ebcff13f8d98b31ab48e3bcce37ed8925d (patch) | |
| tree | 34f4b537c8170fee11fa5a191d0d69d267139d8e /drivers/mmc/mmc.c | |
| parent | 8a0cf4901021b09c283f172d4d29d0d05721b0ba (diff) | |
| download | u-boot-imx-9cf199ebcff13f8d98b31ab48e3bcce37ed8925d.zip u-boot-imx-9cf199ebcff13f8d98b31ab48e3bcce37ed8925d.tar.gz u-boot-imx-9cf199ebcff13f8d98b31ab48e3bcce37ed8925d.tar.bz2 | |
mmc: the ext_csd data may be used during init even if reading failed
The mmc_startup() function uses the ext_csd data even if reading it
from the mmc device failed. This bug was introduced in commit
bc897b1d4d86597311430dbe7b3e6c807c8c53e5. We now bail out if
reading it fails, this should not be a problem as ext_csd was
introduced in MMC 4.0 and this code is conditional on MMC >= 4.0.
Signed-off-by: Diego Santa Cruz <Diego.SantaCruz@spinetix.com>
Diffstat (limited to 'drivers/mmc/mmc.c')
| -rw-r--r-- | drivers/mmc/mmc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index 403843b..63a1e0c 100644 --- a/drivers/mmc/mmc.c +++ b/drivers/mmc/mmc.c @@ -972,7 +972,9 @@ static int mmc_startup(struct mmc *mmc) if (!IS_SD(mmc) && (mmc->version >= MMC_VERSION_4)) { /* check ext_csd version and capacity */ err = mmc_send_ext_csd(mmc, ext_csd); - if (!err && (ext_csd[EXT_CSD_REV] >= 2)) { + if (err) + return err; + if (ext_csd[EXT_CSD_REV] >= 2) { /* * According to the JEDEC Standard, the value of * ext_csd's capacity is valid if the value is more |