"kishore kadiyala" <[email protected]> writes: > Crash if enabled CONFIG_PM_RUNTIME and power_saving set. > > Use get_sync/put_sync properly so that host->nesting_cnt gets > updated properly.
<blush>This one is my fault for not really understanding the MMC driver. Go ahead and just fold this into the original patch when you incorporate it. Kevin > Cc: Kevin Hilman <[email protected]> > Cc: Benoit Cousson <[email protected]> > Signed-off-by: Kishore Kadiyala <[email protected]> > --- > drivers/mmc/host/omap_hsmmc.c | 5 ++--- > 1 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c > index c113bfd..90a69c3 100644 > --- a/drivers/mmc/host/omap_hsmmc.c > +++ b/drivers/mmc/host/omap_hsmmc.c > @@ -2036,7 +2036,8 @@ static int __init omap_hsmmc_probe(struct > platform_device *pdev) > host->dpm_state = DISABLED; > > pm_runtime_enable(host->dev); > - pm_runtime_get_sync(host->dev); > + if (mmc_host_enable(host->mmc) != 0) > + goto err1; > > if (cpu_is_omap2430()) { > host->dbclk = clk_get(&pdev->dev, "mmchsdb_fck"); > @@ -2139,8 +2140,6 @@ static int __init omap_hsmmc_probe(struct > platform_device *pdev) > > mmc_add_host(mmc); > > - pm_runtime_put(host->dev); > - > if (mmc_slot(host).name != NULL) { > ret = device_create_file(&mmc->class_dev, &dev_attr_slot_name); > if (ret < 0) -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
