On Thu, Aug 22, 2013 at 08:16:28PM +0530, Balaji T K wrote: > On Thursday 22 August 2013 06:26 PM, Dan Carpenter wrote: > >'!' has higher precedence than '&' so this doesn't work as intended > >although since RESETDONE is 1 it would work if none of the other bits > >are set. > > > Hi Dan, > > Thanks for the patch, Indeed other bits are reserved. > however ... > > >Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > >--- > >Untested. > > > >diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c > >index 1865321..7346b15 100644 > >--- a/drivers/mmc/host/omap_hsmmc.c > >+++ b/drivers/mmc/host/omap_hsmmc.c > >@@ -612,7 +612,7 @@ static int omap_hsmmc_context_restore(struct omap_hsmmc_host *host) > > if (host->context_loss == context_loss) > > return 1; > > > >- if (!OMAP_HSMMC_READ(host->base, SYSSTATUS) & RESETDONE) > >+ if (!(OMAP_HSMMC_READ(host->base, SYSSTATUS) & RESETDONE)) > > return 1; > > This check is unnecessary, will send a patch to remove this. What happened with this? regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
