Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757311AbbKFIKF (ORCPT ); Fri, 6 Nov 2015 03:10:05 -0500 Received: from mail-yk0-f171.google.com ([209.85.160.171]:36444 "EHLO mail-yk0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757240AbbKFIKC (ORCPT ); Fri, 6 Nov 2015 03:10:02 -0500 MIME-Version: 1.0 In-Reply-To: References: <1436183618-15330-1-git-send-email-ivan.ivanov@linaro.org> Date: Fri, 6 Nov 2015 09:10:01 +0100 Message-ID: Subject: Re: [PATCH v2] mmc: sdhci-msm: Boost controller core clock From: Ulf Hansson To: Bjorn Andersson Cc: "Ivan T. Ivanov" , Georgi Djakov , Stephen Boyd , Peter Griffin , linux-mmc , "linux-kernel@vger.kernel.org" , linux-arm-msm Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3084 Lines: 84 On 6 November 2015 at 02:42, Bjorn Andersson wrote: > On Mon, Jul 6, 2015 at 4:53 AM, Ivan T. Ivanov wrote: >> Ensure SDCC is working with maximum clock otherwise card >> detection could be extremely slow, up to 7 seconds. >> >> Signed-off-by: Ivan T. Ivanov >> Reviewed-by: Georgi Djakov >> Acked-by: Stephen Boyd >> --- >> >> Changes since v0: >> - s/falied/failed in warning message. >> >> drivers/mmc/host/sdhci-msm.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/drivers/mmc/host/sdhci-msm.c b/drivers/mmc/host/sdhci-msm.c >> index 4a09f76..4bcee03 100644 >> --- a/drivers/mmc/host/sdhci-msm.c >> +++ b/drivers/mmc/host/sdhci-msm.c >> @@ -489,6 +489,11 @@ static int sdhci_msm_probe(struct platform_device *pdev) >> goto pclk_disable; >> } >> >> + /* Vote for maximum clock rate for maximum performance */ >> + ret = clk_set_rate(msm_host->clk, INT_MAX); >> + if (ret) >> + dev_warn(&pdev->dev, "core clock boost failed\n"); >> + > > On my 8974AC devices this results in GCC_SDCC1_APPS_CLK changing from > 100MHz to 200MHz for my eMMC. Unfortunately this results in the > following error: > > [ 5.103241] mmcblk0: retrying because a re-tune was needed > [ 5.109270] mmcblk0: error -84 transferring data, sector 5816322, > nr 2, cmd response 0x900, card status 0xc00 > > Looking at the board specification it's stated that these card should > run in DDR50, so I've tried specifying "max-frequency" in the dt. I > verified in sdhci_set_clock() that we get a divisor of 4, but the > result is a repetition of: I don't follow. Are you saying that changing the clock frequency to 200MHz caused the card to be initialized in HS200 mode instead of DDR50? > > [ 1.702312] mmc1: Switching to 3.3V signalling voltage failed > [ 1.837987] mmc1: power class selection to bus width 8 ddr 0 failed > [ 1.846227] mmc1: error -110 whilst initialising MMC card > [ 1.946303] mmc1: Reset 0x1 never completed. > > I tried to disable HS200 by specifying SDHCI_QUIRK2_BROKEN_HS200. This > makes the card come up nicely again. > > > Is there any other way to force the link to DDR50? Is it acceptable to > expose the broken-hs200 quirk in dt so I can use that for now? We already have these DT bindings. - mmc-ddr-1_8v: eMMC high-speed DDR mode(1.8V I/O) is supported - mmc-ddr-1_2v: eMMC high-speed DDR mode(1.2V I/O) is supported - mmc-hs200-1_8v: eMMC HS200 mode(1.8V I/O) is supported - mmc-hs200-1_2v: eMMC HS200 mode(1.2V I/O) is supported Can't we use these instead? > > (The downstream fix used for this was apparently to just remove all > other caps...) > > Regards, > Bjorn Thanks for reporting the regression! Kind regards Uffe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/