intel_extreme: fix regression introduced in hrev55419.
Apparently my comment about the width and height being swapped in this register was not visible enough, so I make it a bit more obvious by adding some uppercase. Change-Id: I27621032d071ed09f82aa109f37482178351db04 Reviewed-on: https://review.haiku-os.org/c/haiku/+/4664 Reviewed-by: waddlesplash <[email protected]>
This commit is contained in:
committed by
waddlesplash
parent
a1f7cb5d4e
commit
0cabd8891f
@@ -209,11 +209,13 @@ Pipe::ConfigureScalePos(display_mode* target)
|
|||||||
if (gInfo->shared_info->device_type.Generation() <= 4) {
|
if (gInfo->shared_info->device_type.Generation() <= 4) {
|
||||||
// This is "reserved" on G35 and GMA965, but needed on 945 (for which
|
// This is "reserved" on G35 and GMA965, but needed on 945 (for which
|
||||||
// there is no public documentation), and I assume earlier devices as
|
// there is no public documentation), and I assume earlier devices as
|
||||||
// well. Note that the height and width are swapped when compared to
|
// well.
|
||||||
// the other registers.
|
//
|
||||||
|
// IMPORTANT WARNING: height and width are swapped when compared to the other registers!
|
||||||
|
// Be careful when editing this code and don't accidentally swap them!
|
||||||
write32(INTEL_DISPLAY_A_IMAGE_SIZE + fPipeOffset,
|
write32(INTEL_DISPLAY_A_IMAGE_SIZE + fPipeOffset,
|
||||||
((uint32)(target->timing.h_display - 1) << 16)
|
((uint32)(target->timing.v_display - 1) << 16)
|
||||||
| ((uint32)target->timing.v_display - 1));
|
| ((uint32)target->timing.h_display - 1));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user