Sorry about not responding sooner. I've been a bit swamped.
Timo Teras wrote:
> Some broken cards seem to process CMD1 even in stand-by state. The result is
> that the card replies with ILLEGAL_COMMAND error for the next command sent
> after rescanning. Currently the next command is select card, which would
> return the error. But the CMD7 does actually succeed and retries of the
> command will timeout. The solution is to poll card status after the CMD1
> which clears the cached error.
>
> Signed-off-by: Timo Teras <[email protected]>
>
>
I take it these cards do not reply to CMD2?
> ---
>
> drivers/mmc/mmc.c | 7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> 97675ada7049d12523aa9e9908d4613dfd333641
> diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
> index ee8863c..3324b6e 100644
> --- a/drivers/mmc/mmc.c
> +++ b/drivers/mmc/mmc.c
> @@ -1178,14 +1178,17 @@ static void mmc_rescan(void *data)
> {
> struct mmc_host *host = data;
> struct list_head *l, *n;
> + unsigned char power_mode;
>
> mmc_claim_host(host);
>
> - if (host->ios.power_mode == MMC_POWER_ON)
> - mmc_check_cards(host);
> + power_mode = host->ios.power_mode;
>
> mmc_setup(host);
>
> + if (power_mode == MMC_POWER_ON)
> + mmc_check_cards(host);
> +
> if (!list_empty(&host->cards)) {
> /*
> * (Re-)calculate the fastest clock rate which the
>
This change is ok right now, but might come back to bite us in the
future if we implement more intelligent voltage selection (right now new
cards will have to make due with what's already selected).
If we check cards on both sides of mmc_setup(), then we should be covered.
Also, please add some comments about why we do this. Otherwise it will
run the risk of getting removed in the future.
Rgds
--
-- Pierre Ossman
Linux kernel, MMC maintainer http://www.kernel.org
PulseAudio, core developer http://pulseaudio.org
rdesktop, core developer http://www.rdesktop.org
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
