Hi, Seungwon. On 03/21/2014 11:03 PM, Seungwon Jeon wrote: > Adds checking whether the clock is valid. > > Signed-off-by: Seungwon Jeon <[email protected]> > --- > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
