summaryrefslogtreecommitdiffstats
path: root/drivers/mmc/host/msm_sdcc.c
diff options
context:
space:
mode:
authorAlexander Tarasikov2011-08-21 13:52:43 +0200
committerChris Ball2011-10-26 21:43:36 +0200
commit435f3e385962e2b34855e9b34f8b95717c1016a2 (patch)
tree2107757d2bc047fce1880a042dfd7e436d4bab46 /drivers/mmc/host/msm_sdcc.c
parentARM: shmobile: ag5evm, ap4: Named SDHI IRQ sources (diff)
downloadkernel-qcow2-linux-435f3e385962e2b34855e9b34f8b95717c1016a2.tar.gz
kernel-qcow2-linux-435f3e385962e2b34855e9b34f8b95717c1016a2.tar.xz
kernel-qcow2-linux-435f3e385962e2b34855e9b34f8b95717c1016a2.zip
mmc: msm_sdcc: Fix a typo in MSM SDCC driver gpio setup
The use of && instead of || caused a NULL pointer dereference if gpio setup was not passed via platform data Signed-off-by: Alexander Tarasikov <alexander.tarasikov@gmail.com> Acked-by: Sahitya Tummala <stummala@codeaurora.org> Signed-off-by: David Brown <davidb@codeaurora.org> Signed-off-by: Chris Ball <cjb@laptop.org>
Diffstat (limited to 'drivers/mmc/host/msm_sdcc.c')
-rw-r--r--drivers/mmc/host/msm_sdcc.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/msm_sdcc.c b/drivers/mmc/host/msm_sdcc.c
index a4c865a5286b..d06539dff7cc 100644
--- a/drivers/mmc/host/msm_sdcc.c
+++ b/drivers/mmc/host/msm_sdcc.c
@@ -939,7 +939,7 @@ static void msmsdcc_setup_gpio(struct msmsdcc_host *host, bool enable)
struct msm_mmc_gpio_data *curr;
int i, rc = 0;
- if (!host->plat->gpio_data && host->gpio_config_status == enable)
+ if (!host->plat->gpio_data || host->gpio_config_status == enable)
return;
curr = host->plat->gpio_data;