[coreboot-gerrit] Change in ...coreboot[master]: siemens/mc_apl5: Disable PCI clock outputs on XIO bridges

[Mario Scheithauer (Code Review)]

Mario Scheithauer has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/29882

Change subject: siemens/mc_apl5: Disable PCI clock outputs on XIO bridges
......................................................................

siemens/mc_apl5: Disable PCI clock outputs on XIO bridges

On this mainboard there are legacy PCI device, which are connected to
different PCIe root ports via PCIe-2-PCI bridges. This patch disables
the unused PCI clock outputs on the XIO2001 PCI Express to PCI Bridges.

Change-Id: Id36e39c4568f5dd241cd864d2e75365abd0f2a91
Signed-off-by: Mario Scheithauer 
---
M src/mainboard/siemens/mc_apl1/variants/mc_apl5/mainboard.c
1 file changed, 16 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/29882/1

diff --git a/src/mainboard/siemens/mc_apl1/variants/mc_apl5/mainboard.c b/src/mainboard/siemens/mc_apl1/variants/mc_apl5/mainboard.c
index 5ad3172..e859380 100644
--- a/src/mainboard/siemens/mc_apl1/variants/mc_apl5/mainboard.c
+++ b/src/mainboard/siemens/mc_apl1/variants/mc_apl5/mainboard.c
@@ -52,11 +52,6 @@
     */
    pcr_write16(PID_ITSS, 0x314c, 0x0321);

-	/* Disable clock outputs 1-5 (CLKOUT) for XIO2001 PCIe to PCI Bridge. */
-	dev = dev_find_device(PCI_VENDOR_ID_TI, PCI_DEVICE_ID_TI_XIO2001, 0);
-	if (dev)
-		pci_write_config8(dev, 0xd8, 0x3e);
-
    /* Enable CLKRUN_EN for power gating LPC */
    lpc_enable_pci_clk_cntl();

@@ -82,6 +77,22 @@
    	cmd = pci_read_config16(dev, PCI_COMMAND);
    	cmd |= PCI_COMMAND_MASTER;
    	pci_write_config16(dev, PCI_COMMAND, cmd);
+
+		/* Disable clock outputs 0-3 (CLKOUT) for upstream XIO2001 PCIe
+		 * to PCI Bridge. */
+		struct device *parent = dev->bus->dev;
+		if (parent && parent->device == PCI_DEVICE_ID_TI_XIO2001)
+			pci_write_config8(parent, 0xd8, 0x0f);
+	}
+
+	/* Disable clock outputs 1-5 (CLKOUT) for another XIO2001 PCIe to PCI
+	 * Bridge on this mainboard.
+	 */
+	dev = dev_find_device(PCI_VENDOR_ID_SIEMENS, 0x403f, 0);
+	if (dev) {
+		struct device *parent = dev->bus->dev;
+		if (parent && parent->device == PCI_DEVICE_ID_TI_XIO2001)
+			pci_write_config8(parent, 0xd8, 0x3e);
    }
 }