Re: [Intel-gfx] [PATCH] drm/i915/bdw: don't try to check IPS state on BDW v2

2014-01-08 Thread Paulo Zanoni
2014/1/7 Jesse Barnes : > According to Art, we don't have a way to read back the state reliably at > runtime, through the control reg or the mailbox, at least not without risking > disabling it again. So drop the readout and checking on BDW. > > v2: drop TODO comment (Paulo) > move POSTING_REA

Re: [Intel-gfx] [PATCH] drm/i915/bdw: don't try to check IPS state on BDW

2014-01-07 Thread Jesse Barnes
On Tue, 7 Jan 2014 17:49:05 -0200 Paulo Zanoni wrote: > 2014/1/6 Jesse Barnes : > > According to Art, we don't have a way to read back the state reliably at > > runtime, at least not without risking disabling it again. So drop the > > readout and checking on BDW. > > Based on that, we also need

[Intel-gfx] [PATCH] drm/i915/bdw: don't try to check IPS state on BDW v2

2014-01-07 Thread Jesse Barnes
According to Art, we don't have a way to read back the state reliably at runtime, through the control reg or the mailbox, at least not without risking disabling it again. So drop the readout and checking on BDW. v2: drop TODO comment (Paulo) move POSTING_READ of control reg under HSW branch i

Re: [Intel-gfx] [PATCH] drm/i915/bdw: don't try to check IPS state on BDW

2014-01-07 Thread Paulo Zanoni
2014/1/6 Jesse Barnes : > According to Art, we don't have a way to read back the state reliably at > runtime, at least not without risking disabling it again. So drop the > readout and checking on BDW. Based on that, we also need to: - Replace the TODO comment at hsw_enable_ips() with something e

[Intel-gfx] [PATCH] drm/i915/bdw: don't try to check IPS state on BDW

2014-01-06 Thread Jesse Barnes
According to Art, we don't have a way to read back the state reliably at runtime, at least not without risking disabling it again. So drop the readout and checking on BDW. References: https://bugs.freedesktop.org/show_bug.cgi?id=71906 Signed-off-by: Jesse Barnes --- drivers/gpu/drm/i915/intel_d