diff options
| author | Philippe De Swert | 2012-04-11 22:32:05 +0200 |
|---|---|---|
| committer | Chris Ball | 2012-05-10 02:47:49 +0200 |
| commit | f6f44590f231cdf914ce740f3a53a6b815bf4767 (patch) | |
| tree | 9c530896e81e0f7afbb413a12496ff05ca02d70d /drivers/mmc | |
| parent | mmc: card: Avoid null pointer dereference (diff) | |
| download | kernel-qcow2-linux-f6f44590f231cdf914ce740f3a53a6b815bf4767.tar.gz kernel-qcow2-linux-f6f44590f231cdf914ce740f3a53a6b815bf4767.tar.xz kernel-qcow2-linux-f6f44590f231cdf914ce740f3a53a6b815bf4767.zip | |
mmc: core: Remove dead code
bus_width is passed to the function and when 0 (MMC_BUS_WIDTH_1)
will cause the function to return. So in in the second test it
definitely is different from 0, and the third test is redundant.
Signed-off-by: Philippe De Swert <philippedeswert@gmail.com>
Reviewed-by: Namjae Jeon <linkinjeon@gmail.com>
Acked-by: Philip Rakity <prakity@marvell.com>
Signed-off-by: Chris Ball <cjb@laptop.org>
Diffstat (limited to 'drivers/mmc')
| -rw-r--r-- | drivers/mmc/core/mmc.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c index 163046030773..2f0e11c9fd67 100644 --- a/drivers/mmc/core/mmc.c +++ b/drivers/mmc/core/mmc.c @@ -539,14 +539,10 @@ static int mmc_compare_ext_csds(struct mmc_card *card, unsigned bus_width) err = mmc_get_ext_csd(card, &bw_ext_csd); if (err || bw_ext_csd == NULL) { - if (bus_width != MMC_BUS_WIDTH_1) - err = -EINVAL; + err = -EINVAL; goto out; } - if (bus_width == MMC_BUS_WIDTH_1) - goto out; - /* only compare read only fields */ err = !((card->ext_csd.raw_partition_support == bw_ext_csd[EXT_CSD_PARTITION_SUPPORT]) && |