Re: [Intel-gfx] [PATCH] drm/i915: Redoing the PSR setup on resume

2014-01-28 Thread Daniel Vetter
On Tue, Jan 28, 2014 at 03:57:13PM +, C, Ramalingam wrote: > Hi Daniel, > > Thanks for your valuable review. I found that Rodrigo has submitted a > solution for the same issue just before mine. I am seeing that solution is > aligning to the expectations. > http://lists.freedesktop.org/archiv

Re: [Intel-gfx] [PATCH] drm/i915: Redoing the PSR setup on resume

2014-01-28 Thread C, Ramalingam
] [PATCH] drm/i915: Redoing the PSR setup on resume > > On Fri, Jan 24, 2014 at 12:50:17AM +0530, Ramalingam C wrote: > > Due to switch between console and graphics modes multiple psr_enable > > call will be made. On such occasions, to avoid repeated psr_setup, a > > flag called

Re: [Intel-gfx] [PATCH] drm/i915: Redoing the PSR setup on resume

2014-01-25 Thread Daniel Vetter
On Fri, Jan 24, 2014 at 12:50:17AM +0530, Ramalingam C wrote: > Due to switch between console and graphics modes multiple psr_enable > call will be made. On such occasions, to avoid repeated psr_setup, > a flag called psr_setup_done is used. > > On suspend-resume, panel goes for a power cycle. Hen

[Intel-gfx] [PATCH] drm/i915: Redoing the PSR setup on resume

2014-01-23 Thread Ramalingam C
Due to switch between console and graphics modes multiple psr_enable call will be made. On such occasions, to avoid repeated psr_setup, a flag called psr_setup_done is used. On suspend-resume, panel goes for a power cycle. Hence PSR setup should be redone to enable the PSR after suspend-resume. So