On Thu, Feb 05, 2015 at 09:51:17PM -0800, Ray Jui wrote: > > @@ -265,19 +264,19 @@ static int sdhci_bcm_kona_probe(struct > > platform_device *pdev) > > } > > > > /* Get and enable the external clock */ > > Would you mind changing the above comment to "/* Get and enable the core > clock */?
Sure.
>
> > - kona_dev->external_clk = devm_clk_get(dev, NULL);
> > - if (IS_ERR(kona_dev->external_clk)) {
> > + pltfm_priv->clk = devm_clk_get(dev, NULL);
> > + if (IS_ERR(pltfm_priv->clk)) {
> > dev_err(dev, "Failed to get external clock\n");
>
> Change to "Failed to get the core clock"
OK, I will change all the references of "external clock" to "core clock".
> Not sure if this is relevant and should be done in the same patch. But
> both callbacks get_max_clock and get_timeout_clock in this driver can be
> set to sdhci_pltfm_clk_get_max_clock. That will help to get rid of two
> redundant private functions in this driver.
Sounds good. I will add this in the v2.
Thanks,
Kevin
pgpF80wvayNEw.pgp
Description: PGP signature
