[S] Change in coreboot[master]: soc/intel/cmn/graphics: Implement API for IGD to join the MBUS

[Subrata Banik (Code Review)]

Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/76928?usp=email )

Change subject: soc/intel/cmn/graphics: Implement API for IGD to join the MBUS
......................................................................

soc/intel/cmn/graphics: Implement API for IGD to join the MBUS

This patch implements `.final` hooks for the IGD device to perform the
required operations before handing the controlto the payload or OS.

The MBUS (Memory Bus) is a high-speed interface that connects the
graphics controller to the system memory. It provides a dedicated data
path for graphics data, which helps to improve graphics performance.

The MBUS is a key technology that helps to make the Intel i915 driver
powerful and versatile graphics drivers available. It provides the
high-speed data transfer capabilities that are essential for smooth
and responsive graphics performance.

Enable this config to ensure that the Intel GFX controller joins the
MBUS before the i915 driver is loaded. This is necessary to prevent
the i915 driver from re-initializing the display if the firmware has
already initialized it. Without this config, the i915 driver will
initialize the display to bring up the login screen although the
firmware has initialized the display using the GFX MMIO registers and
framebuffer.

When enabled, saves 75ms-80ms of the boot time by avoiding redundant
display initialization by kernel graphics driver (i.e., i915_gfx).

BUG=b:284799726
TEST=Able to build and boot google/rex

Change-Id: I60ae76dc783383e027e66edbcdeeb535472caeb1
Signed-off-by: Subrata Banik 
---
M src/soc/intel/common/block/graphics/Kconfig
M src/soc/intel/common/block/graphics/graphics.c
2 files changed, 34 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/28/76928/1

diff --git a/src/soc/intel/common/block/graphics/Kconfig b/src/soc/intel/common/block/graphics/Kconfig
index d586fd8..30e4f8e 100644
--- a/src/soc/intel/common/block/graphics/Kconfig
+++ b/src/soc/intel/common/block/graphics/Kconfig
@@ -37,4 +37,24 @@
      Ignore BAR0(offset 0x10)'s pre-fetchable attribute to use non-prefetchable
      MMIO to fix OS display driver failure.

+config SOC_INTEL_GFX_MBUS_JOIN
+	bool
+	help
+	  The MBUS (Memory Bus) is a high-speed interface that connects the graphics
+	  controller to the system memory. It provides a dedicated data path for graphics
+	  data, which helps to improve graphics performance.
+
+	  The MBUS is a key technology that helps to make the Intel i915 driver powerful
+	  and versatile graphics drivers available. It provides the high-speed data transfer
+	  capabilities that are essential for smooth and responsive graphics performance.
+
+	  Enable this config to ensure that the Intel GFX controller joins the MBUS before the
+	  i915 driver is loaded. This is necessary to prevent the i915 driver from re-initializing
+	  the display if the firmware has already initialized it. Without this config, the i915
+	  driver will initialize the display to bring up the login screen although the firmware
+	  has initialized the display using the GFX MMIO registers and framebuffer.
+
+	  When enabled, saves 75ms-80ms of the boot time by avoiding redundent display
+	  initialization by kernel graphics driver (i.e., i915_gfx).
+
 endif
diff --git a/src/soc/intel/common/block/graphics/graphics.c b/src/soc/intel/common/block/graphics/graphics.c
index 9b48e1d..556af79 100644
--- a/src/soc/intel/common/block/graphics/graphics.c
+++ b/src/soc/intel/common/block/graphics/graphics.c
@@ -16,6 +16,10 @@
 #include <soc/pci_devs.h>
 #include <types.h>

+#define GFX_MBUS_CTL			0x4438C
+#define GFX_MBUS_JOIN			BIT(31)
+#define GFX_MBUS_HASHING_MODE		BIT(30)
+
 /* SoC Overrides */
 __weak void graphics_soc_panel_init(struct device *dev)
 {
@@ -187,12 +191,21 @@
    }
 }

+static void graphics_dev_final(struct device *dev)
+{
+	pci_dev_request_bus_master(dev);
+
+	if (CONFIG(SOC_INTEL_GFX_MBUS_JOIN))
+		graphics_gtt_rmw(GFX_MBUS_CTL, ~0, GFX_MBUS_JOIN | GFX_MBUS_HASHING_MODE);
+
+}
+
 const struct device_operations graphics_ops = {
    .read_resources		= graphics_dev_read_resources,
    .set_resources		= pci_dev_set_resources,
    .enable_resources	= pci_dev_enable_resources,
    .init			= gma_init,
-	.final			= pci_dev_request_bus_master,
+	.final			= graphics_dev_final,
    .ops_pci		= &pci_dev_ops_pci,
 #if CONFIG(HAVE_ACPI_TABLES)
    .acpi_fill_ssdt		= gma_generate_ssdt,