Re: [PATCH 2/2] mmc: tegra: use mmc_of_parse to get the support of standard MMC DT bindings

[Joseph Lo <josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]

On Thu, 2013-02-21 at 01:07 +0800, Stephen Warren wrote:
> On 02/20/2013 12:05 AM, Joseph Lo wrote:
> > Updating the sdhci-tegra driver to use mmc_of_parse to support standard
> > MMC DT bindings. Then we can remove the redundant code that already support
> 
> >  static unsigned int tegra_sdhci_get_ro(struct sdhci_host *host)
> >  {
> > -	struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host);
> > -	struct sdhci_tegra *tegra_host = pltfm_host->priv;
> > -
> > -	if (!gpio_is_valid(tegra_host->wp_gpio))
> > -		return -1;
> > -
> > -	return gpio_get_value(tegra_host->wp_gpio);
> > +	return mmc_gpio_get_ro(host->mmc);
> >  }
> 
> It'd be nice if there was a standard version of this function that could
> be plugged directly into struct sdhci_ops, so that each individual
> driver doesn't have to re-invent this wrapper.
> 
> > @@ -220,15 +203,12 @@ static void sdhci_tegra_parse_dt(struct device *dev,
> >  					struct sdhci_tegra *tegra_host)
> >  {
> ...
> > +	struct sdhci_host *host;
> ...
> > +	host = platform_get_drvdata(to_platform_device(dev));
> > +	mmc_of_parse(host->mmc);
> >  }
> 
> It might be simpler to change the function prototype to simply pass in
> the host object too.

It's a interface problem that I can't fix now. If sdhci core is going to
integrate mmc_of_parse into sdhci_get_of_property and mmc_gpio_get_ro
into somethere sdhci_do_get_ro, then we can refine later.

Thanks,
Joseph