On 04/25/2011 10:34 PM, Sahitya Tummala wrote: > @@ -659,8 +662,13 @@ msmsdcc_pio_irq(int irq, void *dev_id) > { > struct msmsdcc_host *host = dev_id; > uint32_t status; > + u32 mci_mask0 = 0; Is this assignment there to silence an "unused variable" warning? If it is, please use the uninitialized_var macro. If it isn't please remove the assignment. > > status = msmsdcc_readl(host, MMCISTATUS); > + mci_mask0 = msmsdcc_readl(host, MMCIMASK0); > + > + if (((mci_mask0 & status) & MCI_IRQ_PIO) == 0) > + return IRQ_NONE; > > do { > unsigned long flags; -- 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