On Sat, 12 Jun 2010 14:44:50 +0900
Kyungmin Park <kyungmin.park@xxxxxxxxxxx> wrote:
> From: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
>
> Some host controller can set mmc->caps before sdhci_add_host.
>
> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> ---
> drivers/mmc/host/sdhci.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
> index 4321e0c..142419c 100644
> --- a/drivers/mmc/host/sdhci.c
> +++ b/drivers/mmc/host/sdhci.c
> @@ -1791,7 +1791,7 @@ int sdhci_add_host(struct sdhci_host *host)
> else
> mmc->f_min = host->max_clk / 256;
> mmc->f_max = host->max_clk;
> - mmc->caps = MMC_CAP_SDIO_IRQ;
> + mmc->caps |= MMC_CAP_SDIO_IRQ;
>
> if (!(host->quirks & SDHCI_QUIRK_FORCE_1_BIT_DATA))
> mmc->caps |= MMC_CAP_4_BIT_DATA;
A great shower of MMC patches have magically turned up in linux-next,
apparently via some tree of Grant's. Those patches changed the above
code to look like:
if (!(host->quirks & SDHCI_QUIRK_NO_SDIO_IRQ))
mmc->caps |= MMC_CAP_SDIO_IRQ;
So it appears that this bug is fixed in that code as well. So I'll
drop your patch. If the above changes end up not getting merged into
mainline then your fix will be lost.
That fix was unchangelogged. In fact the patch was completely
unchangelogged and I haven't looked at it at all and as far as I can
tell none of it has been sent to the mmc list.
--
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
