Re: [PATCH] drm/tegra: add support for runtime pm

2013-06-10 Thread Terje Bergström
On 10.06.2013 23:43, Thierry Reding wrote: > Can you post the corresponding wrappers to make it easier to discuss > them? If they just wrap runtime PM calls then they don't solve the > locality problems that Terje brought up. I don't think the wrappers are applicable here. They're there in downstr

Re: [PATCH] drm/tegra: add support for runtime pm

2013-06-10 Thread Thierry Reding
On Tue, Jun 04, 2013 at 02:11:27PM +0530, Mayuresh Kulkarni wrote: > On Tuesday 28 May 2013 02:40 PM, Thierry Reding wrote: > >* PGP Signed by an unknown key > > > >On Tue, May 28, 2013 at 08:45:03AM +0300, Terje Bergström wrote: > >>On 27.05.2013 18:45, Thierry Reding wrote: > >>>On Mon, May 27, 2

Re: [PATCH] drm/tegra: add support for runtime pm

2013-06-04 Thread Mayuresh Kulkarni
On Tuesday 28 May 2013 02:40 PM, Thierry Reding wrote: * PGP Signed by an unknown key On Tue, May 28, 2013 at 08:45:03AM +0300, Terje Bergström wrote: On 27.05.2013 18:45, Thierry Reding wrote: On Mon, May 27, 2013 at 07:19:28PM +0530, Mayuresh Kulkarni wrote: +#ifdef CONFIG_PM_RUNTIME +stati

Re: [PATCH] drm/tegra: add support for runtime pm

2013-05-28 Thread Thierry Reding
On Tue, May 28, 2013 at 08:45:03AM +0300, Terje Bergström wrote: > On 27.05.2013 18:45, Thierry Reding wrote: > > On Mon, May 27, 2013 at 07:19:28PM +0530, Mayuresh Kulkarni wrote: > >> +#ifdef CONFIG_PM_RUNTIME > >> +static int host1x_runtime_suspend(struct device *dev) > >> +{ > >> + struct host

Re: [PATCH] drm/tegra: add support for runtime pm

2013-05-27 Thread Terje Bergström
On 27.05.2013 18:45, Thierry Reding wrote: > On Mon, May 27, 2013 at 07:19:28PM +0530, Mayuresh Kulkarni wrote: >> +#ifdef CONFIG_PM_RUNTIME >> +static int host1x_runtime_suspend(struct device *dev) >> +{ >> +struct host1x *host; >> + >> +host = dev_get_drvdata(dev); >> +if (IS_ERR_OR_N

Re: [PATCH] drm/tegra: add support for runtime pm

2013-05-27 Thread Thierry Reding
On Mon, May 27, 2013 at 07:19:28PM +0530, Mayuresh Kulkarni wrote: > - as of now host1x and gr2d module's clocks are enabled > in their driver's .probe and never disabled > - this commit adds support for runtime pm in host1x and > gr2d drivers > - during boot-up clocks are enabled in .probe and dis

[PATCH] drm/tegra: add support for runtime pm

2013-05-27 Thread Mayuresh Kulkarni
- as of now host1x and gr2d module's clocks are enabled in their driver's .probe and never disabled - this commit adds support for runtime pm in host1x and gr2d drivers - during boot-up clocks are enabled in .probe and disabled on its exit - when new work is submitted, clocks are enabled in submit