Commit 2886c92c authored Apr 03, 2013 by Sahitya Tummala Committed by Xiaonian Wang Apr 07, 2017

mmc: sdhci-msm: Fix issue with MSM bus bandwidth voting

The driver is using wrong clock rate to calculate the required
bandwidth and due to this voting is happening for more bandwidth
than it is required. This is ultimately preventing system core
voltage from entering into low power mode.

The sdhci_host clock indicates the clock rate as requested by MMC
core layer and the actual rate that is set is indicated by clk_rate
within struct sdhci_msm_host. As of now, sdhci_host clock is being
used to calculate the bandwidth whereas bus-bw-vectors-bps indicates
only supported bandwidths and hence a mismatch. Fix this by using the
right clock rate which is clk_rate within struct sdhci_msm_host.

Change-Id: If7d81e44a9b479c4c8e9fbaa7e092af2afb9cb9f
Signed-off-by: Sahitya Tummala <stummala@codeaurora.org>

parent 68adcae2

drivers/mmc/host/sdhci-msm.c

+4 −1

Original line number	Diff line number	Diff line
		@@ -944,9 +944,12 @@ static struct sdhci_msm_pltfm_data sdhci_msm_populate_pdata(struct device dev)
		static unsigned int sdhci_get_bw_required(struct sdhci_host *host,
		struct mmc_ios *ios)
		{
		struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host);
		struct sdhci_msm_host *msm_host = pltfm_host->priv;

		unsigned int bw;

		bw = host->clock;
		bw = msm_host->clk_rate;
		/*
		* For DDR mode, SDCC controller clock will be at
		* the double rate than the actual clock that goes to card.