Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Viresh Kumar
On 01-12-16, 10:45, Joonyoung Shim wrote: > Already i tried and the error was gone but sometimes system resume is > halt. That looks to be a different problem which may not be related to cpufreq at all. > I'm not sure that it has any effect so i need to more dig. Right. -- viresh

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Viresh Kumar
On 01-12-16, 10:45, Joonyoung Shim wrote: > Already i tried and the error was gone but sometimes system resume is > halt. That looks to be a different problem which may not be related to cpufreq at all. > I'm not sure that it has any effect so i need to more dig. Right. -- viresh

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Joonyoung Shim
On 11/30/2016 05:05 PM, Viresh Kumar wrote: > On 30-11-16, 15:19, Joonyoung Shim wrote: >> Hi Viresh, >> >> On 11/30/2016 12:59 PM, Viresh Kumar wrote: >>> From: Stephen Boyd >>> >>> Joonyoung Shim reported an interesting problem on his ARM octa-core >>> Odoroid-XU3

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Joonyoung Shim
On 11/30/2016 05:05 PM, Viresh Kumar wrote: > On 30-11-16, 15:19, Joonyoung Shim wrote: >> Hi Viresh, >> >> On 11/30/2016 12:59 PM, Viresh Kumar wrote: >>> From: Stephen Boyd >>> >>> Joonyoung Shim reported an interesting problem on his ARM octa-core >>> Odoroid-XU3 platform. During system

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Viresh Kumar
On 30-11-16, 09:29, Viresh Kumar wrote: > +struct opp_table *dev_pm_opp_set_regulator(struct device *dev, const char > *name) > { > struct opp_table *opp_table; > struct regulator *reg; > - int ret; > > mutex_lock(_table_lock); > > opp_table = _add_opp_table(dev);

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Viresh Kumar
On 30-11-16, 09:29, Viresh Kumar wrote: > +struct opp_table *dev_pm_opp_set_regulator(struct device *dev, const char > *name) > { > struct opp_table *opp_table; > struct regulator *reg; > - int ret; > > mutex_lock(_table_lock); > > opp_table = _add_opp_table(dev);

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Viresh Kumar
On 30-11-16, 15:19, Joonyoung Shim wrote: > Hi Viresh, > > On 11/30/2016 12:59 PM, Viresh Kumar wrote: > > From: Stephen Boyd > > > > Joonyoung Shim reported an interesting problem on his ARM octa-core > > Odoroid-XU3 platform. During system suspend,

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-30 Thread Viresh Kumar
On 30-11-16, 15:19, Joonyoung Shim wrote: > Hi Viresh, > > On 11/30/2016 12:59 PM, Viresh Kumar wrote: > > From: Stephen Boyd > > > > Joonyoung Shim reported an interesting problem on his ARM octa-core > > Odoroid-XU3 platform. During system suspend, dev_pm_opp_put_regulator() > > was failing

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-29 Thread Joonyoung Shim
Hi Viresh, On 11/30/2016 12:59 PM, Viresh Kumar wrote: > From: Stephen Boyd > > Joonyoung Shim reported an interesting problem on his ARM octa-core > Odoroid-XU3 platform. During system suspend, dev_pm_opp_put_regulator() > was failing for a struct device for which

Re: [PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-29 Thread Joonyoung Shim
Hi Viresh, On 11/30/2016 12:59 PM, Viresh Kumar wrote: > From: Stephen Boyd > > Joonyoung Shim reported an interesting problem on his ARM octa-core > Odoroid-XU3 platform. During system suspend, dev_pm_opp_put_regulator() > was failing for a struct device for which dev_pm_opp_set_regulator() is

[PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-29 Thread Viresh Kumar
From: Stephen Boyd Joonyoung Shim reported an interesting problem on his ARM octa-core Odoroid-XU3 platform. During system suspend, dev_pm_opp_put_regulator() was failing for a struct device for which dev_pm_opp_set_regulator() is called earlier. This happened because an

[PATCH V4] PM / OPP: Pass opp_table to dev_pm_opp_put_regulator()

2016-11-29 Thread Viresh Kumar
From: Stephen Boyd Joonyoung Shim reported an interesting problem on his ARM octa-core Odoroid-XU3 platform. During system suspend, dev_pm_opp_put_regulator() was failing for a struct device for which dev_pm_opp_set_regulator() is called earlier. This happened because an earlier call to