On Tue, Oct 23, 2012 at 03:01:17PM +0800, r66...@freescale.com wrote:
> From: Jerry Huang <chang-ming.hu...@freescale.com>
> 
> The current eSDHC driver use the poll mode to detect
> if the SD/MMC card is inserted or removed, which will generate
> many interrupts and impact the performance. 
> Therefore, change the default card detect to interrupt mode,
> if the board can't support this mode, we still use the poll mode.
> 
> Signed-off-by: Jerry Huang <chang-ming.hu...@freescale.com>
> CC: Anton Vorontsov <cbouatmai...@gmail.com>
> CC: Chris Ball <c...@laptop.org>
> ---

IIRC, the card detection is broken SOC-revision-wise, not board-wise. So
the change seems wrong.

Also, take a look at this:

        http://lkml.org/lkml/2010/7/14/127

I started the work but never finished, unfortunately it caused some
regressions for non-FSL hardware...

>  drivers/mmc/host/sdhci-of-esdhc.c |    7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/host/sdhci-of-esdhc.c 
> b/drivers/mmc/host/sdhci-of-esdhc.c
> index ffc1226..5dc362f 100644
> --- a/drivers/mmc/host/sdhci-of-esdhc.c
> +++ b/drivers/mmc/host/sdhci-of-esdhc.c
> @@ -196,6 +196,11 @@ static void esdhc_of_detect_limitation(struct 
> platform_device *pdev,
>       if (vvn == VENDOR_V_22)
>               pdata->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23;
>  
> +     /* P4080DS and MPC837XMDS board don't support interrupt mode */
> +     if (of_machine_is_compatible("fsl,mpc837xmds") ||
> +         of_machine_is_compatible("fsl,P4080DS"))
> +             pdata->quirks |= SDHCI_QUIRK_BROKEN_CARD_DETECTION;
> +
>       iounmap(ioaddr);
>  end:
>       return;
> @@ -223,7 +228,7 @@ static struct sdhci_pltfm_data sdhci_esdhc_pdata = {
>        * card detection could be handled via GPIO
>        * eSDHC cannot support End Attribute in NOP ADMA descriptor
>        */
> -     .quirks = ESDHC_DEFAULT_QUIRKS | SDHCI_QUIRK_BROKEN_CARD_DETECTION
> +     .quirks = ESDHC_DEFAULT_QUIRKS
>               | SDHCI_QUIRK_NO_CARD_NO_RESET
>               | SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
>       .ops = &sdhci_esdhc_ops,
> -- 
> 1.7.9.5
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to