From 42b0e8f4382fc872ee07af0568dbda75602aa251 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Fri, 26 Jun 2020 18:03:53 +0200 Subject: [PATCH] soc/amd/picasso/soc_util: rework reduced I/O chip detection Both Dali and Pollock chips have less PCIe, USB3 and DisplayPort connectivity. While Dali can either be fused-down PCO or RV2 silicon, Pollock is always RV2 silicon. Since we have all boards using this code in tree right now, soc_is_dali() can be renamed and generalized to soc_is_reduced_io_sku(). Change-Id: I9eb57595da6f806305552128b0c077ceeb7c4661 Signed-off-by: Felix Held Reviewed-on: https://review.coreboot.org/c/coreboot/+/42833 Reviewed-by: Angel Pons Reviewed-by: Rob Barnes Tested-by: build bot (Jenkins) --- src/mainboard/amd/mandolin/mainboard.c | 2 +- .../google/zork/variants/baseboard/fsps_baseboard_trembyle.c | 4 ++-- src/soc/amd/picasso/include/soc/soc_util.h | 2 +- src/soc/amd/picasso/soc_util.c | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/mainboard/amd/mandolin/mainboard.c b/src/mainboard/amd/mandolin/mainboard.c index 2e66c65832..54f5775772 100644 --- a/src/mainboard/amd/mandolin/mainboard.c +++ b/src/mainboard/amd/mandolin/mainboard.c @@ -310,7 +310,7 @@ void mainboard_get_pcie_ddi_descriptors( const fsp_ddi_descriptor **ddi_descs, size_t *ddi_num) { /* Dali */ - if (soc_is_dali()) { + if (soc_is_reduced_io_sku()) { *pcie_descs = dali_pcie_descriptors; *pcie_num = ARRAY_SIZE(dali_pcie_descriptors); *ddi_descs = dali_ddi_descriptors; diff --git a/src/mainboard/google/zork/variants/baseboard/fsps_baseboard_trembyle.c b/src/mainboard/google/zork/variants/baseboard/fsps_baseboard_trembyle.c index f5f4842ed0..df42f6b87e 100644 --- a/src/mainboard/google/zork/variants/baseboard/fsps_baseboard_trembyle.c +++ b/src/mainboard/google/zork/variants/baseboard/fsps_baseboard_trembyle.c @@ -114,7 +114,7 @@ static const fsp_pcie_descriptor dali_pcie_descriptors[] = { const fsp_pcie_descriptor *baseboard_get_pcie_descriptors(size_t *num) { /* Type 2 or Type 1 fused like Type 2. */ - if (soc_is_dali()) { + if (soc_is_reduced_io_sku()) { *num = ARRAY_SIZE(dali_pcie_descriptors); return dali_pcie_descriptors; } else { @@ -176,7 +176,7 @@ static const fsp_ddi_descriptor dali_ddi_descriptors[] = { const fsp_ddi_descriptor *baseboard_get_ddi_descriptors(size_t *num) { /* Type 2 or Type 1 fused like Type 2. */ - if (soc_is_dali()) { + if (soc_is_reduced_io_sku()) { *num = ARRAY_SIZE(dali_ddi_descriptors); return dali_ddi_descriptors; } else { diff --git a/src/soc/amd/picasso/include/soc/soc_util.h b/src/soc/amd/picasso/include/soc/soc_util.h index 2459207f08..0de0643249 100644 --- a/src/soc/amd/picasso/include/soc/soc_util.h +++ b/src/soc/amd/picasso/include/soc/soc_util.h @@ -34,7 +34,7 @@ void print_silicon_type(void); void print_soc_type(void); /* function to determine the connectivity feature set */ -bool soc_is_dali(void); +bool soc_is_reduced_io_sku(void); /* function to determine the iGPU type */ bool soc_is_raven2(void); diff --git a/src/soc/amd/picasso/soc_util.c b/src/soc/amd/picasso/soc_util.c index 2aa9daa2bc..64a97a059e 100644 --- a/src/soc/amd/picasso/soc_util.c +++ b/src/soc/amd/picasso/soc_util.c @@ -199,9 +199,9 @@ void print_soc_type(void) } } -bool soc_is_dali(void) +bool soc_is_reduced_io_sku(void) { - return get_soc_type() == SOC_DALI; + return get_silicon_type() == SILICON_RV2 || get_soc_type() == SOC_DALI; } bool soc_is_raven2(void)