summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/mainboard/google/zork/variants/baseboard/devicetree_dalboz.cb12
-rw-r--r--src/mainboard/google/zork/variants/baseboard/devicetree_trembyle.cb12
-rw-r--r--src/soc/amd/picasso/chip.h9
-rw-r--r--src/soc/amd/picasso/fsp_params.c13
-rw-r--r--src/vendorcode/amd/fsp/picasso/FspsUpd.h8
5 files changed, 23 insertions, 31 deletions
diff --git a/src/mainboard/google/zork/variants/baseboard/devicetree_dalboz.cb b/src/mainboard/google/zork/variants/baseboard/devicetree_dalboz.cb
index 4a01a12cc0..6f72c8581b 100644
--- a/src/mainboard/google/zork/variants/baseboard/devicetree_dalboz.cb
+++ b/src/mainboard/google/zork/variants/baseboard/devicetree_dalboz.cb
@@ -54,7 +54,7 @@ chip soc/amd/picasso
register "has_usb2_phy_tune_params" = "1"
# Controller0 Port0 Default
- register "usb_2_port_0_tune_params" = "{
+ register "usb_2_port_tune_params[0]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -67,7 +67,7 @@ chip soc/amd/picasso
}"
# Controller0 Port1 Default
- register "usb_2_port_1_tune_params" = "{
+ register "usb_2_port_tune_params[1]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -80,7 +80,7 @@ chip soc/amd/picasso
}"
# Controller0 Port2 Default
- register "usb_2_port_2_tune_params" = "{
+ register "usb_2_port_tune_params[2]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -93,7 +93,7 @@ chip soc/amd/picasso
}"
# Controller0 Port3 Default
- register "usb_2_port_3_tune_params" = "{
+ register "usb_2_port_tune_params[3]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -106,7 +106,7 @@ chip soc/amd/picasso
}"
# Controller1 Port0 Default
- register "usb_2_port_4_tune_params" = "{
+ register "usb_2_port_tune_params[4]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -119,7 +119,7 @@ chip soc/amd/picasso
}"
# Controller1 Port1 Default
- register "usb_2_port_5_tune_params" = "{
+ register "usb_2_port_tune_params[5]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
diff --git a/src/mainboard/google/zork/variants/baseboard/devicetree_trembyle.cb b/src/mainboard/google/zork/variants/baseboard/devicetree_trembyle.cb
index 986c444e14..d43063ca8e 100644
--- a/src/mainboard/google/zork/variants/baseboard/devicetree_trembyle.cb
+++ b/src/mainboard/google/zork/variants/baseboard/devicetree_trembyle.cb
@@ -54,7 +54,7 @@ chip soc/amd/picasso
register "has_usb2_phy_tune_params" = "1"
# Controller0 Port0 Default
- register "usb_2_port_0_tune_params" = "{
+ register "usb_2_port_tune_params[0]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -67,7 +67,7 @@ chip soc/amd/picasso
}"
# Controller0 Port1 Default
- register "usb_2_port_1_tune_params" = "{
+ register "usb_2_port_tune_params[1]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -80,7 +80,7 @@ chip soc/amd/picasso
}"
# Controller0 Port2 Default
- register "usb_2_port_2_tune_params" = "{
+ register "usb_2_port_tune_params[2]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -93,7 +93,7 @@ chip soc/amd/picasso
}"
# Controller0 Port3 Default
- register "usb_2_port_3_tune_params" = "{
+ register "usb_2_port_tune_params[3]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -106,7 +106,7 @@ chip soc/amd/picasso
}"
# Controller1 Port0 Default
- register "usb_2_port_4_tune_params" = "{
+ register "usb_2_port_tune_params[4]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
@@ -119,7 +119,7 @@ chip soc/amd/picasso
}"
# Controller1 Port1 Default
- register "usb_2_port_5_tune_params" = "{
+ register "usb_2_port_tune_params[5]" = "{
.com_pds_tune = 0x03,
.sq_rx_tune = 0x3,
.tx_fsls_tune = 0x3,
diff --git a/src/soc/amd/picasso/chip.h b/src/soc/amd/picasso/chip.h
index d3c9a0721b..64258b24ca 100644
--- a/src/soc/amd/picasso/chip.h
+++ b/src/soc/amd/picasso/chip.h
@@ -37,6 +37,8 @@ struct __packed usb2_phy_tune {
uint8_t tx_res_tune;
};
+#define USB_PORT_COUNT 6
+
struct soc_amd_picasso_config {
struct soc_amd_common_config common_config;
/*
@@ -134,12 +136,7 @@ struct soc_amd_picasso_config {
uint8_t xhci0_force_gen1;
uint8_t has_usb2_phy_tune_params;
- struct usb2_phy_tune usb_2_port_0_tune_params;
- struct usb2_phy_tune usb_2_port_1_tune_params;
- struct usb2_phy_tune usb_2_port_2_tune_params;
- struct usb2_phy_tune usb_2_port_3_tune_params;
- struct usb2_phy_tune usb_2_port_4_tune_params;
- struct usb2_phy_tune usb_2_port_5_tune_params;
+ struct usb2_phy_tune usb_2_port_tune_params[USB_PORT_COUNT];
};
typedef struct soc_amd_picasso_config config_t;
diff --git a/src/soc/amd/picasso/fsp_params.c b/src/soc/amd/picasso/fsp_params.c
index d280bffc47..8e6703e436 100644
--- a/src/soc/amd/picasso/fsp_params.c
+++ b/src/soc/amd/picasso/fsp_params.c
@@ -98,17 +98,16 @@ static void fsp_fill_pcie_ddi_descriptors(FSP_S_CONFIG *scfg)
static void fsp_usb_oem_customization(FSP_S_CONFIG *scfg,
const struct soc_amd_picasso_config *cfg)
{
- size_t num = sizeof(struct usb2_phy_tune);
+ ASSERT(FSPS_UPD_USB2_PORT_COUNT == USB_PORT_COUNT);
scfg->xhci0_force_gen1 = cfg->xhci0_force_gen1;
if (cfg->has_usb2_phy_tune_params) {
- memcpy(scfg->fch_usb_2_port0_phy_tune, &cfg->usb_2_port_0_tune_params, num);
- memcpy(scfg->fch_usb_2_port1_phy_tune, &cfg->usb_2_port_1_tune_params, num);
- memcpy(scfg->fch_usb_2_port2_phy_tune, &cfg->usb_2_port_2_tune_params, num);
- memcpy(scfg->fch_usb_2_port3_phy_tune, &cfg->usb_2_port_3_tune_params, num);
- memcpy(scfg->fch_usb_2_port4_phy_tune, &cfg->usb_2_port_4_tune_params, num);
- memcpy(scfg->fch_usb_2_port5_phy_tune, &cfg->usb_2_port_5_tune_params, num);
+ for (size_t i = 0; i < FSPS_UPD_USB2_PORT_COUNT; i++) {
+ memcpy(scfg->fch_usb_2_port_phy_tune[i],
+ &cfg->usb_2_port_tune_params[i],
+ sizeof(scfg->fch_usb_2_port_phy_tune[0]));
+ }
}
}
diff --git a/src/vendorcode/amd/fsp/picasso/FspsUpd.h b/src/vendorcode/amd/fsp/picasso/FspsUpd.h
index 4298b11b5f..5adbb81f6c 100644
--- a/src/vendorcode/amd/fsp/picasso/FspsUpd.h
+++ b/src/vendorcode/amd/fsp/picasso/FspsUpd.h
@@ -11,6 +11,7 @@
#define FSPS_UPD_DXIO_DESCRIPTOR_COUNT 8
#define FSPS_UPD_DDI_DESCRIPTOR_COUNT 4
+#define FSPS_UPD_USB2_PORT_COUNT 6
typedef struct __packed {
/** Offset 0x0020**/ uint32_t emmc0_mode;
@@ -21,12 +22,7 @@ typedef struct __packed {
/** Offset 0x00D0**/ uint8_t unused2[16];
/** Offset 0x00E0**/ uint8_t fch_usb_version_major;
/** Offset 0x00E1**/ uint8_t fch_usb_version_minor;
- /** Offset 0x00E2**/ uint8_t fch_usb_2_port0_phy_tune[9];
- /** Offset 0x00EB**/ uint8_t fch_usb_2_port1_phy_tune[9];
- /** Offset 0x00F4**/ uint8_t fch_usb_2_port2_phy_tune[9];
- /** Offset 0x00FD**/ uint8_t fch_usb_2_port3_phy_tune[9];
- /** Offset 0x0106**/ uint8_t fch_usb_2_port4_phy_tune[9];
- /** Offset 0x010F**/ uint8_t fch_usb_2_port5_phy_tune[9];
+ /** Offset 0x00E2**/ uint8_t fch_usb_2_port_phy_tune[FSPS_UPD_USB2_PORT_COUNT][9];
/** Offset 0x0118**/ uint8_t fch_usb_device_removable;
/** Offset 0x0119**/ uint8_t fch_usb_3_port_force_gen1;
/** Offset 0x011A**/ uint8_t fch_usb_u3_rx_det_wa_enable;