Mike Rapoport wrote: > Hi David, > > David Vrabel wrote: >> These patches add support for SDIO cards to the omap_hsmmc driver. Less >> restrictions on the sizes of transfers, power management changes to >> prevent SDIO cards from being turned off and losing all state, and card >> interrupts. >> >> I've been unable to test these exact patches as I only have an N900 for >> testing and the N900 support in mainline is incomplete. > > I've tried your patches on CM-T35 that has 8686 SDIO and the result was > absence of MMC/SDIO devices at all :( Morover, kmmcd hangs: > > [ 399.427764] INFO: task kmmcd:149 blocked for more than 120 seconds. > [...] > [ 399.485290] [<c01e6264>] (mmc_wait_for_cmd+0x64/0x74) from > [<c01e86b0>] (mmc_go_idle+0x60/0xac) I suspect interrupts aren't enabled correctly as this command does not complete. Try reverting this hunk. @@ -426,12 +431,6 @@ omap_hsmmc_start_command(struct omap_hsmmc_host *host, struct mmc_command *cmd, mmc_hostname(host->mmc), cmd->opcode, cmd->arg); host->cmd = cmd; - /* - * Clear status bits and enable interrupts - */ - OMAP_HSMMC_WRITE(host->base, STAT, STAT_CLEAR); - OMAP_HSMMC_WRITE(host->base, ISE, INT_EN_MASK); - if (host->use_dma) OMAP_HSMMC_WRITE(host->base, IE, INT_EN_MASK & ~(BRR_ENABLE | BWR_ENABLE)); David -- David Vrabel, Senior Software Engineer, Drivers CSR, Churchill House, Cambridge Business Park, Tel: +44 (0)1223 692562 Cowley Road, Cambridge, CB4 0WZ http://www.csr.com/ Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom -- 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