On Thu, Apr 24, 2014 at 09:32:30AM +0200, Ulf Hansson wrote:
> On 23 April 2014 21:08, Russell King <rmk+kernel@xxxxxxxxxxxxxxxx> wrote:
> > We don't need these hooks in order to insert code in these paths, we
> > can just provide our own handlers and call the main sdhci handlers as
> > appropriate.
> >
> > Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx>
> > ---
> > drivers/mmc/host/sdhci-of-esdhc.c | 55 +++++++++++++++++++++++++--------------
> > 1 file changed, 36 insertions(+), 19 deletions(-)
> >
> > diff --git a/drivers/mmc/host/sdhci-of-esdhc.c b/drivers/mmc/host/sdhci-of-esdhc.c
> > index fcaeae5f55b8..605815e52f5f 100644
> > --- a/drivers/mmc/host/sdhci-of-esdhc.c
> > +++ b/drivers/mmc/host/sdhci-of-esdhc.c
> > @@ -241,20 +241,6 @@ static void esdhc_of_set_clock(struct sdhci_host *host, unsigned int clock)
> > mdelay(1);
> > }
> >
> > -#ifdef CONFIG_PM
...
> > +#ifdef CONFIG_PM
>
> This should be CONFIG_PM_SLEEP
That may be the case, but that would be an additional modification, and so
should be a separate patch.
>
> > +
> > +static u32 esdhc_proctl;
> > +static int esdhc_of_suspend(struct device *dev)
> > +{
> > + struct sdhci_host *host = dev_get_drvdata(dev);
> > +
> > + esdhc_proctl = sdhci_be32bs_readl(host, SDHCI_HOST_CONTROL);
> > +
> > + return sdhci_suspend_host(host);
> > +}
> > +
> > +static void esdhc_of_resume(device *dev)
> > +{
> > + struct sdhci_host *host = dev_get_drvdata(dev);
> > + int ret = sdhci_resume_host(host);
> > +
> > + if (ret == 0) {
> > + /* Isn't this already done by sdhci_resume_host() ? --rmk */
> > + esdhc_of_enable_dma(host);
> > + sdhci_be32bs_writel(host, esdhc_proctl, SDHCI_HOST_CONTROL);
> > + }
> > +
> > + return ret;
> > +}
> > +
> > +static const struct dev_pm_ops esdhc_pmops = {
> > + .suspend = esdhci_of_suspend,
> > + .resume = esdhci_of_resume,
> > +};
> > +#define ESDHC_PMOPS (&esdhc_pmops)
> > +#else
> > +#define ESDHC_PMOPS NULL
> > +#endif
> > +
>
> I would suggest to use the SIMPLE_DEV_PM_OPS macro to simplify code a bit.
That is dependent on changing the ifdef. As I say, that should be a
separate patch - possibly one which comes before this one.
--
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.
--
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
