[Intel-gfx] [PATCH] drm/i915: Fix incorrect definition of ADPA HSYNC and VSYNC bits

[Patrik Jakobsson]

On Wed, Mar 6, 2013 at 12:17 AM, Paul Menzel
<paulepanter at users.sourceforge.net> wrote:
> Dear Patrik,

Hi Paul

> Am Dienstag, den 05.03.2013, 19:09 +0100 schrieb Patrik Jakobsson:
>> Disable bits for ADPA HSYNC and VSYNC where mixed up resulting in suspend
>> becoming standby and vice versa.
>
> nice find. Could you elaborate on the symptoms please as I have never
> experienced any issues with suspend and resume on my ASUS Eee PC 701 4G?

Ville did all the finding, I just wrote the patch to make sure it wasn't
forgotten.

This isn't related to suspend/resume of the system, but putting external
monitors in suspend or standby state. When in standby, the monitor consumes more
power than in suspend but wakes up faster.

>> Fixed by swapping their bit position.
>>
>> Reported-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Where did Ville report this? Maybe my questions are already answered
> there.

He found it while looking at another patch I've sent:
http://lists.freedesktop.org/archives/intel-gfx/2013-March/025418.html

-Patrik