Re: [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

["S, Deepak" <deepak.s@intel.com>]

 >> I was polling VLV_GTLC_SURVIVABILITY_REG and VLV_GTLC_PW_STATUS to make
 >> sure both render and media wells and the gfx clock remain off, and I
 >> was also monitoring vnn via svid. While that was going on I just rewrote
 >> PUNIT_REG_GPU_FREQ_REQ to make Punit change the frequency, and sure
 >> enough it did, and svid showed me that vnn had also changed. So it
 >> appears there's no need to have the gfx clock on to change its frequency
 >> on this BYT.
 >>
 >> I wonder if this part of the workaround was only needed on older parts.
 >> Deepak, any ideas?
 >
 > Yes ville, this was added initial for older parts and force gfx clock
 > was part of the workaround.
 > We have not verified this on newer parts. Let me check with hw guys to
 > see if workaround still exits and when this was fixed.

Hi Ville, Got the confirmation from HW team, this WA as been fixed in 
latest stepping, we can remove the force gfx clock, mask and request 
only the min freq when we are idle.

You will submit patch will fix or you want me to do it?

Thanks
Deepak