Re: [PATCH] PM / Domains: Don't power on at attach for the multi PM domain case

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday, June 29, 2018 11:51:46 AM CEST Viresh Kumar wrote:
> On 29-06-18, 11:15, Ulf Hansson wrote:
> > There are no legacy behavior in drivers to consider while attaching a
> > device to genpd - for the multiple PM domain case.
> > 
> > For that reason, let's instead require the driver to runtime resume the
> > device, via calling pm_runtime_get_sync() for example, when it needs to
> > power on the corresponding PM domain.
> > 
> > This allows us to improve the situation during attach. Instead of always
> > power on the PM domain, which may be unnecessary, let's leave it in its
> > current state. Additionally, to avoid the PM domain to stay powered on,
> > let's schedule a power off work.
> > 
> > Fixes: 3c095f32a92b ("PM / Domains: Add support for multi PM domains...")
> > Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> > ---
> >  drivers/base/power/domain.c | 16 +++++++++-------
> >  1 file changed, 9 insertions(+), 7 deletions(-)
> > 
> > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
> > index c298de8a8308..9e8484189034 100644
> > --- a/drivers/base/power/domain.c
> > +++ b/drivers/base/power/domain.c
> > @@ -2235,7 +2235,7 @@ static void genpd_dev_pm_sync(struct device *dev)
> >  }
> >  
> >  static int __genpd_dev_pm_attach(struct device *dev, struct device_node *np,
> > -				 unsigned int index)
> > +				 unsigned int index, bool power_on)
> >  {
> >  	struct of_phandle_args pd_args;
> >  	struct generic_pm_domain *pd;
> > @@ -2271,9 +2271,11 @@ static int __genpd_dev_pm_attach(struct device *dev, struct device_node *np,
> >  	dev->pm_domain->detach = genpd_dev_pm_detach;
> >  	dev->pm_domain->sync = genpd_dev_pm_sync;
> >  
> > -	genpd_lock(pd);
> > -	ret = genpd_power_on(pd, 0);
> > -	genpd_unlock(pd);
> > +	if (power_on) {
> > +		genpd_lock(pd);
> > +		ret = genpd_power_on(pd, 0);
> > +		genpd_unlock(pd);
> > +	}
> >  
> >  	if (ret)
> >  		genpd_remove_device(pd, dev);
> > @@ -2307,7 +2309,7 @@ int genpd_dev_pm_attach(struct device *dev)
> >  				       "#power-domain-cells") != 1)
> >  		return 0;
> >  
> > -	return __genpd_dev_pm_attach(dev, dev->of_node, 0);
> > +	return __genpd_dev_pm_attach(dev, dev->of_node, 0, true);
> >  }
> >  EXPORT_SYMBOL_GPL(genpd_dev_pm_attach);
> >  
> > @@ -2359,14 +2361,14 @@ struct device *genpd_dev_pm_attach_by_id(struct device *dev,
> >  	}
> >  
> >  	/* Try to attach the device to the PM domain at the specified index. */
> > -	ret = __genpd_dev_pm_attach(genpd_dev, dev->of_node, index);
> > +	ret = __genpd_dev_pm_attach(genpd_dev, dev->of_node, index, false);
> >  	if (ret < 1) {
> >  		device_unregister(genpd_dev);
> >  		return ret ? ERR_PTR(ret) : NULL;
> >  	}
> >  
> > -	pm_runtime_set_active(genpd_dev);
> >  	pm_runtime_enable(genpd_dev);
> > +	genpd_queue_power_off_work(dev_to_genpd(genpd_dev));
> >  
> >  	return genpd_dev;
> >  }
> 
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> 
> 

Patch applied and queued up for 4.18-rc4.

Thanks!


--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux