From e109a6a47f2c1e3fc6ad162645d3654d190ea455 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Thu, 13 Oct 2022 15:16:45 +0200 Subject: [PATCH] mb/amd/padmelon/bootblock/OemCustomize: add TODO for Prairie Falcon The PCIe port descriptor list seems to be specific to Merlin Falcon and Prairie Falcon has a different PCIe root port configuration. Since I neither have the board nor the different APUs, I just add a comment about this instead of trying to come up with a PCIe port descriptor list that may or may not work properly on Prairie Falcon APUs. Signed-off-by: Felix Held Change-Id: I8e1eb67a8f684297bbefc6e2593250d7bd45593f Reviewed-on: https://review.coreboot.org/c/coreboot/+/68407 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/mainboard/amd/padmelon/bootblock/OemCustomize.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/mainboard/amd/padmelon/bootblock/OemCustomize.c b/src/mainboard/amd/padmelon/bootblock/OemCustomize.c index 6937429934..1f4bd3cb39 100644 --- a/src/mainboard/amd/padmelon/bootblock/OemCustomize.c +++ b/src/mainboard/amd/padmelon/bootblock/OemCustomize.c @@ -2,6 +2,12 @@ #include +/* + * TODO: + * Check if a separate PCIe port list is needed for Prairie Falcon APUs. Only Merlin Falcon has + * PCIe root ports on the functions of bus 0 device 3. + */ + static const PCIe_PORT_DESCRIPTOR PortList[] = { /* * Init Port descriptor (PCIe port, Lanes 8-15,