commit 005f9cabe66fb6e5075b8aa8c0a54192b253ace5
author Nico Huber <nico.huber@secunet.com> Wed Jul 03 16:57:56 2024 +0200
committer Nico Huber <nico.h@gmx.de> Sat Jun 06 10:28:13 2026 +0000
tree 4b83ce3f3bde0e6ef522c23d285b91b5340d4b78
parent fa9a0d295928c5a9703662601e1b19facc244822

gma ironlake: Move FDI override into Connectors.Prepare()

Tested w/ Kontron/KTQM77 and three monitors. If the second pipe
is enabled late, after the third pipe, with a high-resolution
monitor, we bail out correctly with

  "Mode requirements exceed available bandwidth!"

Change-Id: Iac5334e91b1e706dca21e74845a3227c3b9b6503
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.sourcearcade.org/c/libgfxinit/+/463
Tested-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>

common/hw-gfx-gma-config_helpers.adb

diff --git a/common/hw-gfx-gma-config_helpers.adb b/common/hw-gfx-gma-config_helpers.adb
index b43f9e2..348c3e2 100644
--- a/common/hw-gfx-gma-config_helpers.adb
+++ b/common/hw-gfx-gma-config_helpers.adb
@@ -14,7 +14,6 @@
 
 with HW.GFX.GMA.Connector_Info;
 with HW.GFX.GMA.DP_Info;
-with HW.GFX.GMA.Registers;
 
 with HW.Debug;
 
@@ -172,25 +171,12 @@
          Port_Cfg.Mode.H_Visible = Port_Cfg'Old.Mode.H_Visible and
          Port_Cfg.Mode.V_Visible = Port_Cfg'Old.Mode.V_Visible
    is
-      FDI_TX_CTL_FDI_TX_ENABLE : constant := 1 * 2 ** 31;
-      Enabled : Boolean;
    begin
       Port_Cfg.FDI.Receiver_Caps.Max_Link_Rate    := DP_Bandwidth_2_7;
       Port_Cfg.FDI.Receiver_Caps.Max_Lane_Count   :=
          Config.FDI_Lane_Count (Port_Cfg.Port);
       Port_Cfg.FDI.Receiver_Caps.Enhanced_Framing := True;
 
-      if Config.Has_FDI_C and then Port_Cfg.Port = DIGI_C then
-         -- if DIGI_D enabled: (FDI names are off by one)
-         Registers.Is_Set_Mask
-           (Register => Registers.FDI_TX_CTL_C,
-            Mask     => FDI_TX_CTL_FDI_TX_ENABLE,
-            Result   => Enabled);
-         if Enabled then
-            Port_Cfg.FDI.Receiver_Caps.Max_Lane_Count := DP_Lane_Count_2;
-         end if;
-      end if;
-
       DP_Info.Preferred_Link_Setting (Port_Cfg.FDI, Port_Cfg.Mode, Success);
    end Configure_FDI_Link;
 

common/ironlake/hw-gfx-gma-connectors.adb

diff --git a/common/ironlake/hw-gfx-gma-connectors.adb b/common/ironlake/hw-gfx-gma-connectors.adb
index 11ecdf6..d813fe4 100644
--- a/common/ironlake/hw-gfx-gma-connectors.adb
+++ b/common/ironlake/hw-gfx-gma-connectors.adb
@@ -14,6 +14,8 @@
 --
 
 with HW.GFX.GMA.Config;
+with HW.GFX.GMA.DP_Info;
+with HW.GFX.GMA.Registers;
 with HW.GFX.GMA.Panel;
 with HW.GFX.GMA.Connectors.EDP;
 with HW.GFX.GMA.Connectors.FDI;
@@ -39,8 +41,33 @@
       Port_Cfg : in out Port_Config;
       Success  :    out Boolean)
    is
+      -- Override lane count for FDI_B if FDI_C is used
+      procedure Override_FDI_Link
+      with
+         Post => Port_Cfg.Mode = Port_Cfg'Old.Mode
+      is
+         FDI_TX_CTL_FDI_TX_ENABLE : constant := 1 * 2 ** 31;
+         Enabled : Boolean;
+      begin
+         if Config.Has_FDI_C and then Port_Cfg.Port = DIGI_C then
+            -- if DIGI_D enabled: (FDI names are off by one)
+            Registers.Is_Set_Mask
+              (Register => Registers.FDI_TX_CTL_C,
+               Mask     => FDI_TX_CTL_FDI_TX_ENABLE,
+               Result   => Enabled);
+            if Enabled then
+               Port_Cfg.FDI.Receiver_Caps.Max_Lane_Count := DP_Lane_Count_2;
+            end if;
+         end if;
+
+         DP_Info.Preferred_Link_Setting (Port_Cfg.FDI, Port_Cfg.Mode, Success);
+      end Override_FDI_Link;
    begin
-      Success := True;
+      if Port_Cfg.Is_FDI then
+         Override_FDI_Link;
+      else
+         Success := True;
+      end if;
    end Prepare;
 
    ----------------------------------------------------------------------------