On Sun, Aug 21, 2011 at 03:52:43PM +0400, Alexander Tarasikov wrote: > 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@xxxxxxxxx> > --- > drivers/mmc/host/msm_sdcc.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > 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; Both of your patches have somehow gotten mangled in transit (the context lines have lost their leading space). Did you perhaps edit the patch files with an editor that is configured to fold spaces into tabs? I was able to fix up the files and apply the patches, and will include these once Sahitya has a chance to review them. Thanks, David -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
