Re: [Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-16 Thread Imre Deak
On Wed, 2013-10-16 at 00:09 +0200, Daniel Vetter wrote: On Tue, Oct 15, 2013 at 8:15 PM, Imre Deak imre.d...@intel.com wrote: Related to this: I made intel_encoder_get_hw_state() only check if the power well is on and return false if it's not to indicate that the encoder is off. I also

Re: [Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-15 Thread Ville Syrjälä
On Mon, Oct 14, 2013 at 04:07:44PM -0700, Jesse Barnes wrote: This set adds bits needed for runtime power support, currently only lightly tested on VLV/BYT: 1) suspend/resume callbacks for different platforms 2) save/restore of display state across a power well toggle 3) get/put of

Re: [Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-15 Thread Daniel Vetter
On Mon, Oct 14, 2013 at 04:07:44PM -0700, Jesse Barnes wrote: 5) more testing - I think the load time ref is still busted here and on HSW I've chatted quite a bit yesterday with Paulo about how we can test runtime pm better, since he wanted to get started with testing D3 while vpg is

Re: [Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-15 Thread Jesse Barnes
On Tue, 15 Oct 2013 15:16:11 +0300 Imre Deak imre.d...@intel.com wrote: On Tue, 2013-10-15 at 11:06 +0300, Ville Syrjälä wrote: On Mon, Oct 14, 2013 at 04:07:44PM -0700, Jesse Barnes wrote: This set adds bits needed for runtime power support, currently only lightly tested on VLV/BYT:

Re: [Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-15 Thread Imre Deak
On Tue, 2013-10-15 at 09:23 -0700, Jesse Barnes wrote: On Tue, 15 Oct 2013 15:16:11 +0300 Imre Deak imre.d...@intel.com wrote: On Tue, 2013-10-15 at 11:06 +0300, Ville Syrjälä wrote: On Mon, Oct 14, 2013 at 04:07:44PM -0700, Jesse Barnes wrote: This set adds bits needed for runtime

Re: [Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-15 Thread Daniel Vetter
On Tue, Oct 15, 2013 at 8:15 PM, Imre Deak imre.d...@intel.com wrote: Related to this: I made intel_encoder_get_hw_state() only check if the power well is on and return false if it's not to indicate that the encoder is off. I also thought of doing the same as you and take a ref instead, not

[Intel-gfx] [RFC] Runtime display PM for VLV/BYT

2013-10-14 Thread Jesse Barnes
This set adds bits needed for runtime power support, currently only lightly tested on VLV/BYT: 1) suspend/resume callbacks for different platforms 2) save/restore of display state across a power well toggle 3) get/put of display power well in critical places The TODO list still has a few