Re: [Intel-gfx] [PATCH 2/2] drm/i915/bxt: Fix sanity check for BIOS RC6 setup

2016-07-01 Thread Kamble, Sagar A
On 7/1/2016 2:45 PM, Imre Deak wrote: On pe, 2016-07-01 at 12:19 +0530, Kamble, Sagar A wrote: Have seen BIOS having option "RC6" disabled and "GTPM" enabled for cases where there are RC6 specific issues. It's possible although I haven't seen any based on the specs I have and the tests I ran

Re: [Intel-gfx] [PATCH 2/2] drm/i915/bxt: Fix sanity check for BIOS RC6 setup

2016-07-01 Thread Imre Deak
On pe, 2016-07-01 at 12:19 +0530, Kamble, Sagar A wrote: > Have seen BIOS having option "RC6" disabled and "GTPM" enabled for casesĀ  > where there are RC6 specific issues. It's possible although I haven't seen any based on the specs I have and the tests I ran. In any case the checks I added shoul

Re: [Intel-gfx] [PATCH 2/2] drm/i915/bxt: Fix sanity check for BIOS RC6 setup

2016-06-30 Thread Kamble, Sagar A
Have seen BIOS having option "RC6" disabled and "GTPM" enabled for cases where there are RC6 specific issues. GTPM option entails setup for other features as well I guess. In such cases - Can we output some DRM_INFO log saying BIOS has disabled RC6 although setup is available. Do we need to al

[Intel-gfx] [PATCH 2/2] drm/i915/bxt: Fix sanity check for BIOS RC6 setup

2016-06-29 Thread Imre Deak
BXT BIOS has two options related to GPU power management: "RC6(Render Standby)" and "GT PM Support". The assumption so far was that disabling either of these options would leave RC6 uninitialized. According to my tests this isn't so: for a proper RC6 setup we only need the "GT PM Support" option to