Re: SD8686 fast power-cycle problems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



sdio_reset is harmless to non-sdio devices.  Especially since it is in the main power up path.

I see no reason this should be done.  Can you prepare a patch ?

snip from core.c

static int mmc_rescan_try_freq(struct mmc_host *host, unsigned freq)
{
	host->f_init = freq;

#ifdef CONFIG_MMC_DEBUG
	pr_info("%s: %s: trying to init card at %u Hz\n",
		mmc_hostname(host), __func__, host->f_init);
#endif
	mmc_power_up(host);

	/*
	 * sdio_reset sends CMD52 to reset card.  Since we do not know
	 * if the card is being re-initialized, just send it.  CMD52
	 * should be ignored by SD/eMMC cards.
	 */
	sdio_reset(host);
	mmc_go_idle(host);

	mmc_send_if_cond(host, host->ocr_avail);


regards,

Philip

On Jun 17, 2011, at 9:39 AM, Daniel Drake wrote:

> Hi,
> 
> Thanks for your help and explanations with earlier issues around the
> power-on and reset sequence for the SD8686.
> 
> We are still struggling a little with this, in the context of enabling
> runtime power management.
> 
> If we power down the card, and then *immediately* power it on again
> (and performing the CMD5,5,3,7 init sequence), the card fails to
> respond to the first CMD5.
> 
> We have identified two workarounds to this problem:
> 1. If we wait 300ms after powering off the card before turning it on
> again, it works fine.
> 
> 2. If we perform a SDIO I/O reset after powering on, before starting
> the 5,5,3,7 sequence, it works fine.
> 
> Do you have any comments or possible explanations for this issue? Is
> it something you are aware of?
> 
> Thanks,
> Daniel
> --
> 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

--
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


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux