Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-22 Thread Jani Nikula
On Thu, 17 May 2018, Dhinakaran Pandiyan wrote: > Yeah, agreed on starting with a clean slate. > > The offending commit claims alternate mode was intended to be used for > PSR testing. I don't see any psr_basic failures in BAT or any other PSR > sub-tests failing on shards, must have been a non-CI

Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-18 Thread Dhinakaran Pandiyan
On Thursday, May 17, 2018 12:44:30 PM PDT Dhinakaran Pandiyan wrote: > On Thu, 2018-05-17 at 10:33 +0300, Jani Nikula wrote: > > On Thu, 17 May 2018, Jani Nikula wrote: > > > On Wed, 16 May 2018, Dhinakaran Pandiyan > > > > > .com> wrote: > > > > On Wed, 2018-05-16 at 11:01 +0300, Jani Nikula wr

Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-17 Thread Dhinakaran Pandiyan
On Thu, 2018-05-17 at 10:33 +0300, Jani Nikula wrote: > On Thu, 17 May 2018, Jani Nikula wrote: > > > > On Wed, 16 May 2018, Dhinakaran Pandiyan > .com> wrote: > > > > > > On Wed, 2018-05-16 at 11:01 +0300, Jani Nikula wrote: > > > > > > > > This reverts commit dc911f5bd8aacfcf8aabd5c26c88e04c

Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-17 Thread Dhinakaran Pandiyan
On Thu, 2018-05-17 at 10:33 +0300, Jani Nikula wrote: > On Thu, 17 May 2018, Jani Nikula wrote: > > > > On Wed, 16 May 2018, Dhinakaran Pandiyan > .com> wrote: > > > > > > On Wed, 2018-05-16 at 11:01 +0300, Jani Nikula wrote: > > > > > > > > This reverts commit dc911f5bd8aacfcf8aabd5c26c88e04c

Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-17 Thread Jani Nikula
On Thu, 17 May 2018, Jani Nikula wrote: > On Wed, 16 May 2018, Dhinakaran Pandiyan > wrote: >> On Wed, 2018-05-16 at 11:01 +0300, Jani Nikula wrote: >>> This reverts commit dc911f5bd8aacfcf8aabd5c26c88e04c837a938e. >>> >>> Per the report, no matter what display mode you select with xrandr, >>>

Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-17 Thread Jani Nikula
On Wed, 16 May 2018, Dhinakaran Pandiyan wrote: > On Wed, 2018-05-16 at 11:01 +0300, Jani Nikula wrote: >> This reverts commit dc911f5bd8aacfcf8aabd5c26c88e04c837a938e. >> >> Per the report, no matter what display mode you select with xrandr, >> the >> i915 driver will always select the alternate

Re: [Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-16 Thread Dhinakaran Pandiyan
On Wed, 2018-05-16 at 11:01 +0300, Jani Nikula wrote: > This reverts commit dc911f5bd8aacfcf8aabd5c26c88e04c837a938e. > > Per the report, no matter what display mode you select with xrandr, > the > i915 driver will always select the alternate fixed mode. For the > reporter this means that the disp

[Intel-gfx] [PATCH] Revert "drm/i915/edp: Allow alternate fixed mode for eDP if available."

2018-05-16 Thread Jani Nikula
This reverts commit dc911f5bd8aacfcf8aabd5c26c88e04c837a938e. Per the report, no matter what display mode you select with xrandr, the i915 driver will always select the alternate fixed mode. For the reporter this means that the display will always run at 40Hz which is quite annoying. This may be d