Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Imre Deak
On Fri, Feb 23, 2018 at 12:58:44AM +0200, Pandiyan, Dhinakaran wrote: > On Fri, 2018-02-23 at 00:42 +0200, Imre Deak wrote: > > On Fri, Feb 23, 2018 at 12:30:14AM +0200, Pandiyan, Dhinakaran wrote: > > > On Fri, 2018-02-23 at 00:05 +0200, Imre Deak wrote: > > > > On Thu, Feb 22, 2018 at 11:53:30PM

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Pandiyan, Dhinakaran
On Fri, 2018-02-23 at 00:42 +0200, Imre Deak wrote: > On Fri, Feb 23, 2018 at 12:30:14AM +0200, Pandiyan, Dhinakaran wrote: > > On Fri, 2018-02-23 at 00:05 +0200, Imre Deak wrote: > > > On Thu, Feb 22, 2018 at 11:53:30PM +0200, Imre Deak wrote: > > > > On Thu, Feb 22, 2018 at 11:33:10PM +0200, Pand

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Rodrigo Vivi
On Thu, Feb 22, 2018 at 10:10:24PM +, Pandiyan, Dhinakaran wrote: > > > On Thu, 2018-02-22 at 23:47 +0200, Ville Syrjälä wrote: > > On Thu, Feb 22, 2018 at 09:33:10PM +, Pandiyan, Dhinakaran wrote: > > > > > > > > > > > > On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > > > > O

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Imre Deak
On Fri, Feb 23, 2018 at 12:30:14AM +0200, Pandiyan, Dhinakaran wrote: > On Fri, 2018-02-23 at 00:05 +0200, Imre Deak wrote: > > On Thu, Feb 22, 2018 at 11:53:30PM +0200, Imre Deak wrote: > > > On Thu, Feb 22, 2018 at 11:33:10PM +0200, Pandiyan, Dhinakaran wrote: > > > > > > > > > > > > > > > > O

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Pandiyan, Dhinakaran
On Fri, 2018-02-23 at 00:05 +0200, Imre Deak wrote: > On Thu, Feb 22, 2018 at 11:53:30PM +0200, Imre Deak wrote: > > On Thu, Feb 22, 2018 at 11:33:10PM +0200, Pandiyan, Dhinakaran wrote: > > > > > > > > > > > > On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > > > > On Thu, Feb 22, 2018 a

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Pandiyan, Dhinakaran
On Thu, 2018-02-22 at 23:47 +0200, Ville Syrjälä wrote: > On Thu, Feb 22, 2018 at 09:33:10PM +, Pandiyan, Dhinakaran wrote: > > > > > > > > On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > > > On Thu, Feb 22, 2018 at 12:28:50PM -0800, Dhinakaran Pandiyan wrote: > > > > PSR on CNL r

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Imre Deak
On Thu, Feb 22, 2018 at 11:53:30PM +0200, Imre Deak wrote: > On Thu, Feb 22, 2018 at 11:33:10PM +0200, Pandiyan, Dhinakaran wrote: > > > > > > > > On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > > > On Thu, Feb 22, 2018 at 12:28:50PM -0800, Dhinakaran Pandiyan wrote: > > > > PSR on CNL

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Imre Deak
On Thu, Feb 22, 2018 at 11:33:10PM +0200, Pandiyan, Dhinakaran wrote: > > > > On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > > On Thu, Feb 22, 2018 at 12:28:50PM -0800, Dhinakaran Pandiyan wrote: > > > PSR on CNL requires AUX IO wells to be kept on and the existing AUX domain > > > for

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Ville Syrjälä
On Thu, Feb 22, 2018 at 09:33:10PM +, Pandiyan, Dhinakaran wrote: > > > > On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > > On Thu, Feb 22, 2018 at 12:28:50PM -0800, Dhinakaran Pandiyan wrote: > > > PSR on CNL requires AUX IO wells to be kept on and the existing AUX domain > > > for

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Pandiyan, Dhinakaran
On Thu, 2018-02-22 at 12:44 -0800, Rodrigo Vivi wrote: > On Thu, Feb 22, 2018 at 12:28:50PM -0800, Dhinakaran Pandiyan wrote: > > PSR on CNL requires AUX IO wells to be kept on and the existing AUX domain > > for AUX-A enables DC_OFF well too. This is not required, so add a new > > AUX_IO_A doma

Re: [Intel-gfx] [PATCH v3] drm/i915/psr: New power domain for AUX IO.

2018-02-22 Thread Rodrigo Vivi
On Thu, Feb 22, 2018 at 12:28:50PM -0800, Dhinakaran Pandiyan wrote: > PSR on CNL requires AUX IO wells to be kept on and the existing AUX domain > for AUX-A enables DC_OFF well too. This is not required, so add a new > AUX_IO_A domain for AUX-A to allow DC states to remain enabled. Other AUX > cha