<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 26, 2015 at 6:06 PM, Ulf Hansson <span dir="ltr"><<a href="mailto:ulf.hansson@linaro.org" target="_blank">ulf.hansson@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 21 May 2015 at 10:43, Suman Tripathi <<a href="mailto:stripathi@apm.com">stripathi@apm.com</a>> wrote:<br>
> The sdhci framework disables SDR104/SDR50/DDR50 based on only quirk.<br>
> This patch adds the support to disable SDR104/SDR50/DDR50 based on<br>
> reading the capability register 0.<br>
><br>
> Signed-off-by: Suman Tripathi <<a href="mailto:stripathi@apm.com">stripathi@apm.com</a>><br>
> ---<br>
> ---<br>
>  drivers/mmc/host/sdhci.c | 3 ++-<br>
>  1 file changed, 2 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c<br>
> index 58c1770..a3d9b8a 100644<br>
> --- a/drivers/mmc/host/sdhci.c<br>
> +++ b/drivers/mmc/host/sdhci.c<br>
> @@ -3118,7 +3118,8 @@ int sdhci_add_host(struct sdhci_host *host)<br>
>                 }<br>
>         }<br>
><br>
> -       if (host->quirks2 & SDHCI_QUIRK2_NO_1_8_V)<br>
> +       if (host->quirks2 & SDHCI_QUIRK2_NO_1_8_V ||<br>
> +           !(caps[0] & SDHCI_CAN_VDD_180))<br>
>                 caps[1] &= ~(SDHCI_SUPPORT_SDR104 | SDHCI_SUPPORT_SDR50 |<br>
>                        SDHCI_SUPPORT_DDR50);<br>
><br>
> --<br>
> 1.8.2.1<br>
><br>
<br>
</div></div>I have no problem with this patch, except that it would be nice to get<br>
a few "tested by" to make sure it doesn't break UHS support for some<br>
SoCs.<br>
<br>
Kind regards<br>
Uffe<br>
</blockquote></div><div class="gmail_extra"><br></div>Can anyone test this in some other SoC ? Appreciate your help ..<br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Thanks,</div><div>with regards,</div>Suman Tripathi</div></div>
</div></div>