Patch Set 27:

Problem here is can't identify which CPU PCIE is used if just check 06.0.. Still need a flag for CPU port?

	# Enable CPU PCIE RP 1 using PEG CLK 0
register "cpu_pcie_rp[CPU_RP(1)]" = "{
.flags = PCIE_RP_CLK_REQ_UNUSED,
.clk_src = 0,
}"
# Enable PCU PCIE PEG Slot 1 and 2
register "cpu_pcie_rp[CPU_RP(2)]" = "{
.flags = PCIE_RP_CLK_REQ_UNUSED,
.clk_src = 3,
}"
register "cpu_pcie_rp[CPU_RP(3)]" = "{
.flags = PCIE_RP_CLK_REQ_UNUSED,
.clk_src = 4,
}"

in the EDS, I see:
"The ADL-P processor PCI Express* has two interfaces:
• One 8-lane (x8) port supporting PCIE to gen 5.0 or below.
• Two 4-lane (x4) port supporting PCIE gen 4.0 or below"

and I read the rest to say that:
00:01.0 (pcie5) has 1 x8 port (pcie5 or lower)
00:06.0 (pci4_0) has 1 x4 port (pcie4 or lower)
00:06.2 (pci4_1) has 1 x4 port (pcie4 or lower)

also for the PCIE5 port:
"Support is also provided for narrow width and use devices with lower number of lanes (that is, usage on x4 configuration), however further bifurcation is not supported"

and for the PCIE4 ports only support 1x4 and 1x4 reversed, so no way to get bifurcation to support x8 on those ports I think.

View Change

To view, visit change 48340. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0b390e43f8e99b19cfad178139b86a2f77d7a57b
Gerrit-Change-Number: 48340
Gerrit-PatchSet: 27
Gerrit-Owner: EricR Lai <ericr_lai@compal.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Meera Ravindranath <meera.ravindranath@intel.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Tue, 29 Dec 2020 20:14:36 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment