Hi Olof,
On Wed, Dec 22, 2010 at 02:01:10AM -0600, Olof Johansson wrote:
> Don't schedule the finish_tasklet unless the command complete bit is
> set in the interrupt status register.
>
> Signed-off-by: Olof Johansson <olof@xxxxxxxxx>
Could we have some more detail here, please? What are the symptoms of
running without this patch? Should we apply it to the stable tree too?
> ---
> drivers/mmc/host/sdhci.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
> index c0094c1..562aaea 100644
> --- a/drivers/mmc/host/sdhci.c
> +++ b/drivers/mmc/host/sdhci.c
> @@ -1432,7 +1432,8 @@ static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask)
> host->cmd->error = -EILSEQ;
>
> if (host->cmd->error) {
> - tasklet_schedule(&host->finish_tasklet);
> + if (intmask & SDHCI_INT_RESPONSE)
> + tasklet_schedule(&host->finish_tasklet);
> return;
> }
>
Thanks,
--
Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/>
One Laptop Per Child
--
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
