vc/amd/fsp/picasso: document DXIO lane number mapping

Haven't found the official documentation for the DXIO lane mapping on
Pollock, so I had to guess that from the working configurations used in
google/dalboz and amd/cereme.

Change-Id: I53aac0aeba8466ae456f0f935114b587b64eeeaa
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/44063
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Felix Held 2020-07-30 16:13:35 +02:00
parent 7d6dae6870
commit 42d5294793
1 changed files with 26 additions and 0 deletions

View File

@ -113,6 +113,32 @@ typedef struct __packed {
* Picasso DXIO Descriptor: Used for assigning lanes to PCIe/SATA/XGBE engines, configure
* bifurcation and other settings. Beware that the lane numbers in here are the logical and not
* the physical lane numbers!
*
* Picasso DXIO lane mapping:
*
* physical | logical | protocol
* ---------|---------|-----------
* GFX[7:0] | [15:8] | PCIe
* GPP[3:0] | [7:4] | PCIe
* GPP[5:4] | [1:0] | PCIe, XGBE
* GPP[7:6] | [3:2] | PCIe, SATA
*
* Dali has less DXIO connectivity than Picasso:
*
* physical | logical | protocol
* ---------|---------|-----------
* GFX[3:0] | [11:8] | PCIe
* GPP[1:0] | [5:4] | PCIe
* GPP[5:4] | [1:0] | PCIe, XGBE
* GPP[7:6] | [3:2] | SATA
*
* Pollock has even less DXIO lanes and the mapping of GPP lane numbers to the logical lane
* numbers differs to Picasso/Dali:
*
* physical | logical | protocol
* ---------|---------|----------
* GPP[1:0] | [1:0] | PCIe
* GPP[3:2] | [5:4] | PCIe
*/
typedef struct __packed {
uint8_t engine_type;