From: Alexander Tarasikov Date: Sun, 21 Aug 2011 11:52:43 +0000 (+0400) Subject: mmc: msm_sdcc: Fix a typo in MSM SDCC driver gpio setup X-Git-Tag: v3.2-rc1~129^2~60 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=435f3e385962e2b34855e9b34f8b95717c1016a2;p=profile%2Fivi%2Fkernel-adaptation-intel-automotive.git 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 Acked-by: Sahitya Tummala Signed-off-by: David Brown Signed-off-by: Chris Ball --- diff --git a/drivers/mmc/host/msm_sdcc.c b/drivers/mmc/host/msm_sdcc.c index a4c865a..d06539d 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;