On Mon, Feb 21, 2011 at 05:54:55PM +0800, Richard Zhao wrote: > + if (host->quirks & SDHCI_QUIRK_FIX_NO_INT_IN_MULTI_BLK_IO) { > + if (readl(host->ioaddr + SDHCI_VENDOR_SPEC) & 0x2) > + writel(readl(host->ioaddr + SDHCI_VENDOR_SPEC) & ~0x2, > + host->ioaddr + SDHCI_VENDOR_SPEC); Is it necessary to read the register twice, or would: u32 val = readl(host->ioaddr + SDHCI_VENDOR_SPEC); if (val & 0x02) writel(val & ~0x02, host->ioaddr + SDHCI_VENDOR_SPEC); be clearer and more obvious? -- 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