On 09/03/2013 11:29 AM, Ulf Hansson wrote:
> After a write to the MMCICLOCK register data cannot be written to this
> register for three feedback clock cycles. Writes to the MMCIPOWER
> register must be separated by three MCLK cycles. Previously no issues
> has been observered, but using higher ARM clock frequencies on STE-
> platforms has triggered this problem.
>
> The MMCICLOCK register is written to in .set_ios and for some data
> transmissions for SDIO. We do not need a delay at the data transmission
> path, because sending and receiving data will require more than three
> clock cycles. Then we use a simple logic to only delay in .set_ios and
> thus we don't affect throughput performance.
>
> Signed-off-by: Johan Rudholm <jrudholm@xxxxxxxxx>
> Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> Acked-by: Rickard Andersson <rickard.andersson@xxxxxxxxxxxxxx>
> ---
Reviewed-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
+ two questions below.
> drivers/mmc/host/mmci.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c
> index c550b3e..82afcd3 100644
> --- a/drivers/mmc/host/mmci.c
> +++ b/drivers/mmc/host/mmci.c
> @@ -189,6 +189,21 @@ static int mmci_validate_data(struct mmci_host *host,
> return 0;
> }
>
> +static void mmci_reg_delay(struct mmci_host *host)
> +{
> + /*
> + * According to the spec, at least three feedback clock cycles
> + * of max 52 MHz must pass between two writes to the MMCICLOCK reg.
> + * Three MCLK clock cycles must pass between two MMCIPOWER reg writes.
> + * Worst delay time during card init is at 100 kHz => 30 us.
> + * Worst delay time when up and running is at 25 MHz => 120 ns.
> + */
> + if (host->cclk < 20000000)
Shouldn't it be 25000000 ?
What about using macros ?
#define MMC_CLOCK 25000000
#define MMCINIT_DELAY 30
#define MMCUP_DELAY 120
> + udelay(30);
> + else
> + ndelay(120);
> +}
> +
> /*
> * This must be called with host->lock held
> */
> @@ -1264,6 +1279,7 @@ static void mmci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
>
> mmci_set_clkreg(host, ios->clock);
> mmci_write_pwrreg(host, pwr);
> + mmci_reg_delay(host);
>
> spin_unlock_irqrestore(&host->lock, flags);
>
>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
--
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
