Hi, Seungwon.
On 03/21/2014 11:03 PM, Seungwon Jeon wrote:
> Adds checking whether the clock is valid.
>
> Signed-off-by: Seungwon Jeon <tgih.jun@xxxxxxxxxxx>
> ---
> drivers/mmc/host/dw_mmc-exynos.c | 14 +++++++++++---
> 1 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/host/dw_mmc-exynos.c b/drivers/mmc/host/dw_mmc-exynos.c
> index a67e784..89aa019 100644
> --- a/drivers/mmc/host/dw_mmc-exynos.c
> +++ b/drivers/mmc/host/dw_mmc-exynos.c
> @@ -117,7 +117,12 @@ static int dw_mci_exynos_priv_init(struct dw_mci *host)
> static int dw_mci_exynos_setup_clock(struct dw_mci *host)
> {
> struct dw_mci_exynos_priv_data *priv = host->priv;
> - unsigned long rate = clk_get_rate(host->ciu_clk);
> + unsigned long rate;
> +
> + if (IS_ERR(host->ciu_clk))
> + rate = host->bus_hz;
> + else
> + rate = clk_get_rate(host->ciu_clk);
>
> host->bus_hz = rate / (priv->ciu_div + 1);
how about this?
host->bus_hz /= (priv->ciu_div + 1);
host->bus_hz is already set to "host->bus_hz" or "clk_get_rate()" into dw_mmc.c.
Best Regards,
Jaehoon Chung
> return 0;
> @@ -196,8 +201,11 @@ static void dw_mci_exynos_set_ios(struct dw_mci *host, struct mmc_ios *ios)
> mci_writel(host, CLKSEL, priv->sdr_timing);
> }
>
> - /* Don't care if wanted clock is zero */
> - if (!wanted)
> + /*
> + * Don't care if wanted clock is zero or
> + * ciu clock is unavailable
> + */
> + if (!wanted || IS_ERR(host->ciu_clk))
> return;
>
> /* Guaranteed minimum frequency for cclkin */
>
--
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
