Re: [Intel-gfx] [PATCH] drm/i915/psr: Check for SET_POWER_CAPABLE bit at PSR init time.

2018-05-11 Thread Dhinakaran Pandiyan
On Fri, 2018-05-11 at 15:24 +0300, Ville Syrjälä wrote: > On Wed, May 09, 2018 at 02:54:01PM -0700, Dhinakaran Pandiyan wrote: > > > > By moving the check from psr_compute_config() to psr_init_dpcd(), > > we get > > to set the dev_priv->psr.sink_support flag only when the panel is > > capable of

Re: [Intel-gfx] [PATCH] drm/i915/psr: Check for SET_POWER_CAPABLE bit at PSR init time.

2018-05-11 Thread Ville Syrjälä
On Wed, May 09, 2018 at 02:54:01PM -0700, Dhinakaran Pandiyan wrote: > By moving the check from psr_compute_config() to psr_init_dpcd(), we get > to set the dev_priv->psr.sink_support flag only when the panel is > capable of changing power state. An additional benefit is that the check > will be

Re: [Intel-gfx] [PATCH] drm/i915/psr: Check for SET_POWER_CAPABLE bit at PSR init time.

2018-05-09 Thread Souza, Jose
On Wed, 2018-05-09 at 14:54 -0700, Dhinakaran Pandiyan wrote: > By moving the check from psr_compute_config() to psr_init_dpcd(), we > get > to set the dev_priv->psr.sink_support flag only when the panel is > capable of changing power state. An additional benefit is that the > check > will be

[Intel-gfx] [PATCH] drm/i915/psr: Check for SET_POWER_CAPABLE bit at PSR init time.

2018-05-09 Thread Dhinakaran Pandiyan
By moving the check from psr_compute_config() to psr_init_dpcd(), we get to set the dev_priv->psr.sink_support flag only when the panel is capable of changing power state. An additional benefit is that the check will be performed only at init time instead of every atomic_check. This should change