Hi Simon, On Wednesday 13 June 2012 10:17:12 Simon Horman wrote: > On Tue, Jun 12, 2012 at 11:29:35PM +0200, Laurent Pinchart wrote: > > The tmio_mmc_set_ios() function configures the MMC power, clock and bus > > width. When the mmc core requests the driver to power off the card, we > > inform runtime PM, that the controller can be suspended. This can lead > > to the MSTP clock being turned off. > > > > Writing to any 16-bit hardware registers with the MSTP clock off leads > > to timeouts and errors being printed to the kernel log. This can occur > > both when stopping the MMC clock and when configuring the bus width. > > > > To fix this, stop the MMC clock before calling put_runtime_pm(), and > > skip bus width configuration when power is off. > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > Hi Laurent, > > could you let me know if there is a (simple :) way for me to exercise this? Without the patch suspending the Armadillo board to RAM produces a "timeout waiting for SD bus idle" message. Make sure you apply the "mmc: sh_mmcif: Support MMC_SLEEP_AWAKE command" patch first, or suspending the board will fail completely. -- Regards, Laurent Pinchart -- 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
