[coreboot-gerrit] Change in libgfxinit[master]: gma config: Introduce Have_HDMI_Buf_Override

[Nico Huber (Code Review)]

Nico Huber has uploaded this change for review. ( https://review.coreboot.org/27049


Change subject: gma config: Introduce Have_HDMI_Buf_Override
......................................................................

gma config: Introduce Have_HDMI_Buf_Override

Change-Id: Id00ae200cc8d08ce245d57b2886a40f0fea10167
Signed-off-by: Nico Huber <nico.h at gmx.de>
---
M common/haswell/hw-gfx-gma-connectors-ddi-buffers.adb
M common/haswell_shared/hw-gfx-gma-connectors-ddi.adb
M common/hw-gfx-gma-config.ads.template
3 files changed, 4 insertions(+), 7 deletions(-)



  git pull ssh://review.coreboot.org:29418/libgfxinit refs/changes/49/27049/1

diff --git a/common/haswell/hw-gfx-gma-connectors-ddi-buffers.adb b/common/haswell/hw-gfx-gma-connectors-ddi-buffers.adb
index 57066b9..baceb10 100644
--- a/common/haswell/hw-gfx-gma-connectors-ddi-buffers.adb
+++ b/common/haswell/hw-gfx-gma-connectors-ddi-buffers.adb
@@ -120,11 +120,7 @@
    procedure Translations (Trans : out Buf_Trans_Array; Port : Digital_Port)
    is
       HDMI_Trans : constant DDI_HDMI_Buf_Trans_Range :=
-        (if (Config.CPU = Broadwell and
-             Config.DDI_HDMI_Buffer_Translation in Broadwell_HDMI_Range)
-            or
-            (Config.CPU /= Broadwell and
-             Config.DDI_HDMI_Buffer_Translation in Haswell_HDMI_Range)
+        (if Config.Have_HDMI_Buf_Override
          then Config.DDI_HDMI_Buffer_Translation
          else Config.Default_DDI_HDMI_Buffer_Translation);
    begin
diff --git a/common/haswell_shared/hw-gfx-gma-connectors-ddi.adb b/common/haswell_shared/hw-gfx-gma-connectors-ddi.adb
index 88f2b22..7f179ff 100644
--- a/common/haswell_shared/hw-gfx-gma-connectors-ddi.adb
+++ b/common/haswell_shared/hw-gfx-gma-connectors-ddi.adb
@@ -679,8 +679,7 @@
          then
             declare
                HDMI_Level : constant DDI_Phy.HDMI_Buf_Trans_Range :=
-                 (if Config.DDI_HDMI_Buffer_Translation
-                     in DDI_Phy.HDMI_Buf_Trans_Range
+                 (if Config.Have_HDMI_Buf_Override
                   then Config.DDI_HDMI_Buffer_Translation
                   else Config.Default_DDI_HDMI_Buffer_Translation);
             begin
diff --git a/common/hw-gfx-gma-config.ads.template b/common/hw-gfx-gma-config.ads.template
index 20ef5c0..af55575 100644
--- a/common/hw-gfx-gma-config.ads.template
+++ b/common/hw-gfx-gma-config.ads.template
@@ -35,6 +35,8 @@
 
    ----------------------------------------------------------------------------
 
+   Have_HDMI_Buf_Override  : constant Boolean :=
+                                             DDI_HDMI_Buffer_Translation /= -1;
    Default_MMIO_Base_Set   : constant Boolean := Default_MMIO_Base /= 0;
 
    Has_Internal_Display    : constant Boolean := Internal_Display /= None;

-- 
To view, visit https://review.coreboot.org/27049
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: libgfxinit
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id00ae200cc8d08ce245d57b2886a40f0fea10167
Gerrit-Change-Number: 27049
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h at gmx.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180612/b3a4bcc5/attachment-0001.html>